-
Notifications
You must be signed in to change notification settings - Fork 34
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
Margin instead of padding #85
Comments
You're right. Would you submit a pull request? |
I'll set this as a low priority bug for now, maybe somebody picks it up. |
My aim is that SBE is maintained by the community. This shouldn't be hard to fix. I hope someone in the community cares about SBE and submits a pull request about it... |
I would be more than happy to contribute. But I don't know much about QT as I'm a mobile developer. Still, If I get some free time I will look into it. |
I think it would be better to give option to set margin instead of padding. When I set the padding (which I have to, in order to get the required height of the title bar) then the button activates when my cursor is near the border between title bar and the application. This causes accidental clicks when I try access menus in the application.
I couldn't actually capture the cursor but I hope you will get the idea
![image](https://user-images.githubusercontent.com/33973551/108009450-dcc88c00-702a-11eb-839a-05366772cc2a.png)
If I use the breeze window decorations then the button is only active when I hover above it.
The text was updated successfully, but these errors were encountered: