Skip to content

Redesign timeline UI#56

Open
Microck wants to merge 5 commits intomainfrom
feat/timeline-ui
Open

Redesign timeline UI#56
Microck wants to merge 5 commits intomainfrom
feat/timeline-ui

Conversation

@Microck
Copy link
Collaborator

@Microck Microck commented Feb 17, 2026

Description

Redesign timeline UI with modern icon-based controls.

Type of Change

  • New feature (non-breaking change which adds functionality)

Changes Made

  • Replace text buttons with SVG icons
  • Group controls visually with styled containers
  • Style numeric inputs (removed native spinners)
  • Play button with play/pause icon toggle
  • Grid/guide control buttons with icons
  • Guide mode hint chip
  • Custom scrollbar styling
  • Consistent button styling

Testing

  • Tested in Firefox

Test Configuration:

  • OS: Windows 10
  • Browser: Firefox

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have tested my changes thoroughly

Related Issues

N/A

Copy link
Collaborator

@immalloy immalloy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image these 3 buttons are the only ones that seem to not be working

@Microck
Copy link
Collaborator Author

Microck commented Mar 3, 2026

@immalloy pushed updates on this one:

  • fixed the 3 timeline controls on the right side (zoom out / zoom in / hide timeline).
  • fixed the broken timeline refresh calls (stale renderTimeline usage).
  • fixed insert/delete frame handlers using canvas contexts correctly.
  • strengthened hide/show wiring so clicks on nested icon elements still work.
  • added body-level collapse handling so timeline hide/show is reliable with current dom structure.

@Microck Microck requested a review from immalloy March 7, 2026 17:35
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.

2 participants