Scribe-Android icon indicating copy to clipboard operation
Scribe-Android copied to clipboard

Improve non-hold key callouts

Open andrewtavis opened this issue 3 years ago • 18 comments

Terms

Description

This issue is to improve the key callouts that pop up when a key is pressed. As of now the callouts don't cover the key when they're pressed, with the user generally not seeing this as their thumb is covering the key, but the behavior is not correct based on comparisons to system keyboards.

Ideally there would be a full callout that covers the key, which then would transition to the hold callout if the user is holding a key to select alternate characters.

[!NOTE] We also need to match the color of the key callout to that of the key.

Contribution

This issue is thus of low priority as of now, but could be worked on later when the callouts are being investigated further. Others are welcome to contribute!

andrewtavis avatar May 18 '22 18:05 andrewtavis

@andrewtavis Does this issue still exist? Could you explain in details, like a screenshot?:)

Linfye avatar Nov 30 '24 13:11 Linfye

Will send something along later, @Linfye, but yes it does! :)

andrewtavis avatar Nov 30 '24 14:11 andrewtavis

In looking at it, there are a lot of issues with this functionality at this point, @Linfye, and it would be great if you could take a look at it :)

All of this is activated by Show popup on keypress in the settings, or at least the base popup. You can see that in dark mode the key popup is not colored appropriately:

Screenshot 2024-11-30 at 15 23 04 (2)

Beyond that is the original reason for this issue, which is that the callout there is a bit messy. It'd be great if the callout looks more like it does in the iOS app where it visually appears that the callout is coming from the key, not just hovering over it:

Screenshot 2024-11-30 at 15 29 56 (2)

Beyond that, the hold key for alternate characters functionality seems to have been broken along the way, or isn't working well in dark mode. See the image below:

Screenshot 2024-11-30 at 15 23 22 (2)

The above is me holding the A key, but then there's no background behind the options to make it clear that if I swipe I can select one of the alternate A characters.

Let me know if you'd have interest in looking into any of the above! We could start with one PR to fix the color of the base callout in dark mode and make the hold to select callout visible again, and then we could do another PR to improve the callout itself 😊 Feel free to indicate that you have interest, and from there you can get started and I'll assign you later 🚀

andrewtavis avatar Nov 30 '24 14:11 andrewtavis

Yes, I'd like to work on this. Thanks for your explaination. :)

Linfye avatar Nov 30 '24 15:11 Linfye

Nice, @Linfye! Let us know if you have any questions :)

andrewtavis avatar Nov 30 '24 16:11 andrewtavis

Do you want to work on a fix for the long hold accent character selection background now, @Linfye? So a fix for the following above where the optional characters have no background:

Screenshot 2024-11-30 at 15 23 22 (2)

andrewtavis avatar Dec 02 '24 21:12 andrewtavis

Yes 😁

Linfye avatar Dec 03 '24 06:12 Linfye

Hey @Linfye 👋 Wanted to check in and see if you still had interest in working on this issue :) Please let us know if you have any questions. Hope all's well!

andrewtavis avatar Feb 06 '25 20:02 andrewtavis

Yes, but I've been quite busy lately. I plan to continue the scribe-related work after returning to college in March.

If anyone else is willing to take it over, they can do so. :)

Linfye avatar Feb 09 '25 12:02 Linfye

Sounds good, @Linfye! Looking forward to having you back, and all the best till then!

andrewtavis avatar Feb 09 '25 12:02 andrewtavis

Just added the following to the issue description above:

[!NOTE] We also need to match the color of the key callout to that of the key.

Right now the color's a bit off :)

andrewtavis avatar Feb 26 '25 00:02 andrewtavis

I'm trying to clarify the current issue.:

  1. The callout should look like it comes from the key.
  2. There's no backgroud after long pressing the key.
  3. The color's a bit off. (Could you tell me more about this one?)

Linfye avatar Mar 07 '25 13:03 Linfye

I'm trying to replicate the effect of the iOS version, but it's quite challenging (I can't draw such a trapezoid very well). Maybe we can take inspiration from the current methods used by Gboard?

Linfye avatar Mar 07 '25 13:03 Linfye

Thanks for the clarifying questions, @Linfye :) All your deductions are correct, and the color for dark mode is the light mode background, I believe.

How is Gboard doing this?

andrewtavis avatar Mar 07 '25 15:03 andrewtavis

Image Image

As shown in the screenshots, I think this is an intuitive and convenient way.

Linfye avatar Mar 08 '25 10:03 Linfye

CC @angrezichatterbox on the potential design choices here :) Ideally we would have a similar design across the apps, but if it can't work I'm fine with going in a similar direction to Gboard 😊

andrewtavis avatar Mar 08 '25 11:03 andrewtavis

CC @angrezichatterbox on the potential design choices here :) Ideally we would have a similar design across the apps, but if it can't work I'm fine with going in a similar direction to Gboard 😊

It would be great to have a consistent design across both Android and iOS, but Android has traditionally never followed that approach. I'm not sure how well the implementation would work, even if we try to match it. Also, since Android users are more accustomed to the Gboard look, I feel it would be better to follow that direction. Like Apple seems to have trapezoid by default in their keyboards as well.

angrezichatterbox avatar Mar 08 '25 12:03 angrezichatterbox

For now I think that we can go with the circles to make sure that we get the coloration and other fixes in, and then from there we can maybe make another issue to bring the design to a similar place with iOS 😊 Doing this issue in a simple way will also prepare us for maybe doing a switch to trapezoids.

Feel free to proceed with the Gboard approach, @Linfye! Let us know if further questions come up :)

andrewtavis avatar Mar 08 '25 13:03 andrewtavis