Skip to content

Dev3.0.14 popup#223

Open
hiroshisuga wants to merge 57 commits intov3.0.x-releasefrom
dev3.0.12_popup
Open

Dev3.0.14 popup#223
hiroshisuga wants to merge 57 commits intov3.0.x-releasefrom
dev3.0.12_popup

Conversation

@hiroshisuga
Copy link
Owner

@hiroshisuga hiroshisuga commented Aug 27, 2025

Problems remained

  • dark mode does not work (show normal mode)
  • tooltip not shown
  • depending on browser and OS, fullscreen in the same monitor does not expand the presentation (if you move to another monitor, it works). If it happens, you click the screen then tldraw realizes and the screen size has changed (but not always...).
  • the above problem is solved. Now zooming (by wheel or by clicking the zoom button) is the only problem remained. -> accidentally fixed(?) by 5726c9e, after the evaporation of cache???
  • resize by dragging the resize handles does not work (focus is lost); dom.ts in tldraw causes the problem
  • the global variable HTMLElement is injected with that of popupWindow, fixing the problem above, yet other problems to be expected.
  • style panel including the color picker does not work well. When mouse-down, the mouse-up event is not caught, behaving as if mouse is kept down when hovered over the button, picking up every color that mouse hover. The reason is in this file: tldraw-alpha30/packages/tldraw/src/lib/ui/components/primitives/ButtonPicker.tsx
  • Fonts are not properly loaded on the popup, e. g., the var(--tl-font-draw) font which is the default on latin languages.
  • Frequent 3006 error from graphql since 3.0.13 or 14 (not happening on 3.0.11)
  • Firefox suffers an error of too much recursion of font measurement when the text font is changed more than twice (or once?).

Additional features

  • disabled by default, to enable you have to toggle the parameter allowPopupPresentation.
  • on the popup, the hand icon turns red so that it would be more visible in the screen of the real lecture.
  • add Kosugi-Maru font to look a text funnier for the Japanese locale (only for popup)

@hiroshisuga hiroshisuga changed the title Dev3.0.12 popup Dev3.0.13 popup Aug 30, 2025
@hiroshisuga hiroshisuga changed the title Dev3.0.13 popup Dev3.0.14 popup Sep 8, 2025
The style changing panel is now working properly.
A warning on ignoring a graphql subscription error on the popup.
Almost all the styles in the popup had been duplicated before this change, making a dynamic style change difficult.
Merge PR bigbluebutton#23888 before the official merge. This will turn the WB tool button opaque/solid after a toolbar autohide toggle, both on the main window and on the popup.
May occur a conflict when syncing to the main branch.
probably due to the removal of duplicated style copying, or dynamic stylesheet element injection by <StyleSheetManager> which is include in ReactDOM.createPortal.
To use Kosugi-Maru also in the main window
Previously, all the draw glyphs were replaced by KosugiMaru glyphs, even though the unicode range was specified in the public/stylesheets/fonts.css.
Adding the line
    font-family: 'tldraw_draw', 'KosugiMaruSubset', sans-serif;
to the .tl-text-shape__wrapper[data-font='draw']  is critical.
Now KosugiMaru font is added at styles.css. Much simplified.
I didn't know requestAnimationFrame is actually window.requestAnimationFrame...
So that the bbb-icon fonts look a bit smaller
@github-actions
Copy link

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant