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

Margin instead of padding #85

Closed
susonthapa opened this issue Feb 16, 2021 · 4 comments
Closed

Margin instead of padding #85

susonthapa opened this issue Feb 16, 2021 · 4 comments
Labels
bug Something isn't working

Comments

@susonthapa
Copy link

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

If I use the breeze window decorations then the button is only active when I hover above it.

@kupiqu
Copy link
Owner

kupiqu commented Feb 28, 2021

You're right. Would you submit a pull request?

@kupiqu kupiqu added the bug Something isn't working label Feb 28, 2021
@kupiqu
Copy link
Owner

kupiqu commented Apr 9, 2021

I'll set this as a low priority bug for now, maybe somebody picks it up.

@kupiqu
Copy link
Owner

kupiqu commented Nov 1, 2021

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...

@susonthapa
Copy link
Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants