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

[Feature Request] Native Titlebar Patch #62

Closed
Jevil7452 opened this issue Oct 19, 2023 · 5 comments
Closed

[Feature Request] Native Titlebar Patch #62

Jevil7452 opened this issue Oct 19, 2023 · 5 comments

Comments

@Jevil7452
Copy link

In case you might not be aware, there exists a patch for Firefox: https://github.com/ephemeralViolette/firefox-native-controls that replaces the Windows 10-style titlebar (that looks out of place on Windows 7) with a titlebar that is drawn by the OS, so it will always look correct regardless of the theme the user is using. As this is a source code patch, it seems like a good idea to integrate it into the browser, so the users of this browser can enjoy having it be built-in, and not having to replace xul.dll files which will break when Firefox updates, unless the original author of the patch updates it too.

@MasterSlenderTR
Copy link

MasterSlenderTR commented Oct 19, 2023

Yes, please, as a Win7 user I'd like to have this in Mercury. I absolutely despise the forced Win10-scrollbar, it looks so ugly in Win7.

@ephemeralViolette
Copy link

This doesn't currently handle the titlebar (only scrollbars), but it's designed for only ESR 115, which still supports the native Windows 7 titlebar.

@Jevil7452
Copy link
Author

Yeah, i meant the native scrollbars, i said "titlebar" by mistake

@Alex313031
Copy link
Owner

@ephemeralViolette @MasterSlenderTR @Jevil7452 I will be adding this to the Mercury-Win7 repo.

For this repo, gonna leave it as is.

@Alex313031
Copy link
Owner

Alex313031 commented Jan 10, 2024

@ephemeralViolette About the Widevine thing. The .sig files can only be generated with a Widevine license and a script they give you. So, your patch would not affect it either way. You can remove that from the readme to dispell fears.

I will be manually adding the patch, I.E. going to those files and copy/pasting. You should update the patch based on rebased ESR 115.6.0/115.6.1.

@Alex313031 Alex313031 reopened this Jan 10, 2024
@gz83 gz83 closed this as completed Sep 2, 2024
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

No branches or pull requests

5 participants