Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Breakout Rooms] UI and UX improvements #235

Merged
merged 27 commits into from
Dec 19, 2023
Merged

Conversation

dfosco
Copy link
Contributor

@dfosco dfosco commented Dec 12, 2023

Current / New

image

image

image

image

@dfosco dfosco requested a review from fredcido December 12, 2023 17:05
@dfosco dfosco requested a review from a team as a code owner December 12, 2023 17:05
Copy link

vercel bot commented Dec 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
app-examples-wordle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 19, 2023 1:01pm
webhooks-manager ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 19, 2023 1:01pm

@dfosco dfosco marked this pull request as draft December 12, 2023 17:06
@dfosco dfosco changed the title UI and UX improvements UI and UX improvements to Breakout Rooms Dec 12, 2023
@dfosco dfosco changed the title UI and UX improvements to Breakout Rooms [Breakout Rooms] UI and UX improvements Dec 13, 2023
@dfosco
Copy link
Contributor Author

dfosco commented Dec 13, 2023

I'm getting some errors on the latest commit that do not seem to break the app — @fredcido might be related to the realtime storage problems we found?

image

@dfosco dfosco enabled auto-merge (squash) December 19, 2023 12:58
@dfosco dfosco merged commit bd08223 into main Dec 19, 2023
4 checks passed
@dfosco dfosco deleted the dfosco/breakout-rooms-ui-fixes branch December 19, 2023 13:04
dfosco added a commit that referenced this pull request Jan 23, 2024
* Fix UI on empty state button

* Fix list of validations to always show missing steps

* Improve style in validations

* Make copy in user(s) splitting block dynamic

* Update 'Waiting room' copy on user selection dropdown

* UI and copy fixes in Room block

* Button size adjusted in waiting room

* Improve copy and UI for frame selection

* Fix rooms container height

* Fix copy

* Increase timer limit and add +5min option

* CSS fixes

* Remove CSS nesting

* Update toast copy

* Update examples/breakout-rooms/src/components/BreakoutManager/BreakoutManager.tsx

Co-authored-by: Mettin Parzinski <[email protected]>

* Update examples/breakout-rooms/src/components/BreakoutManager/BreakoutManager.tsx

Co-authored-by: Mettin Parzinski <[email protected]>

* Update examples/breakout-rooms/src/components/RoomConfig/RoomConfig.tsx

Co-authored-by: Frederico Estrela <[email protected]>

* Iterate frame title rendering when set to room

* Update styles

* Update relative timer rendering to change timer button text
Update finish session button variant

* Improve styles and dropdown separator

* One more style fix

* One more more style fix

* Update examples/breakout-rooms/src/components/BreakoutManager/BreakoutManager.tsx

Co-authored-by: Mettin Parzinski <[email protected]>

* Fix broken copy

* Add CSS class to avoid li+li

---------

Co-authored-by: Mettin Parzinski <[email protected]>
Co-authored-by: Frederico Estrela <[email protected]>
dfosco added a commit that referenced this pull request Jan 23, 2024
* Fix UI on empty state button

* Fix list of validations to always show missing steps

* Improve style in validations

* Make copy in user(s) splitting block dynamic

* Update 'Waiting room' copy on user selection dropdown

* UI and copy fixes in Room block

* Button size adjusted in waiting room

* Improve copy and UI for frame selection

* Fix rooms container height

* Fix copy

* Increase timer limit and add +5min option

* CSS fixes

* Remove CSS nesting

* Update toast copy

* Update examples/breakout-rooms/src/components/BreakoutManager/BreakoutManager.tsx

Co-authored-by: Mettin Parzinski <[email protected]>

* Update examples/breakout-rooms/src/components/BreakoutManager/BreakoutManager.tsx

Co-authored-by: Mettin Parzinski <[email protected]>

* Update examples/breakout-rooms/src/components/RoomConfig/RoomConfig.tsx

Co-authored-by: Frederico Estrela <[email protected]>

* Iterate frame title rendering when set to room

* Update styles

* Update relative timer rendering to change timer button text
Update finish session button variant

* Improve styles and dropdown separator

* One more style fix

* One more more style fix

* Update examples/breakout-rooms/src/components/BreakoutManager/BreakoutManager.tsx

Co-authored-by: Mettin Parzinski <[email protected]>

* Fix broken copy

* Add CSS class to avoid li+li

---------

Co-authored-by: Mettin Parzinski <[email protected]>
Co-authored-by: Frederico Estrela <[email protected]>
dfosco added a commit that referenced this pull request Jan 24, 2024
* Fix UI on empty state button

* Fix list of validations to always show missing steps

* Improve style in validations

* Make copy in user(s) splitting block dynamic

* Update 'Waiting room' copy on user selection dropdown

* UI and copy fixes in Room block

* Button size adjusted in waiting room

* Improve copy and UI for frame selection

* Fix rooms container height

* Fix copy

* Increase timer limit and add +5min option

* CSS fixes

* Remove CSS nesting

* Update toast copy

* Update examples/breakout-rooms/src/components/BreakoutManager/BreakoutManager.tsx

Co-authored-by: Mettin Parzinski <[email protected]>

* Update examples/breakout-rooms/src/components/BreakoutManager/BreakoutManager.tsx

Co-authored-by: Mettin Parzinski <[email protected]>

* Update examples/breakout-rooms/src/components/RoomConfig/RoomConfig.tsx

Co-authored-by: Frederico Estrela <[email protected]>

* Iterate frame title rendering when set to room

* Update styles

* Update relative timer rendering to change timer button text
Update finish session button variant

* Improve styles and dropdown separator

* One more style fix

* One more more style fix

* Update examples/breakout-rooms/src/components/BreakoutManager/BreakoutManager.tsx

Co-authored-by: Mettin Parzinski <[email protected]>

* Fix broken copy

* Add CSS class to avoid li+li

---------

Co-authored-by: Mettin Parzinski <[email protected]>
Co-authored-by: Frederico Estrela <[email protected]>
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