desktop icon indicating copy to clipboard operation
desktop copied to clipboard

Text on context menus not properly aligned

Open infiniwave opened this issue 1 year ago • 3 comments

What happened?

See images provided image image

Reproducible?

  • [X] I have checked that this issue cannot be reproduced on Mozilla Firefox.

Version

1.0.0-a.13

Severity impact

Low

What platform are you seeing the problem on?

Windows

Relevant log output

No response

infiniwave avatar Aug 07 '24 01:08 infiniwave

This doesnt appear for me, do you have any userChrome.css?

mr-cheffy avatar Aug 07 '24 19:08 mr-cheffy

No, I don't

infiniwave avatar Aug 08 '24 02:08 infiniwave

Still occurs on 1.0.0-a.22

https://github.com/user-attachments/assets/ef31abde-93d4-46b3-b8cf-be1a5ef8350f

infiniwave avatar Aug 17 '24 22:08 infiniwave

Happens to me on 1.0.1-a.5 (Linux)

image

KarimElsayad247 avatar Sep 27 '24 13:09 KarimElsayad247

Hello, any update? "Pin to toolbar" menu item seems still not aligned on v1.0.1-a.18.

Screenshot 2024-11-08 at 12 27 47

Bellisario avatar Nov 10 '24 14:11 Bellisario

@infiniwave Is this still an issue on the latest version (1.0.2-b.0)?

gnc-07 avatar Dec 11 '24 04:12 gnc-07

@gnc-07 Indeed it is: 图片 图片

infiniwave avatar Dec 11 '24 17:12 infiniwave

Was unable to recreate on a fresh profile in b.2. For everyone who is still facing the issue, could you try on a new profile? This will help you move ur data to a new profile if you need.

salamijason avatar Dec 15 '24 08:12 salamijason

Was still able to reproduce my issue https://github.com/zen-browser/desktop/issues/177#issuecomment-2466762069 of "Pin to Toolbar" misalignment on the latest Zen Browser version (1.0.2-b.2).

To reproduce the issue:

  • create a new browser profile
  • follow the initial configuration (I chose "Multiple toolbars")
  • install an extension (ex. uBlock Origin)
  • install another extension (to enable "Extensions" button) (ex. Dark Reader)
  • click on the "Extensions" button
  • click on the three dots of any extension in here
  • see that "Pin to Toolbar" text is misaligned and goes on the pin icon

Bellisario avatar Dec 15 '24 11:12 Bellisario

was able to reproduce on macOS b.2, thank you for all the input! image

salamijason avatar Dec 15 '24 17:12 salamijason

It's there on a build based on a9fb5981343ac1fd382d7cec2b255550a0635ef4 on main branch. I could only see the problem on linux, on windows the context menus look fine.

image

The problem can't be reproduced on Firefox 133 since it doesn't have context menu icons.

KarimElsayad247 avatar Dec 17 '24 05:12 KarimElsayad247

Some specifics:

  • Only extension icons are non-aligned
  • Correctly positioned items have a padding-left of 38px, while the mis-positined ublock item has a padding left of just 5px, but that obviously isn't just a 30px difference. Here's what it looks like if I give it the same 38px padding image

So there is definitely something funky going on with the way menu items are layed here.

KarimElsayad247 avatar Dec 17 '24 05:12 KarimElsayad247

ahhhhh, the other icons are positioned absolutely, that's why they behave differently

image

KarimElsayad247 avatar Dec 17 '24 06:12 KarimElsayad247

So this 38px padding is the padding for the text, not icon

KarimElsayad247 avatar Dec 17 '24 06:12 KarimElsayad247

It now looks good once I give the icon absolute positioning in line with the other icons, and remove the left padding on text

image

KarimElsayad247 avatar Dec 17 '24 06:12 KarimElsayad247

HOWEVER, this still leaves checkbox menu items broken. Still need investigation

image

KarimElsayad247 avatar Dec 17 '24 06:12 KarimElsayad247

Though I guess the fix would be similar, just remove the checkbox from normal flow and absolutely position it.

KarimElsayad247 avatar Dec 17 '24 06:12 KarimElsayad247

Though I do have to ask.... @mr-cheff Why are the icons absolutely positioned instead of simply being positioned normally?

KarimElsayad247 avatar Dec 17 '24 06:12 KarimElsayad247

Is it fixed now?

mr-cheffy avatar Dec 27 '24 19:12 mr-cheffy

Though I do have to ask.... @mr-cheff Why are the icons absolutely positioned instead of simply being positioned normally?

One of those things we inherit from firefox

mr-cheffy avatar Dec 27 '24 19:12 mr-cheffy

Is it fixed now?

Same "Pin to Toolbar" issue on 1.0.2-b.5 for me.

Bellisario avatar Dec 28 '24 13:12 Bellisario

Nope, same on b.5

image

KarimElsayad247 avatar Dec 31 '24 08:12 KarimElsayad247

just to clarify, the text is now properly aligned and the only issue left is the icons? cheff mentioned above that the weird icon alignment is inherited from ff. please let me know if i missed something. will be closing issue now.

salamijason avatar Jan 23 '25 18:01 salamijason

@salamijason seems my issue is still present on the latest beta:

Hello, any update? "Pin to toolbar" menu item seems still not aligned on v1.0.1-a.18.

Screenshot 2024-11-08 at 12 27 47

Bellisario avatar Jan 23 '25 19:01 Bellisario

@Bellisario strange, I can't reproduce that. Could you try a new profile?

salamijason avatar Jan 25 '25 07:01 salamijason

@salamijason yes, I just tried on a new profile and it seems the issue is still present.

These are the steps I took (don't know if it's relevant but I'm using a Mac):

  • create a new profile from about:profiles and open it
  • setup Zen to use multiple toolbars (leaving everything else as default)
  • after setup complete, open "Add-ons manager" by clicking on the "extensions" button
  • install two extensions (I chose uBlock Origin and Dark Reader)
  • make sure only one of the two is pinned (it won't show the bug if there are no "unpinned" extensions) (in my case Dark Reader wasn't pinned by default)
  • click on the "extensions" button, then on the three dots of an extension there (Dark Reader for me)
  • see that the "Pin to toolbar" button is over the pin icon
Image

Bellisario avatar Jan 25 '25 10:01 Bellisario

Seems to be fixed, may we get this closed?

12th-devs avatar Feb 23 '25 12:02 12th-devs

Sure, could I get a confirmation from OP @infiniwave that this issue is fixed? Thanks

salamijason avatar Feb 23 '25 14:02 salamijason

I don't know if this counts but in the account part it still seems to be a bit off (Like the "Manage Account" and "Sync now"):

Image

dev-adal avatar Feb 26 '25 15:02 dev-adal

And this is just a bit weird why task manager is the only item with an icon might not be a bug but rather not nice:

Image

I apologize for the two posts if they sent multiple emails. ❤

dev-adal avatar Feb 26 '25 15:02 dev-adal