Add keyboard shortcuts to titles #5857

kommunarr · 2024-10-13T23:32:04Z

Add keyboard shortcuts to titles

Pull Request Type

Bugfix
Feature Implementation
Documentation
Other

Related issue

closes #1587

Description

Adds keyboard shortcut information to their button labels, e.g., Mute (m)
- This includes the video player buttons, the feed refresh buttons, the side nav History/Settings titles, and the top nav HistoryForward/HistoryBackward/OpenNewWindow buttons, but NOT Electron context menu labels
Implementation note: To do this properly for the built-in Shaka controls that we don't modify in our code, this PR updates the base Shaka label localization values to include the shortcut whenever the Shaka locale is set.
Implementation note: Constants were added for the aforementioned keyboard shortcuts.

Screenshots

Testing

Test that buttons with corresponding keyboard shortcuts have the shortcut referenced in the label, be they in "on" or "off" mode (e.g., Full screen (f) / Exit full screen (f)). See the official list of shortcuts for reference.

Desktop

OS: OpenSUSE
OS Version: TW

Additional context

The addition of the new constants should help make enhancements like #251 easier in the future.

…stants

src/constants.js

PikachuEXE · 2024-10-14T05:51:41Z

Label for captions

kommunarr · 2024-10-14T13:18:10Z

Another good catch; I wasn't testing with a video that had a captions option. Just fixed the issue and added logic preventing label modification if the Shaka localization key does not exist. Also note that this specific Captions label will be the only one with another parenthetical present. This is fine IMO.

…has no matching value

src/constants.js

efb4f5ff-1298-471a-8973-3d47447115dc

Pressing c doesnt enable/disable captions

kommunarr · 2024-10-14T18:59:15Z

Added localization of shortcuts & multi-key shortcuts and caught a dangling instance of the wrong constant name for CAPTIONS

efb4f5ff-1298-471a-8973-3d47447115dc · 2024-10-14T19:04:55Z

Question: Navigating to the History tab and Settings tab also got shortcuts. Is there a reason those weren't added?

…onstants

kommunarr · 2024-10-14T19:11:31Z

@efb4f5ff-1298-471a-8973-3d47447115dc I specifically added the shortcuts with corresponding button labels. The closest equivalent to those would be the corresponding side nav options, which I suppose would work. I can add those as well.

…e naming

PikachuEXE

Tested video player, sidebar, top nav button labels, refresh button

Only on MacOS though

github-actions · 2024-11-02T10:37:11Z

This pull request has conflicts, please resolve those before we can evaluate the pull request.

efb4f5ff-1298-471a-8973-3d47447115dc · 2024-11-02T14:44:27Z

Just fixing full window to have the shortcut show in the overflow menu?

correct

…at/add-shortcut-labels

…ube into feat/add-shortcut-labels

github-actions · 2024-11-23T14:22:12Z

Conflicts have been resolved. A maintainer will review the pull request shortly.

src/renderer/components/ft-shaka-video-player/ft-shaka-video-player.js

src/renderer/views/Popular/Popular.js

* Update title for custom FreeTube button labels with keyboard shortcuts * Add keyboard shortcut to unmodified Shaka control labels * Replace in-code use of available shortcuts with the corresponding constants * Add explanatory comments * Fix captions constant name * Prevent creating new shortcut localization if Shaka localization key has no matching value * Add util functions to localize special keys * Replace more video player shortcut usage in code with corresponding constants * Add labels for History and Settings app shortcuts, and update variable naming * Display 'Option' instead of 'Alt' for Mac users * Use Mac icons in keyboard shortcuts for Macs * Update Mac arrow icon choice * Update KeyboardShortcuts constant organization in preparation for keyboard shorcut modal * Adjust nameSpan._textContent to be set to same value as aria-label * Add code comment explaining shakaControlKeysToShortcuts * Move changes from deleted Popular.js to Popular.vue

kommunarr added 2 commits October 13, 2024 10:53

Update title for custom FreeTube button labels with keyboard shortcuts

fe5de7f

Add keyboard shortcut to unmodified Shaka control labels

3134053

github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Oct 13, 2024

FreeTubeBot enabled auto-merge (squash) October 13, 2024 23:32

Replace in-code use of available shortcuts with the corresponding con…

0d3882e

…stants

kommunarr force-pushed the feat/add-shortcut-labels branch from 24b233d to 0d3882e Compare October 13, 2024 23:35

kommunarr requested a review from efb4f5ff-1298-471a-8973-3d47447115dc October 13, 2024 23:35

PikachuEXE reviewed Oct 14, 2024

View reviewed changes

src/constants.js Outdated Show resolved Hide resolved

src/constants.js Outdated Show resolved Hide resolved

Add explanatory comments

443c4af

kommunarr requested a review from PikachuEXE October 14, 2024 02:35

Fix captions constant name

42c3128

kommunarr force-pushed the feat/add-shortcut-labels branch from ad59178 to 6202ff7 Compare October 14, 2024 13:25

Prevent creating new shortcut localization if Shaka localization key …

7c379e2

…has no matching value

kommunarr force-pushed the feat/add-shortcut-labels branch from 6202ff7 to 7c379e2 Compare October 14, 2024 13:26

ChunkyProgrammer reviewed Oct 14, 2024

View reviewed changes

src/constants.js Outdated Show resolved Hide resolved

