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

[bug] hide window buttons #57

Closed
nkapila6 opened this issue Aug 18, 2024 · 7 comments · Fixed by #59
Closed

[bug] hide window buttons #57

nkapila6 opened this issue Aug 18, 2024 · 7 comments · Fixed by #59

Comments

@nkapila6
Copy link

This theme does not work.

I'm currently on macOS and the picture on the theme page looks like it was made for Windows. Could this be a reason?

image
@mauro-balades
Copy link
Member

Are these buttons injected by macos or by Firefox? @n7itro

@n7itro
Copy link
Member

n7itro commented Aug 18, 2024

It seems like MacOS does not allow individually removing the buttons. I suggest we change the themes description to specify it is for Windows (and Linux?) only and add another theme for MacOS.

@mauro-balades
Copy link
Member

I wonder if it's like native or if it can be inspected though

@n7itro
Copy link
Member

n7itro commented Aug 18, 2024

We would need a mac user to try... It's most likely just one element because I've heard that the following does work for macOS
.titlebar-buttonbox-container { display: none }. For now, I'll create a separate theme for mac

@nkapila6
Copy link
Author

nkapila6 commented Aug 18, 2024

Yeah, I was going to comment the same thing.

@media (-moz-mac-big-sur-theme: 1) {
  hbox.titlebar-buttonbox-container { display: none !important; }
}

This works.

image

@n7itro
Copy link
Member

n7itro commented Aug 18, 2024

Yeah, I was going to comment the same thing.

@media (-moz-mac-big-sur-theme: 1) {
  hbox.titlebar-buttonbox-container { display: none !important; }
}

This works.

I can add this to the theme so it removes the window control buttons automatically for mac users. The options won't do anything on mac though. Thank you!

@mauro-balades
Copy link
Member

This will be fixed with the addition of https://docs.zen-browser.app/themes-store/themes-marketplace-preferences#operating-system-specific-settings in the next release 👍🏽

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.

3 participants