Advaith3600 / PowerToys-Run-Currency-Converter

PowerToys Run plugin which will convert real and crypto currencies.
MIT License
159 stars 8 forks source link

Quick conversion is confusing when I've added an 'extra currency' #6

Closed untitled500 closed 6 months ago

untitled500 commented 6 months ago

Assuming the intent of converting something quickly is to convert from a home curency (SGD in this example) to other currencies (USD, EUR in this example): It gets confusing when I see 13.77 EUR and 18.63 EUR. I need to spend a minute or more to see which direction and what amount is being converted.

MacOS does it well: https://techstacker.com/static/1785b23e59320ecd54efe14e0880a579/bfdc0/mac-spotlight-search-currency-conversion.png

Current UI (ignore the red box): Untitled

Advaith3600 commented 6 months ago

Hey, unfortunately, windows powertoys run doesn't give many options to change the UI of the output, therefore we are stuck with such text-based outputs. I also prefer the way some applications on Mac do it, especially Raycast:

image

The plugin is built in such a way that the order in which this is shown is predictable:

  1. Local to Global
  2. Local to Extra
  3. Global to Local
  4. Global to Extra
  5. Extra to local (coming soon - optional)
  6. Extra to global (coming soon - optional)

Plus the global is an optional field that will remove unnecessary conversions if you do not want it.

untitled500 commented 6 months ago

Yeah Raycast does it well too. I assume all Powertoys lets you do is to send plain text and subtext and one icon as output. I suggest something like this: USD 30 = INR 2501.42 USD 30 = SGD 40.48

Bonus of doing so - is to eliminate confusion in the existing tool where I type 30 USD and it shows 22 USD Currency conversion from SGD to USD. Untitled

Also, why the distinction between Global and Extra? Can I have multiple comma separated Globals instead? That would make some things easier?

  1. Local to Global1
  2. Local to Global2
  3. Global1 to Local
  4. Global2 to Local
Advaith3600 commented 6 months ago

That indeed makes sense, thanks for the idea, I will do some changes

Advaith3600 commented 6 months ago

I have made some changes, please try the following and let me know your feedback.

CurrencyConverter-1.1.2-x64.zip

untitled500 commented 6 months ago

I like the output style setting. Works well for someone who is familiar with the existing style, and will work for new users like me. May I suggest Full Text as the default output style for new users. Also, there is a spelling mistake for Conversion -- in "Quick Convertion Direction" and "Quick Convertion Local Currency".

Advaith3600 commented 6 months ago

Thank you for bringing this to my attention, I will do the necessary in the next version release. Closing the issue for now. If you have anything else, please feel free to open this again or a new issue.