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

issues with theme on macos #1

Closed
linusrg1 opened this issue Jan 30, 2022 · 78 comments
Closed

issues with theme on macos #1

linusrg1 opened this issue Jan 30, 2022 · 78 comments
Labels
wontfix This will not be worked on

Comments

@linusrg1
Copy link

I think to to the nature of the placementof mac os's title bar buttons the theme has trouble functioning on macos firefox. The menu bar button is on the left of the firefox window for some reason, underlaying the buttons. Otherwise the theme looks great!

@Godiesc
Copy link
Owner

Godiesc commented Jan 30, 2022

I don't have a MacOS to test the theme, so its so difficult to me fix o make compatible with that OS, Sorry. 💙 For now you could delete the ogx_menu.css file and the theme will works fine.

@Godiesc Godiesc added the wontfix This will not be worked on label Feb 22, 2022
@FormalSnake
Copy link
Contributor

I don't have a MacOS to test the theme, so its so difficult to me fix o make compatible with that OS, Sorry. 💙 For now you could delete the ogx_menu.css file and the theme will works fine.

This is pretty annoying, please fix.
Screenshot 2022-10-11 at 13 56 20

@FormalSnake
Copy link
Contributor

And the tabs get overlapped with the ogx_menu.css removed
Screenshot 2022-10-11 at 14 03 35

@Godiesc
Copy link
Owner

Godiesc commented Oct 11, 2022

you could try adding this code to the end of userChrome.css.

#tabbrowser-tabs{
  padding-inline-start: 24px !important;
}

try a smaller or bigger pixels number to look it well to you.
PD: delete the lines: 58-72 from ogx_left-sidebar.css if you using that config, if not just the first code will be necessary.

@FormalSnake
Copy link
Contributor

Thanks, i will try it.
Opera GX
Screenshot 2022-10-11 at 16 40 49
Firefox GX
Screenshot 2022-10-11 at 16 40 59

Maybe there is a way to move the window controls and stuff, and there is like no spacing between the window borders and the tabs, other than that it is perfect!

@Godiesc
Copy link
Owner

Godiesc commented Oct 11, 2022

like I said I don´t have that OS to play with the css to make it compatible, I just can give you some code to try solve a little things, to make a space in the top add this too:

#TabsToolbar{
  padding-block-start: 6px !important;
}

And for the window controls I give up jeje, I would need that OS to test.

@FormalSnake
Copy link
Contributor

like I said I don´t have that OS to play with the css to make it compatible, I just can give you some code to try solve a little things, to make a space in the top add this too:

#TabsToolbar{
  padding-block-start: 6px !important;
}

And for the window controls I give up jeje, I would need that OS to test.

I might actually research that myself, might make a pull request some day, i am going to edit the config now!

@Godiesc
Copy link
Owner

Godiesc commented Oct 11, 2022

that would be perfect.

@FormalSnake
Copy link
Contributor

that would be perfect.

I am learning chromeCSS right now, i know how to move the buttons from left to right, but no vertical layout yet.

@Godiesc
Copy link
Owner

Godiesc commented Oct 11, 2022

for default in macOS that buttons stay in the left, are you changing them to the right?

@Godiesc
Copy link
Owner

Godiesc commented Oct 12, 2022

you could try this code to make the window controls vertical:

@media (-moz-platform: macos) {:root:is([tabsintitlebar]) .titlebar-buttonbox{
    display: flex !important;
    flex-direction: column !important;
}}

@FormalSnake
Copy link
Contributor

you could try this code to make the window controls vertical:

@media (-moz-platform: macos) {:root:is([tabsintitlebar]) .titlebar-buttonbox{
    display: flex !important;
    flex-direction: column !important;
}}

That did not work, but i can try to modify this code

@Godiesc
Copy link
Owner

Godiesc commented Oct 12, 2022

do you use the developer console? In the first post of reddit/r/firefoxcss has the steps to enable. then you can select a element with the selector and see the id and classes to edit with css.
and if you apply only this part of the code still nothing happen?

.titlebar-buttonbox{
    display: flex !important;
    flex-direction: column !important;
}

@FormalSnake
Copy link
Contributor

do you use the developer console? In the first post of reddit/r/firefoxcss has the steps to enable. then you can select a element with the selector and see the id and classes to edit with css. and if you apply only this part of the code still nothing happen?

.titlebar-buttonbox{
    display: flex !important;
    flex-direction: column !important;
}

i am using the dev console, ill try just that out

@FormalSnake
Copy link
Contributor

