simpleMenuWizard icon indicating copy to clipboard operation
simpleMenuWizard copied to clipboard

Published 20 hours ago • verified publisher icon stonecrusher

zoom menu item in main hamburger doesn't work

Open wordpure opened this issue 1 year ago • 5 comments

After uncommenting some of the comments in the image below in main hamburger, the menu items such as printing are hidden normally, but the zoom menu item is still displayed.

图片

图片

wordpure avatar Aug 08 '22 07:08 wordpure

Strange. I just tested it with FF 103.0.1 and it works as it should. I'm sure you already restarted the browser. And toolkit.legacyUserProfileCustomizations.stylesheets is already true, otherwise it wouldn't have removed the other items.

The code you screenshotted looks good to me.

Can you test it with a fresh profile? Just go to about:profiles and make a new one. You need to install SMW there again.

stonecrusher avatar Aug 08 '22 17:08 stonecrusher

Sorry, I forgot to include my Firefox version in the last report.

I applied your suggestion in Firefox 103.0 and 104.0b6 (the version I used in the last report). 103.0 works well, but 104.0b6 still fails to hide the zoom menu item.

Perhaps this is a change brought about by a browser update?

wordpure avatar Aug 09 '22 05:08 wordpure

Yes then there will probably be a change of identifiers. This project only targets current release versions, not beta / nightly. But I'll leave this open in preparation for the change.

stonecrusher avatar Aug 09 '22 09:08 stonecrusher

hint: New ID is simply #appMenu-zoom-controls without the 2 Same goes for the toolbarseparator below

stonecrusher avatar Aug 09 '22 09:08 stonecrusher

After changing to the new id, the zoom menu item can also be hidden.

Thanks for your patient reply.

wordpure avatar Aug 09 '22 10:08 wordpure