Skip to content

fix: apply dynamic time format to seekbar tooltip (#62)#64

Merged
Samillion merged 1 commit intoSamillion:mainfrom
nekoxuee:fix/dynamic-time-tooltip
Oct 20, 2024
Merged

fix: apply dynamic time format to seekbar tooltip (#62)#64
Samillion merged 1 commit intoSamillion:mainfrom
nekoxuee:fix/dynamic-time-tooltip

Conversation

@nekoxuee
Copy link
Copy Markdown
Contributor

This PR addresses the inconsistency where the seekbar tooltip wasn't following the same time format as the rest of the OSC.

Changes

  • MM:SS only when seeking positions under 1 hour (e.g., 05:30)
  • HH:MM:SS when seeking past 1-hour positions (e.g., 01:23:45)

This matches how the main time display works and follows the behavior of popular video players like YouTube.

Closes #62

- Update seekbar tooltip to match MM:SS format for sub-hour positions
- Show HH:MM:SS only when seeking past 1-hour mark
- Maintain consistency with time display formatting
@Samillion
Copy link
Copy Markdown
Owner

Works perfectly. Thank you very much.

@Samillion Samillion merged commit 9be889a into Samillion:main Oct 20, 2024
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.

Tool tip showing hours for short videos

2 participants