mienaiyami / yomikiru

An offline desktop reader for manga, comics, and novels. Offers a customizable reading experience with extensive settings, shortcuts, themes and layouts. Manga reader / Manhwa reader / Webtoon Reader / EPUB Reader. **Not a downloader**
MIT License
217 stars 10 forks source link

Add New in shortcut keys tab and one other misc #335

Closed zeoint closed 3 months ago

zeoint commented 3 months ago

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

  1. 'Add New' should be a button type and should be linked to the button values in theme maker.
  2. Assigned shortcut keys should not be highlighted.

❌ Actual Behavior

  1. 'Add New' is not a button type and is not linked to the button values in theme maker.
  2. Assigned shortcut keys can be highlighted.
github-actions[bot] commented 3 months ago

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.

mienaiyami commented 3 months ago

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.

zeoint commented 3 months ago

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.

mienaiyami commented 3 months ago

It is the input placeholder color, which you might also see in search inputs. I will see if there is a better way.

mienaiyami commented 3 months ago

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.

zeoint commented 3 months ago

Cool. Thanks for trying.

zeoint commented 3 months ago

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.