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

macOS: Tab close button is on the wrong side #128

Closed
phedlund opened this issue Jul 7, 2017 · 9 comments
Closed

macOS: Tab close button is on the wrong side #128

phedlund opened this issue Jul 7, 2017 · 9 comments
Assignees
Labels
Component-GUI Status-Partially-Fixed Part of bug/feature is fixed/implemented. Type-Enhancement This is request for brand new feature.

Comments

@phedlund
Copy link
Contributor

phedlund commented Jul 7, 2017

Brief description of the issue.

Tab close buttons are placed on the right instead of the left side

How to reproduce the bug?

  1. Open an article in a tab.
  2. The close button is on the right side

What is the expected result?

On macOS close buttons are expected to be on the left side

What actually happened?

The close button is on the right side

Other information (logs, see Wiki)

screen shot 2017-07-07 at 10 28 02 am

@martinrotter
Copy link
Owner

Those buttons are drawn manually on the right side on all platforms, I will check what I can do.

@martinrotter martinrotter self-assigned this Jul 8, 2017
@martinrotter martinrotter added Component-GUI Status-Accepted I will eventually implement this. Type-Enhancement This is request for brand new feature. labels Jul 8, 2017
@martinrotter martinrotter added Status-Fixed Ticket is resolved. and removed Status-Accepted I will eventually implement this. labels Jul 8, 2017
@martinrotter
Copy link
Owner

Probably fixed, please check.

@phedlund
Copy link
Contributor Author

phedlund commented Jul 9, 2017

Now drawn on the left, but there is an overlap between the favicon and the title. A fancy way of doing this would be to have the favicon turn into a close button when hovering over it with the mouse.

screen shot 2017-07-09 at 2 24 36 pm

@martinrotter
Copy link
Owner

Oh, I see. I will check this out. Actually it is not totally bad idea to turn X icon into favicon if there is one. But it would make sense probably only on Mac OS X because on other platforms this Qt bug is not there, since X buttons are on the right side.

I believe that this is actually Qt bug, and not RSS Guard bug but I will check.

@martinrotter martinrotter reopened this Jul 10, 2017
@martinrotter martinrotter added Status-Partially-Fixed Part of bug/feature is fixed/implemented. and removed Status-Fixed Ticket is resolved. labels Jul 10, 2017
@martinrotter
Copy link
Owner

Yes, confirmed Qt bug - https://bugreports.qt.io/browse/QTBUG-61742?jql=text%20~%20%22tab%20icon%20button%22

It seems they have much trouble with icon/button/title of tab rendering.

Maybe. Could it "fix" the situation if the title of the tab would contain some spaces in its beginning? It would probably create some space between the icon and title.

martinrotter pushed a commit that referenced this issue Jul 10, 2017
@martinrotter
Copy link
Owner

Wait for c623da6 gets compiled and then please test, I added two spaces before tab title.

@martinrotter
Copy link
Owner

Confirm fixed pls.

@phedlund
Copy link
Contributor Author

Yes, this fixes the overlap for now.

martinrotter pushed a commit that referenced this issue Jul 13, 2017
@martinrotter
Copy link
Owner

OK, I will close this. I also fixed the title for rest of use-cases (tab "Downloads") etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component-GUI Status-Partially-Fixed Part of bug/feature is fixed/implemented. Type-Enhancement This is request for brand new feature.
Projects
None yet
Development

No branches or pull requests

2 participants