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

Fix caps warning occuring when pressing caps on Login section #2237

Closed
wants to merge 1 commit into from
Closed

Fix caps warning occuring when pressing caps on Login section #2237

wants to merge 1 commit into from

Conversation

Frontear
Copy link
Contributor

@Frontear Frontear commented Mar 18, 2018

No longer will the caps warning appear except for when you are focused on the password box. This closes #2234

No longer will the caps warning appear except for when you are focused on the password box.
@peppy
Copy link
Sponsor Member

peppy commented Mar 19, 2018

Thanks for the PR! I've submitted an alternative fix which addresses the issue at the source (textbox shouldn't be handling key input in the first place) rather than adding another conditional.

Take a look over at ppy/osu-framework#1471.

@smoogipoo
Copy link
Contributor

Closed by referencing PR.

@smoogipoo smoogipoo closed this Mar 20, 2018
@Frontear Frontear deleted the fix-caps-warning branch March 21, 2018 17:13
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.

Caps lock warning should only appear when typing password
3 participants