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

Add caps lock indicator #692

Merged
merged 10 commits into from
May 19, 2024
Merged

Add caps lock indicator #692

merged 10 commits into from
May 19, 2024

Conversation

codokie
Copy link
Contributor

@codokie codokie commented Apr 13, 2024

I made some changes to the different shift states icons.
(note: the color of the theme is used in shifted/caps lock states, below I used the Brown color but in practice it can be whatever color is chosen in the theme settings)
unshifted state:
[material]
unshifted_material

[holo] (unchanged)
unshifted_holo

shifted state:
[material]
material_shifted

[holo] (unchaged)
shifter_holo

caps lock:
[material]
material_locked

[holo]
holo_locked

Credit to Google/Material Icons for the base icons, and @Uranusek for the ideas for the icons
Rounded theme has the same icons as the material theme, only that the icons are slightly rounded
(fixes #683)

@Uranusek
Copy link

Uranusek commented Apr 13, 2024

In my opinion, this icon does not fit other themes than Holo. What do you think about something like this?

no shift - regular icon
pressed shift - add accent
caps lock - add a second arrow above the current one

And for the Holo style, maybe add a line under the arrow when caps lock is pressed 🤔

@Uranusek
Copy link

Looks good. My idea was to change the icon only after pressing Caps Lock + adding an accent. Pressing Shift would only add an accent, not change the icon. I like the accent when I press the Shift key.

@Uranusek
Copy link

I understand. In Gboard, the icon changed when I pressed Caps Lock. I thought it would be easy to do the same in HeliBoard.

@Uranusek
Copy link

For now, I don't have a good idea for the third icon. Maybe add a second ^ above the current one 🤔

I think two icons would also be sufficient. Just like in Gboard. For most, I think it would be intuitive.

@BlackyHawky
Copy link
Contributor

Perhaps you could use a simple arrow for Caps Lock mode while keeping the bottom bar?
In any case, the double arrow for shifted state is a good idea.

Quick question: will these icons be rounded for the Rounded style?

@Uranusek
Copy link

Uranusek commented Apr 14, 2024

For Holo it's great. For the rest of the themes I would do this:

unshifted:
322207478-23b19853-f4de-4a56-bffc-b72adad288e4

shifted (with accent):
322207478-23b19853-f4de-4a56-bffc-b72adad288e4

caps lock (with accent):
322260062-7d393cd2-6e5e-4323-ab3b-a13ecf3b1b0c

@Uranusek
Copy link

Yes. I think two icons would be more intuitive. I'm a little confused when I see three icons when switching. Moreover, each icon has become different. In my opinion, this is unnecessary complication :/ It was enough to add an icon for caps lock and leave the rest as they were.

codokie added 2 commits April 14, 2024 16:33
@Uranusek
Copy link

I think this icon might be too small and not look good.
322260202-213618b4-468b-4dc5-9bd5-a0ea2afecd9b

It would be good to record what these icons look like on the keyboard. Both the current ones and those from my idea. It will be easier to decide which ones look better.

@Uranusek
Copy link

Looks nice. I wonder what the keyboard would look like with the changes I proposed 🤔

@Uranusek
Copy link

Thanks. I like my option a bit more, but both are cool.
Helium said it would be possible to customize the color of this icon in the future. Everyone will be able to choose their own, so the accent will be clearly visible.
#529

@BabyOilJohnson
Copy link

Hi, @codokie can u try my idea.
Just like the ideas of Uranusek, instead of the addition arrow replace it with another dash above the arrow.

@Helium314
Copy link
Owner

Helium314 commented May 18, 2024

Finally some basic review:
Code changes are fine, and so is using 3 icons.

But I have to say I'm not happy with the icons, mainly I just don't like the double arrows.
My suggestion would be using the single ^ for unshifted state, and

  • either the current shift icon for shifted state (but normal color), and set accent color when locked
    • I ike this more
  • or the ^ icon also for shifted state (accent color), and the current icon (with accent color) when locked
    • this is more consistent with when the bar is added below the icon

(Holo looks mostly fine, but the width and position change of the arrow when locking is somewhat weird)

codokie added 3 commits May 19, 2024 11:07
# Conflicts:
#	app/src/main/java/helium314/keyboard/keyboard/internal/KeyboardIconsSet.java
#	app/src/main/java/helium314/keyboard/keyboard/internal/keyboard_parser/KeyboardParser.kt
@Helium314
Copy link
Owner

@Helium314 thanks for the feedback! I prefer the second option you mentioned because this icon is called "Keyboard Capslock".

Using Android defaults is certainly a good reason to do it like that, thank for noticing this.

I updated the main post so you can see how it looks

Thanks, though you missed the holo shift icon (it's fine in app, I already checked)

@Helium314 Helium314 merged commit bb3fe9d into Helium314:main May 19, 2024
1 check passed
@codokie codokie deleted the shift_lock branch May 31, 2024 22:00
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.

Add caps lock indicator
5 participants