efb4f5ff-1298-471a-8973-3d47447115dc reviewed Oct 14, 2024

View reviewed changes

Add util functions to localize special keys

764cace

kommunarr requested review from ChunkyProgrammer and efb4f5ff-1298-471a-8973-3d47447115dc October 14, 2024 18:59

Replace more video player shortcut usage in code with corresponding c…

6798c3a

…onstants

Add labels for History and Settings app shortcuts, and update variabl…

6f8d212

…e naming

kommunarr force-pushed the feat/add-shortcut-labels branch from b934703 to 6f8d212 Compare October 14, 2024 19:35

efb4f5ff-1298-471a-8973-3d47447115dc previously approved these changes Oct 14, 2024

View reviewed changes

PikachuEXE previously approved these changes Oct 15, 2024

View reviewed changes

ChunkyProgrammer requested a review from absidue October 15, 2024 00:54

github-actions bot added the PR: merge conflicts / rebase needed label Nov 2, 2024

kommunarr added 3 commits November 23, 2024 08:01

Merge branch 'development' of github.com:FreeTubeApp/FreeTube into fe…

dd1fffa

…at/add-shortcut-labels

Adjust nameSpan._textContent to be set to same value as aria-label

89b6861

Merge branch 'feat/add-shortcut-labels' of github.com:kommunarr/FreeT…

fdae9b4

…ube into feat/add-shortcut-labels

kommunarr dismissed PikachuEXE’s stale review via fdae9b4 November 23, 2024 14:21

kommunarr requested a review from efb4f5ff-1298-471a-8973-3d47447115dc November 23, 2024 14:22

github-actions bot removed the PR: merge conflicts / rebase needed label Nov 23, 2024

kommunarr requested review from absidue and PikachuEXE November 23, 2024 14:22

PikachuEXE reviewed Nov 24, 2024

View reviewed changes

src/renderer/components/ft-shaka-video-player/ft-shaka-video-player.js Show resolved Hide resolved

Add code comment explaining shakaControlKeysToShortcuts

304fa82

kommunarr force-pushed the feat/add-shortcut-labels branch from 8410df1 to 304fa82 Compare November 24, 2024 01:03

PikachuEXE previously approved these changes Nov 24, 2024

View reviewed changes

ChunkyProgrammer reviewed Nov 26, 2024

View reviewed changes

src/renderer/views/Popular/Popular.js Outdated Show resolved Hide resolved

kommunarr dismissed PikachuEXE’s stale review via 84deb9e November 26, 2024 22:10

kommunarr force-pushed the feat/add-shortcut-labels branch from 84deb9e to cb7f395 Compare November 26, 2024 22:12

Move changes from deleted Popular.js to Popular.vue

532fcc7

kommunarr force-pushed the feat/add-shortcut-labels branch from cb7f395 to 532fcc7 Compare November 26, 2024 22:13

PikachuEXE approved these changes Nov 27, 2024

View reviewed changes

ChunkyProgrammer added the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 27, 2024

absidue approved these changes Dec 1, 2024

View reviewed changes

efb4f5ff-1298-471a-8973-3d47447115dc approved these changes Dec 2, 2024

View reviewed changes

FreeTubeBot merged commit 112fd32 into FreeTubeApp:development Dec 2, 2024
5 checks passed

github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Dec 2, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add keyboard shortcuts to titles #5857

Add keyboard shortcuts to titles #5857

kommunarr commented Oct 13, 2024 •

edited

Loading

PikachuEXE commented Oct 14, 2024

kommunarr commented Oct 14, 2024 •

edited

Loading

efb4f5ff-1298-471a-8973-3d47447115dc left a comment

kommunarr commented Oct 14, 2024 •

edited

Loading

efb4f5ff-1298-471a-8973-3d47447115dc commented Oct 14, 2024 •

edited

Loading

kommunarr commented Oct 14, 2024

PikachuEXE left a comment

github-actions bot commented Nov 2, 2024

efb4f5ff-1298-471a-8973-3d47447115dc commented Nov 2, 2024

github-actions bot commented Nov 23, 2024

Add keyboard shortcuts to titles #5857

Add keyboard shortcuts to titles #5857

Conversation

kommunarr commented Oct 13, 2024 • edited Loading

Add keyboard shortcuts to titles

Pull Request Type

Related issue

Description

Screenshots

Testing

Desktop

Additional context

PikachuEXE commented Oct 14, 2024

kommunarr commented Oct 14, 2024 • edited Loading

efb4f5ff-1298-471a-8973-3d47447115dc left a comment

Choose a reason for hiding this comment

kommunarr commented Oct 14, 2024 • edited Loading

efb4f5ff-1298-471a-8973-3d47447115dc commented Oct 14, 2024 • edited Loading

kommunarr commented Oct 14, 2024

PikachuEXE left a comment

Choose a reason for hiding this comment

github-actions bot commented Nov 2, 2024

efb4f5ff-1298-471a-8973-3d47447115dc commented Nov 2, 2024

github-actions bot commented Nov 23, 2024

kommunarr commented Oct 13, 2024 •

edited

Loading

kommunarr commented Oct 14, 2024 •

edited

Loading

kommunarr commented Oct 14, 2024 •

edited

Loading

efb4f5ff-1298-471a-8973-3d47447115dc commented Oct 14, 2024 •

edited

Loading