Skip to content

Conversation

@Keith94
Copy link
Contributor

@Keith94 Keith94 commented Nov 21, 2024

Improves the hit boxes for time stamps to be more accurate, and adjusts the chapter title separator to be more consistent when time stamp width changed and for the tooltip.

Improves the hit boxes for time stamps to be more accurate, and adjusts the chapter title separator to be more consistent when time stamp width changed and for the tooltip.
@Samillion
Copy link
Owner

Thank you very much for your efforts.

By any chance you're using a custom font for mpv? I think that's why we're seeing different results.

image

@Keith94
Copy link
Contributor Author

Keith94 commented Nov 21, 2024

Actually, yeah I am. Oops. I will test again.

@Keith94
Copy link
Contributor Author

Keith94 commented Nov 22, 2024

Crap. The font was messing things up. I was using Open Sans Semi Bold.

Everything looks ok with default font. Nevermind lol.

@Keith94 Keith94 closed this Nov 22, 2024
@Keith94 Keith94 deleted the geometry_fixes branch November 22, 2024 02:55
@Samillion
Copy link
Owner

It honestly brings up a good point. One of the main reasons why I hate OSC being libass based. It is essentially tricking the system into displaying text as elements.

Which makes something as simple as changing font type a breaking change.

I agree with you that the default font isn't good, but I can't think of a good way to use a better replacement that wouldn't apply workload on the users.

Maybe one day we should discuss this in detail.

@Keith94
Copy link
Contributor Author

Keith94 commented Nov 22, 2024

Yeah, I had no idea the geometry was that delicate until you mentioned custom font.

On one hand, it's nice that people can choose their own font, but it's very much "at your own risk" because of the breakage.

We would have to either support a new, specific font type, which would break a lot of configurations, or simply leave it as is.

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