Skip to content

feat: add poll shape#16

Merged
antobinary merged 24 commits into
bigbluebutton:mainfrom
danielpetri1:poll-shape
Sep 18, 2024
Merged

feat: add poll shape#16
antobinary merged 24 commits into
bigbluebutton:mainfrom
danielpetri1:poll-shape

Conversation

@danielpetri1
Copy link
Copy Markdown

When BigBlueButton displays a poll on the canvas, the client currently extends Tldraw with a custom shape. As a result, only the Tldraw instances tied to bbb-html5 profit from the functionality, despite other components, such as bbb-playback, bbb-export-annotations, and bbb-presentation-video needing to display such a poll.

This PR is a stepstone towards centralizing the poll shape directly in Tldraw, which would close the following issues:

Development plan

  1. Add the custom shape to the Tldraw fork

  2. Change BBB 3.0 core to include the poll shape data in tldraw.json for bbb-playback
    2.1. This automatically makes the new polls shapes appear in the recordings

  3. Remove duplicated custom shape from BBB's client, using the new implementation instead (we are here)

  4. Change Tldraw to include an image representation in the shape's meta parameter

  5. Use the poll image in the annotation export

  6. Use the poll image in the video

For the poll image, we can follow the same approach used by the slide snapshot: bigbluebutton/bigbluebutton#21005

Demo

tldraw.polls.mov

Tainan404 pushed a commit that referenced this pull request May 20, 2026
* fix type and build errors

* update GitHub Actions runner to 4 cores for better availability

* update GitHub Actions runner for better availability

* fix linter and lint issues

* fix tests / disable broken tests

* add types

* test update type

* test update type

* test update type

* test update type

* fix playwright e2e tests

* remove unused config

* update config

* Add poll shape

* Revert tsconfig

* Use relative path for TLUiTranslationKey

* Import defineMigration via relative pth

* Run Prettier

* Yarn lint

* Fix yarn lint - disable extraneous dependencies

* Add public-yarn lock

* Fix webpack load issue

* Allow whiteboardID parameter

---------

Co-authored-by: KDSBrowne <kert.browne85@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants