-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Bump electron from 38.4.0 to 39.0.0 #8215
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
Conversation
|
https://www.electronjs.org/blog/electron-39-0#breaking-changes |
c8a0622 to
30ea95f
Compare
|
getAccentColor is available on linux as well now. IIRC, we talked about being able to use this in the past for theming? |
That would be cool to implement |
|
There is a bug report open on the Electron repo about the system theme detection being broken on with Electron 39: electron/electron#48736 |
Yes, the main blocker was never that it wasn't easily accessible (I had hacked together code that manually called the dbus API and checked a few other sources to get it) but that I couldn't come up with a good way to infer/calculate all the other colour properties (my old branch is here: https://github.com/absidue/FreeTube/commits/system-accent-color/) Jason did suggest at the time that instead of trying to calculate the other properties, we could try to match the system accent colour to one of FreeTube's existing ones with a colour similarity algorithm. Unfortunately that only worked well if FreeTube actually had a similar one, I remember grey being particularly difficult as FreeTube didn't have anything even remotely close to that at the time. |
Cant confirm the theming issue due to having wayland issues
Could someone else try to verify this
30ea95f to
56eb71f
Compare
Bumps [electron](https://github.com/electron/electron) from 38.4.0 to 39.0.0. - [Release notes](https://github.com/electron/electron/releases) - [Changelog](https://github.com/electron/electron/blob/main/docs/breaking-changes.md) - [Commits](electron/electron@v38.4.0...v39.0.0) --- updated-dependencies: - dependency-name: electron dependency-version: 39.0.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
56eb71f to
46c6030
Compare
Couldnt reproduce this bug in Manjaro VM. Switching from light to dark and vice versa also switches theme in FT. Do we want to approve and merge? |
|
We could merge it but keep in mind that if any users report that issue that it's an upstream one. |
|
Or we can wait for next version to see if bug fixed (and include other bug fixes |
|
Im fine with waiting |
|
39.1.0 is out, the changelog doesn't specifically mention the system theme, but does mention another wayland fix and includes a Chromium update. |
|
Nothing theme related (I read Chromium changelog too |
|
Superseded by #8259. |
Pull request was closed

Bumps electron from 38.4.0 to 39.0.0.
Release notes
Sourced from electron's releases.
... (truncated)
Commits
0abda74docs: modify the thickFrame doc (#48677)4e8a552fix: logical bug in install.js env var handling (#48673)d83383bdocs: fix Ubuntu version used to build Electron (#48643)496db94chore: bump chromium to 142.0.7444.52 (39-x-y) (#48641)00627c6fix: crash on empty dialog extensions array on Windows (#48658)7319e5cdocs: security.md mark 'Enable process sandboxing' as active by defau… (#48647)1056280feat: enable more granular a11y feature management (#48625)4fda94bfeat: AddgetAccentColoron Linux (#48628)e3715b0fix:systemPreferences.getAccentColorinverted color (#48624)90674e0fix: icon in Windows toast notification (#48629)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)