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

No spaces between the buttons in many themes #54

Open
nana-4 opened this issue May 30, 2018 · 3 comments
Open

No spaces between the buttons in many themes #54

nana-4 opened this issue May 30, 2018 · 3 comments

Comments

@nana-4
Copy link

nana-4 commented May 30, 2018

Expected behaviour

mate-calc has spaces between the buttons, like gnome-calculator:

image

Actual behaviour

No spaces between the buttons in many themes, e.g.

Adwaita:
image

BlackMATE:

image

Arc:

image

Materia:

image

Please note that a few themes like Menta already have spaces because they use border-image instead of border for the button spacing.

Steps to reproduce the behaviour

Change the theme.

MATE general version

1.20.1

Package version

1.20.1

Linux Distribution

Ubuntu 18.04

Link to downstream report of your Distribution

This was originally reported here: nana-4/materia-theme#237

@raveit65
Copy link
Member

Why is this an error?
Only because a gnome app looks like this.
Personal i do not agree with you.

@nana-4
Copy link
Author

nana-4 commented May 30, 2018

This is not an error, but in general it is considered that the double border is not good looking. Not only in GNOME project (e.g. KDE, Windows, macOS, Web, etc.), usually there is space between the (raised) buttons.

But if you don't like this approach, please feel free to close this issue.

@vkareh
Copy link
Member

vkareh commented May 30, 2018

Oh wow, I hadn't even noticed the double border until now. Unless there is an accessibility issue with the buttons, I'd say this is a theme designer decision.

That materia theme really looks awful on the calc! That's not our theme though...

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