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

Readibiliy improvements for calculator component #6

Closed
mtaki14 opened this issue Mar 3, 2016 · 10 comments
Closed

Readibiliy improvements for calculator component #6

mtaki14 opened this issue Mar 3, 2016 · 10 comments
Milestone

Comments

@mtaki14
Copy link

mtaki14 commented Mar 3, 2016

I would love to see some readability improvements for the calculator. As it stands the result is not very clearly indicated.

For example making the font bigger for the result and the font colour more prominent would help considerably to make it more readable.

@mtaki14 mtaki14 changed the title Readibiliy improvements for calculator functionality Readibiliy improvements for calculator component Mar 3, 2016
@polyvertex
Copy link
Member

Have you tried tweaking the font_size setting of the [gui] section in application configuration to see if it helps?
It is not possible to change theme's colors yet I'm afraid.

Edit: check out the Configuration chapter if you need help on how to edit the configuration files.

@mtaki14
Copy link
Author

mtaki14 commented Mar 3, 2016

Yes, already tried tweaking the font_size but unfortunately that didn't help much.

Even if you simply hardcoded the result to show slightly bigger and with a different colour, other than grey, that would be fine. No need to for an elaborate solution that involves themes.

@mtaki14
Copy link
Author

mtaki14 commented Mar 3, 2016

...or display the result on the same line as where you input the calculation just like lauchy does it. That's even better imo for readability.

Edit:
Screenshot of launchy's implementation
Keypirinha's Implementation

@polyvertex
Copy link
Member

The only thing I can do for the next release is to make the hard-coded color significantly lighter to improve readability. Here is the preview with font_size = xl (and no zoom in):
preview

Keypirinha is a very generic tool so this is not something that can be applied to the Calc package only unfortunately. In other words, the second line of the item has to remain darker than the first one, otherwise a bigger results list would just break your eyes forever.

Edit: Would that change illustrated by the preview be acceptable enough for you?

@mtaki14
Copy link
Author

mtaki14 commented Mar 4, 2016

Thanks for your response. Although not ideal your suggested solution should be sufficient for now. I'll be awaiting for better customization options in the future, whenever that happens.

@polyvertex
Copy link
Member

Modification will be available in the next release.

Question: considering your use case; and that complete theme customization (like Launchy) not being a priority; would you consider this issue fixed if you had settings for color control?

@polyvertex
Copy link
Member

@mtaki14: even better, the selected result can just be highlighted, as shown below :)
brightness

@mtaki14
Copy link
Author

mtaki14 commented Mar 7, 2016

Yes, that looks much better. When is a new release expected btw?

@polyvertex
Copy link
Member

Probably by the end of the week.

@polyvertex
Copy link
Member

Released! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants