ttu-ttu / ebook-reader

Online e-book reader that supports Yomichan
https://reader.ttsu.app
BSD 3-Clause "New" or "Revised" License
695 stars 63 forks source link

Show toolbar on hover instead of on click on desktop devices #316

Open StarScape opened 6 months ago

StarScape commented 6 months ago

First of all, thanks for developing ッツ Reader. I had hacked together something similar on my own to be able to read ePub books with Yomichan, but ッツ is way more polished and a much better solution.

My proposed change: currently you have to click on the top of the screen to show/hide the toolbar. I think this behavior makes perfect sense on a touchscreen, but it feels a little unintuitive to me on a desktop browser (case in point, it took me a bit to figure out that's how you do it). I think it would be better if instead, the toolbar showed when you hovered over it, and then automatically disappear once the mouse had been moved off of it for a certain amount of time, say 1-2s. The current behavior could stay on mobile devices, if desired.

Thoughts? I'd be happy to implement this myself if the maintainers are willing to merge it.

Renji-XD commented 6 months ago

Hi,

i can't really comment if there was any other technical reason or just an ux decision. I don't really have hard feelings against it - not sure if that would be a problem in reality but upper part of text is quite close/slightly overlapping the header area.. that would be maybe annoying to some people which follow the lines with their cursor while reading...

so probably best to keep it behind an option with current behaviour as default and hidden/ignored on mobile (though that kind of defeats the purpose i guess :> )... of course need to be ensured that other functionality and things like menus/dialogs are not overlapped/impacted

feel free to submit a pr if you like