Update: if i try changing the background color, it does nothing, only in the buttonbox container

@Godiesc
Copy link
Owner

Godiesc commented Oct 12, 2022

imagen
for example I used that class to edit the code I sent you and has this change:

imagen

DONT FORGET TO ADD !IMPORTANT; TO THE END OF EACH CSS RULE.

@FormalSnake
Copy link
Contributor

On mac this doesn't work, as you can see, i added the CSS
Screenshot 2022-10-12 at 06 26 19

And this is the result:
Screenshot 2022-10-12 at 06 26 39

@Godiesc
Copy link
Owner

Godiesc commented Oct 12, 2022

show me how looks you selecting that window controls with the console. like this image:
imagen

@FormalSnake
Copy link
Contributor

I did have to clear all my tabs in the css to be able to select it.
Screenshot 2022-10-12 at 06 34 58

@Godiesc
Copy link
Owner

Godiesc commented Oct 12, 2022

if you try to give a max-width and min-width to 40px for example and a bigger height maybe could work ejje, well, it my time to sleep, maybe you could play with some rules to see if apply.

@FormalSnake
Copy link
Contributor

Buenas noches

@FormalSnake
Copy link
Contributor

.titlebar-buttonbox {
  display: flex !important;
  flex-direction: column !important;
  max-width: 40px !important;
  min-width: 40px !important;
  height: 60px !important;
  background-color: red !important;
}

He conseguido mover los botones, pero no se ponen en column
Screenshot 2022-10-12 at 13 56 50

@Godiesc
Copy link
Owner

Godiesc commented Oct 12, 2022

the first tab always was so near to the border of the windows?
and if you use background: red instead of background-color makes a difference?

@FormalSnake
Copy link
Contributor

the first tab always was so near to the border of the windows? and if you use background: red instead of background-color makes a difference?

I removed the padding tab rule to test out if it was conflicting in a way
and the background: red did nothing

@Godiesc
Copy link
Owner

Godiesc commented Oct 12, 2022

with the developer console in the right side after selected that element are you tried to uncheck some css rule to see what happens?

@FormalSnake
Copy link
Contributor

with the developer console in the right side after selected that element are you tried to uncheck some css rule to see what happens?

¿? ¿What element? ¿And what rule?

@Godiesc
Copy link
Owner

Godiesc commented Oct 12, 2022

would be perfect to use teamviewer or some app to test in your computer jeje
and I mean the default css rules that are in the default Firefox theme:

imagen

and try to add: appearance: none !important; maybe the OS has control of that style and for that all the css rules we are trying are failing.

@FormalSnake
Copy link
Contributor

Yeah, maybe we should call on discord some day

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

I tried this code in linux and works, maybe this could work in macOS:

/* Linux Controls */
@media (-moz-platform: macos){ #tabbrowser-tabs{
  padding-inline-start: calc(var(--uc-vertical-toolbar-width) - 1px) !important;
    }
}
@media (-moz-platform: macos){ .titlebar-button:hover {
    appearance: none !important;
    background-color: transparent !important;
    }
}
@media (-moz-platform: macos) { .titlebar-buttonbox {
    appearance: none !important;
    position: fixed !important;
    display: flex !important;
    top: 0px !important;
    left: 0px !important;
    width: calc(var(--uc-vertical-toolbar-width) - 1px) !important;
    flex-direction: column !important;
    background: var(--lwt-frame) !important;
    direction: ltr !important;
}
}
@media (-moz-platform: macos){ .titlebar-button image{ height: 26px !important; width: 26px !important; }
    
@media (-moz-platform: macos){ .titlebar-close image { list-style-image: url("../images/closebuttonlinux.svg") !important; } }
@media (-moz-platform: macos){ .titlebar-close:hover image{ list-style-image: url("../images/closebuttonhoverlinux.svg") !important; } }
    
@media (-moz-platform: macos){ .titlebar-min image { list-style-image: url("../images/minbuttonlinux.svg") !important; } }
@media (-moz-platform: macos){ .titlebar-min:hover image{ list-style-image: url("../images/minbuttonhoverlinux.svg") !important; } }
    
@media (-moz-platform: macos){ .titlebar-max image { list-style-image: url("../images/maxbuttonlinux.svg") !important; } }
@media (-moz-platform: macos){ .titlebar-max:hover image{ list-style-image: url("../images/maxbuttonhoverlinux.svg") !important; } }
    
@media (-moz-platform: macos){ .titlebar-restore image { list-style-image: url("../images/maxbuttonlinux.svg") !important; } }
@media (-moz-platform: macos){ .titlebar-restore:hover image{ list-style-image: url("../images/maxbuttonhoverlinux.svg") !important; } }

PD: The code that change the icons isn't necessary.

imagen

@FormalSnake
Copy link
Contributor

One moment, i realised i have a bug, easy fix

@FormalSnake
Copy link
Contributor

Step 1: Remove OGX_MENU.css
Step 2: Add this code where it is needed:

@media (-moz-platform: macos) {#TabsToolbar{
  padding-block-start: 6px !important;
}
#tabbrowser-tabs{
  padding-inline-start: 10px !important;
}
}

@FormalSnake
Copy link
Contributor

Here you go!
Screenshot 2022-10-13 at 18 16 30
Screenshot 2022-10-13 at 18 17 13

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

please you could update the ogx_left-sidebar.css with the one that I updated in the github page, I did a little update to fix the menu button when the ogx_menu.css is deleted, and after that pls give me other SS, sorry for the inconvenient.

@FormalSnake
Copy link
Contributor

what did it fix? i see no difference

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

the hamburger menu dont show a black background by default. in your SS see that background.

@FormalSnake
Copy link
Contributor

oh yes i see it now

@FormalSnake
Copy link
Contributor

Screenshot 2022-10-13 at 18 35 34

Screenshot 2022-10-13 at 18 36 15

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

thanks for your help, sad that the window control buttons didn't make it work jeje. I appreciate your time.

@FormalSnake
Copy link
Contributor

thanks for your help, sad that the window control buttons didn't make it work jeje. I appreciate your time.

No problem, im happy to help when you need me!

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

one last questions, the top padding we added is for maximized and normal windows right? you deleted that part of the code cause that only applied in maximized window?

@FormalSnake
Copy link
Contributor

yeah, if i put it in fullscreen, the tabs go to the edge of the window, but it can be fixed on your end, and if i put it in maximized, in most firefox themes the padding goes away, which is also fixed here now.

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

Step 1: Remove OGX_MENU.css Step 2: Add this code where it is needed:

@media (-moz-platform: macos) {#TabsToolbar{
  padding-block-start: 6px !important;
}
#tabbrowser-tabs{
  padding-inline-start: 10px !important;
}
}

I will add the platform: macos in the second rule, you forgot to added jeje

@media (-moz-platform: macos) { #TabsToolbar{
  padding-block-start: 6px !important;
}
@media (-moz-platform: macos) { #tabbrowser-tabs{
  padding-inline-start: 10px !important;
}
}

@FormalSnake
Copy link
Contributor

lol, there is another issue, related with fonts
Uploading Screenshot 2022-10-13 at 18.52.15.png…

@FormalSnake
Copy link
Contributor

Screenshot 2022-10-13 at 18 52 15

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

I updated ogx_contextual-menu.css , check it to update that file and make me know if that solves the problem.

@FormalSnake
Copy link
Contributor

ok, ill check it out

@FormalSnake
Copy link
Contributor

it looks a looooott better now

@FormalSnake
Copy link
Contributor

identical with opera GX

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

glad to know it. :)

@FormalSnake
Copy link
Contributor

ill report any further issues, im glad to have worked with you! Can i update the readme.md so that i appear as a contributor?

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

to do that you can fork my repository and after that edit the file ogx_tabs-bar.css and add to the end of the file:

/* --------------------- config to MacOS ---------------------------
   ------------- partial compatibility to window Controls ----------*/
@media (-moz-platform: macos) { #TabsToolbar{
  padding-block-start: 6px !important;
}
@media (-moz-platform: macos) { #tabbrowser-tabs{
  padding-inline-start: 10px !important;
}
}

And after that you make a pull request, I'll accept and after that you'll see you as a contributor.

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

or I put your name in the MacOS section, an easy way.

@FormalSnake
Copy link
Contributor

i made a pr, and maybe adding a name that links to my profile would be nice, it is the first time i contribute to a project

@FormalSnake
Copy link
Contributor

Thanks a lot! I am honored

@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

it's done now. ❤

@Godiesc Godiesc closed this as completed Oct 13, 2022
@Godiesc
Copy link
Owner

Godiesc commented Oct 13, 2022

and other thing, the close button in the favicon maybe will interfere with the sound icon button, I dont know how opera work with sound tabs.

@FormalSnake
Copy link
Contributor

and other thing, the close button in the favicon maybe will interfere with the sound icon button, I dont know how opera work with sound tabs.

I will make another issue for this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants