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

Fixed button blinking after clicking on it. #44

Merged
merged 2 commits into from
Mar 17, 2013
Merged

Fixed button blinking after clicking on it. #44

merged 2 commits into from
Mar 17, 2013

Conversation

Therzok
Copy link
Collaborator

@Therzok Therzok commented Mar 16, 2013

Fixed a bug where the Refresh and Browse for a Repository buttons would've continuously blinked after a click on it.

Therzok added a commit that referenced this pull request Mar 17, 2013
Fixed button blinking after clicking on it.
@Therzok Therzok merged commit ee1deaa into master Mar 17, 2013
@Therzok Therzok deleted the refreshIcon branch March 17, 2013 17:24
@Therzok
Copy link
Collaborator Author

Therzok commented Mar 17, 2013

I considered this safe to merge, so I merged it. :p

@haacked
Copy link
Owner

haacked commented Mar 18, 2013

Nice! I assume the bug was that both buttons were fighting each other for focus?

@Therzok
Copy link
Collaborator Author

Therzok commented Mar 18, 2013

No, it was never getting unfocused after click. Click one button, and it remains like that.

@haacked
Copy link
Owner

haacked commented Mar 18, 2013

Even if you clicked another button? Seems like a bug in WPF, no?

@Therzok
Copy link
Collaborator Author

Therzok commented Mar 18, 2013

Yeah, it looks like a bug in WPF, not a code bug. Searched info on it and found out it's a general button problem. When you click it, focused is set to true and haven't seen the blinking stop. Therefore, clicking Refresh would make it permanently blink.

@haacked
Copy link
Owner

haacked commented Mar 18, 2013

Interesting. Well, nice fix. Thanks!

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 this pull request may close these issues.

None yet

2 participants