Skip to content
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

feat: improve and add new shortcuts #556

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

United600
Copy link
Collaborator

@United600 United600 commented Feb 7, 2025

  • adds missing keyboard resources to the .csproj
  • adds VLC-like Save Snapshot shortcut (Shift + S)
  • adds YouTube-like Next button shortcut (Shift + N)
  • adds YouTube-like Previous button shortcut (Shift + P)
  • fixes some unwanted search shortcut behaviors
  • removes the selection row opacity to fix the focus visual visibility
  • changes the toggle player visibility controller shortcut to use the View button, freeing up the Y button to be used for search, as per the documentation here and here

add alternative previous and next button shortcut
hide list view item selection indicator
fix undesirable search shortcut behavior
@@ -461,52 +461,76 @@
<LastGenOutput>Resources.generated.cs</LastGenOutput>
<SubType>Designer</SubType>
</PRIResource>
<PRIResource Include="Strings\ar-SA\KeyboardResources.resw" />
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh we were missing the keyboard resources huh

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙄 My bad, I thought that it was automatic with the Crowdin action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants