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

Refactor Menu Navigation and Update Media Control Icons #2783

Merged
merged 2 commits into from
Dec 28, 2024

Conversation

yilmazcabuk
Copy link
Contributor

1. Refactor Menu Navigation

  • Migrated to the navigationHistory API.
  • Replaced deprecated webContents navigation methods (canGoBack, goBack, canGoForward, goForward) with the navigationHistory API.
  • Updated mainMenuTemplate to utilize navigationHistory destructured from win.webContents.

2. Update Media Control Icons

  • Refreshed "next," "pause," "play," and "previous" icons in the media-icons-black folder.
  • Improved visual consistency and ensured the latest icon designs are applied.

Fixes:

Testing:

  • Verified proper navigation functionality using the navigationHistory API.
  • Confirmed new media icons render correctly in the UI.

Impact:

  • Modernizes the codebase by adopting the new navigationHistory API, ensuring compatibility with future updates.
  • Enhances the aesthetic appeal and consistency of media control icons.

Updated "next," "pause," "play," and "previous" icons in the media-icons-black folder. This improves visual consistency and ensures the latest icon designs are applied.
- Replaced deprecated `webContents` navigation methods (`canGoBack`, `goBack`, `canGoForward`, `goForward`) with `navigationHistory` API.
- Updated `mainMenuTemplate` to use the new `navigationHistory` destructured from `win.webContents`.
@JellyBrick JellyBrick added bug Something isn't working enhancement New feature or request labels Dec 27, 2024
@JellyBrick JellyBrick merged commit 6b8fed3 into th-ch:master Dec 28, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Taskbar media control buttons look outdated
2 participants