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

Popup sizing issues #1314

Closed
Tomalak opened this issue Sep 11, 2023 · 3 comments · Fixed by #1317
Closed

Popup sizing issues #1314

Tomalak opened this issue Sep 11, 2023 · 3 comments · Fixed by #1317

Comments

@Tomalak
Copy link

Tomalak commented Sep 11, 2023

Sometimes a vertical scrollbar appears in the collapsed view of the popup, but goes away in the expanded view. This messes with the line break position and the last line of the entry gets pushed down into the "fade-out" area, making it harder to read.

Example 餌やり

image

After clicking an entry, the expanded view is resized to fit the context menu, but upon clicking "Cancel", the size does not reset.

image

Also, the text highlight vanishes in this situation, but I feel it should probably be maintained?

I'm on Firefox 117 and extension version 1.15.1.

@birtles
Copy link
Member

birtles commented Sep 11, 2023

Hi! Thanks for filing this.

Sometimes a vertical scrollbar appears in the collapsed view of the popup, but goes away in the expanded view. This messes with the line break position and the last line of the entry gets pushed down into the "fade-out" area, making it harder to read.

Example 餌やり

Yep, that doesn't look right. What OS are you on? I can't reproduce this on Windows but it looks like you have thicker scrollbars so I guess that's why. If so, I think the solution is probably to set up a scrollbar gutter.

After clicking an entry, the expanded view is resized to fit the context menu, but upon clicking "Cancel", the size does not reset.

That part is deliberate. If we reset the popup size after you clicked cancel, your mouse might end up outside the popup causing the popup to disappear which would be annoying. We reset the popup size if it is pinned or if you're on a touch device since you don't need to worry about the popup disappearing in those cases.

Also, the text highlight vanishes in this situation, but I feel it should probably be maintained?

That seems like a separate bug. I'm not sure why that's happening.

@Tomalak
Copy link
Author

Tomalak commented Sep 11, 2023

What OS are you on?

I'm on Windows 10, I didn't think that would make much of a difference. But I do have the "Automatically hide scrollbars" system setting switched off, not sure if Firefox uses that here, but it might have an influence.

That part is deliberate.

Ah, makes sense.

That seems like a separate bug.

Yeah, but creating two issues seemed a bit excessive. :)

@birtles
Copy link
Member

birtles commented Sep 12, 2023

What OS are you on?

I'm on Windows 10, I didn't think that would make much of a difference. But I do have the "Automatically hide scrollbars" system setting switched off, not sure if Firefox uses that here, but it might have an influence.

Thanks. That seems to be it. With that setting applied and tabs turned off, I can reproduce it.

That seems like a separate bug.

Yeah, but creating two issues seemed a bit excessive. :)

It's fine to split them out. I've filed this as #1315 for now but as I mentioned there, it might not be worth fixing this since it should be fixed once Firefox ships the Highlight API on all channels.

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 a pull request may close this issue.

2 participants