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

Menu item to select the default currency symbol

Open andrewtavis opened this issue 2 years ago • 15 comments

Terms

Description

This issue would create a menu item whereby the user would be taken to a separate screen to select a default currency symbol for the keyboard that appears on the 123 keys. The process is detailed in the designs in Figma. The option would be in general keyboard settings and for those on a per-keyboard basis.

  • Blocked by #16 and #309

Contribution

This issue is a part of Google Summer of Code 2023 😊 Happy to assist as needed!

andrewtavis avatar May 21 '23 13:05 andrewtavis

Place that this is shown in the Figma is: https://www.figma.com/file/c8945w2iyoPYVhsqW7vRn6/scribe_public_designs?type=design&node-id=513%3A1616&mode=design&t=dh3jyNuvsEhKhmR2-1

andrewtavis avatar Oct 07 '23 14:10 andrewtavis

I would be happy to work on this issue :)

lillian-mo avatar Oct 07 '23 14:10 lillian-mo

Hello @andrewtavis 👋🏻, I would like try to work on this.

vickcoo avatar Aug 21 '24 12:08 vickcoo

Nice @vickcoo! Would be great to get this added in :) In thinking on it, we might need #476 merged in as that has the code to make the radio button menus. Do you want to look into the code there a bit and also at what would be needed for #238 that would implement actually changing the main currency symbol? Note that the change here is only for iPhones as iPads have them all on one view :)

andrewtavis avatar Aug 21 '24 16:08 andrewtavis

If all sounds good, feel free to write in #238 and I'll assign that one too! Then we can work to get both of these done! For that one the question is how to make the currency symbol on the number keys variable and replace it with a different one while also updating those that are on the symbol keys :)

andrewtavis avatar Aug 21 '24 16:08 andrewtavis

So, do you mean we need to merge #476 before starting this issue, or do you just want me to reference it first? And I need a little time to figure out #238, because I'm not sure what the difference between this issue and that one. btw, my English isn't very good, so I might not fully understand what you mean 🫣

vickcoo avatar Aug 21 '24 16:08 vickcoo

No worries whatsoever, @vickcoo! At least on my end I'm understanding you well, and happy to explain things as needed to make things clear 😊

What I'm considering is:

  • You're right that we need to merge #476 before this one as it has code that we need
  • The differences between this issue and #238 are:
    • This issue is to add the menu option to select the currency symbol (change is within Scribe-iOS/Scribe, the application's menu)
    • #238 is to reflect the change of the currency symbol in the keyboard UI (change is within Scribe-iOS/Keyboards, the change on the keyboards based on the menu selection here)

What you could do is look into the places that would need the change in #238 and we could discuss them over there? Let me head over and document it a bit for you :)

andrewtavis avatar Aug 21 '24 17:08 andrewtavis

Thanks for your patience and explanation @andrewtavis ! I got it. Let's discuss it further over there, Here's a summary of the steps as I understand them

  1. solve the #238
  2. wait for #476 merged
  3. solve this issue

vickcoo avatar Aug 21 '24 23:08 vickcoo

Yes you can definitely solve #238 in the meantime, and the other PR should be in by the end of the week 😊 Thanks, @vickcoo!

andrewtavis avatar Aug 22 '24 06:08 andrewtavis