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

Endless transparency keyboard shortcut toggles #651

Closed
thardev opened this issue Oct 26, 2015 · 1 comment
Closed

Endless transparency keyboard shortcut toggles #651

thardev opened this issue Oct 26, 2015 · 1 comment

Comments

@thardev
Copy link
Contributor

thardev commented Oct 26, 2015

Hello! I've noticed that when you use the keyboard shorcut to increase/decrease transparency if you keep pushing the key combination, the transparency goes to the higher level but, as I see in the code the transparency function that controls the level, doesn't have a limit, so you can increase it endless and the result is that when you want to decrease it doesn't works as expected...

Code:

guake/src/guake/guake_app.py

Lines 1117 to 1129 in 589131a

def accel_increase_transparency(self, *args):
"""Callback to increase transparency.
"""
transparency = self.client.get_int(KEY('/style/background/transparency'))
self.client.set_int(KEY('/style/background/transparency'), int(transparency) + 2)
return True
def accel_decrease_transparency(self, *args):
"""Callback to decrease transparency.
"""
transparency = self.client.get_int(KEY('/style/background/transparency'))
self.client.set_int(KEY('/style/background/transparency'), int(transparency) - 2)
return True

PS: It's a bit hard to explain this issue because I cannot show it with screenshots, GIF's, etc 😫

@gsemet
Copy link
Member

gsemet commented Oct 26, 2015

i understand good point !

@gsemet gsemet added this to the 0.8.4 milestone Oct 26, 2015
@gsemet gsemet closed this as completed in 4020ba1 Oct 28, 2015
gsemet added a commit that referenced this issue Oct 28, 2015
Fix #651 and hide on lose focus small changes
@gsemet gsemet modified the milestones: 0.8.4, 0.8.5 Feb 29, 2016
gsemet added a commit that referenced this issue Jan 30, 2017
Fix #651 and hide on lose focus small changes
gsemet added a commit that referenced this issue Jan 30, 2017
Fix #651 and hide on lose focus small changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants