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

Keyboard UX/UI: Access to caps lock - clearer icons #34362

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

ugtthis
Copy link
Contributor

@ugtthis ugtthis commented Jan 12, 2025

Was reminded again to do this once I saw the comment from discord
discord-double-tap


Comparison

Current Shift Key UI/UX Proposed Shift Key UI/UX
current-caps-key proposed-caps-key

Proposed UI/UX Enlarged

proposed-caps-key

@github-actions github-actions bot added the ui label Jan 12, 2025
@ugtthis ugtthis changed the title Keyboard UX/UI: Double tap for all caps - clearer caps icons Keyboard UX/UI: Double tap for all caps - clearer icons Jan 12, 2025
Copy link
Contributor

github-actions bot commented Jan 12, 2025

UI Preview

keyboard : $${\color{red}\text{DIFFERENT}}$$
master proposed
diff composite diff
keyboard_uppercase : $${\color{red}\text{DIFFERENT}}$$
master proposed
diff composite diff
All Screenshots

@ugtthis ugtthis marked this pull request as ready for review January 12, 2025 07:38
@ugtthis
Copy link
Contributor Author

ugtthis commented Jan 16, 2025

Now it's a simpler implementation. I do like the double tap feature cause I think it is the better UX solution but it's not yet that important especially since there is a simpler solution that solves the main problem, which is to have access to caps lock

New UX - no double tap

no-more-double-tap

@ugtthis ugtthis changed the title Keyboard UX/UI: Double tap for all caps - clearer icons Keyboard UX/UI: Access to caps lock - clearer icons Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants