-
-
Notifications
You must be signed in to change notification settings - Fork 13
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 New in shortcut keys tab and one other misc #335
Comments
Thank you for your feedback and suggestions. Please note that the author might only respond on weekends, so we kindly ask for your patience and understanding. |
This is because they are "text inputs" and not buttons, previous version was also "text inputs". Using buttons here is not a great idea because buttons are meant to do some action on click, and there is none here. Here "Add new" is capturing many key clicks so its not a button. As for the highlight, its not possible to remove it without disabling the whole element's mouse capture. |
Then can the 'Add New' text color be improved? Its linked to text color property, but its shade is darker than other text color related properties. |
It is the input placeholder color, which you might also see in search inputs. |
Tried using button for "add new" and replacing it with input on click, but results in many unwanted behaviors. So for now I will keep it same. |
Cool. Thanks for trying. |
Previously because of the dark shade it looked like kind of not allowed to click text. But now in v2.19.1, it looks correct, which is great. And forget about the button, I thought that it was a button, so I opened the issue in bug. The button is unnecessary. Thanks for adding the color to it, its what I wanted from it. |
Type of installation
Setup (.exe)
Type of reader (if reader related)
None
Steps to reproduce
Open Yomikiru settings and go to shortcut keys tab.
1. 'Add New' is not a button type and is not linked to button values in theme maker.
2. Assigned shortcut key highlight.
✔️ Expected Behavior
❌ Actual Behavior
The text was updated successfully, but these errors were encountered: