Skip to content

refactor(ui): improve session play button and active status display #5659

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.

Sign up for GitHub

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

Jump to bottom
Merged
gustavosbarreto merged 1 commit into from
Dec 27, 2025
Merged

refactor(ui): improve session play button and active status display #5659

gustavosbarreto merged 1 commit into from
Dec 27, 2025

Conversation

@luannmoreira
Copy link
Member

@luannmoreira luannmoreira commented Dec 19, 2025 •
edited
Loading

Description

This PR improves the UI/UX of the session details page by:

  • Replacing the old "Play Session" list item with a top-level Play button.
    • Now more prominent and easier to access.
    • Uses the existing <SessionPlay> component.
  • Moving the active session status icon into the session details field area.
    • Adds clearer labeling: "Active session:".
    • Shows mdi-check-circle for active, mdi-alert-circle for inactive.
  • Removing the now-unused tooltip and redundant layout code.

Tests & Snapshots

Updated unit test to assert presence of:

  1. session-details-play-btn
  2. session-active-field

Snapshot updated to reflect the new DOM structure.

@luannmoreira
refactor(ui): improve session play button and active status display
2f46c60 
Replaces session play list item with a top-level button for visibility.
Moves active session icon into session details with clearer labeling.
Removes unused tooltip and streamlines layout.
@luannmoreira luannmoreira self-assigned this Dec 19, 2025
@luannmoreira luannmoreira requested a review from a team as a code owner December 19, 2025 19:22
@gustavosbarreto gustavosbarreto merged commit 8daa525 into master Dec 27, 2025
24 checks passed
@gustavosbarreto gustavosbarreto deleted the refactor/session-play-details-ui branch December 27, 2025 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gustavosbarreto gustavosbarreto Awaiting requested review from gustavosbarreto

Assignees

@luannmoreira luannmoreira

Labels

area/ui kind/refactor status/ready-for-testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@luannmoreira @gustavosbarreto