FreeTubeApp / FreeTube

An Open Source YouTube app for privacy
https://freetubeapp.io/
GNU Affero General Public License v3.0
13.25k stars 823 forks source link

Show on hover if icon has shortcut #1587

Open efb4f5ff-1298-471a-8973-3d47447115dc opened 3 years ago

efb4f5ff-1298-471a-8973-3d47447115dc commented 3 years ago

Is your feature request related to a usage problem (not a bug)? Please describe. I often go back to the docs because i need to find out what shortcut i need for a task. This wont be the case if i can look it up fast on just a hover.

Describe the solution you'd like to see implemented See video. This also needs to be the case when the rewrite is done and u can choose ur own shortcuts.

Describe alternatives you've considered None.

Video

https://user-images.githubusercontent.com/73130443/129462389-88c2a52b-eb9c-4517-933f-89f358e496c3.mp4

Additional context None.

vallode commented 2 years ago

Spent some time looking into this, sadly it's not as easy as I thought it ought to be. Videojs doesn't provide a direct way of changing the buttons' labels (and most of them are dynamically changing) so for a clean way of changing this I think we'd need to extend every single (around 15) of the Videojs buttons we use.

When/if I am bored I'll get this done because I'd like to see it too ;)

kommunarr commented 1 year ago

Boosting this issue / keeping it in mind for any possible future shakaup.

shadycloud commented 11 months ago

Screenshot from 2023-11-12 15-48-58

From left to right the respective to the player... Skipping the buttons with multiple options (like quality, captions, and playback speed) I am having a hard time grepping for a few of them. (i.e. Play, Pause, Replay, Mute, Pic in Pic, and Full Screen)

Play (Shortcut 'Space Bar') Pause (Shortcut 'Space Bar') Replay (NA) Mute (Shortcut 'M' or 'm') Toggle Loop (N/A) https://github.com/FreeTubeApp/FreeTube/issues/4298 Picture-in-Picture (Shortcut 'I' or 'i') Full Window (Shortcut 'S' or 's') Full Screen (Shortcut 'F' or 'f')

cloud@cloud:~/FreeTube$ grep -r "button.title"
grep: src/renderer/components/ft-video-player/.ft-video-player.js.swo: binary file matches
src/renderer/components/ft-video-player/ft-video-player.js:          button.title = 'Toggle Loop'
src/renderer/components/ft-video-player/ft-video-player.js:          button.title = 'Full Window' 
src/renderer/components/ft-video-player/ft-video-player.js:          button.title = 'Toggle Theatre Mode'
src/renderer/components/ft-video-player/ft-video-player.js:          button.title = 'Screenshot'
src/renderer/components/ft-video-player/ft-video-player.js:          button.title = 'Select Quality'

replaced by the following to at least show the shortcut on hover of full window and toggle loop buttons...

src/renderer/components/ft-video-player/ft-video-player.js:          button.title = 'Full Window (Shortcut S or s)'
src/renderer/components/ft-video-player/ft-video-player.js:          button.title = 'Toggle Loop (Shortcut R or r)'