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

Dark mode/theme error #3127

Closed
lansingthomas opened this issue Aug 17, 2023 · 5 comments
Closed

Dark mode/theme error #3127

lansingthomas opened this issue Aug 17, 2023 · 5 comments

Comments

@lansingthomas
Copy link
Contributor

Hi y'all. please advise.

Describe the bug
Both dark-mode from the toggle modes menu and the Use Dark Theme button in the Common Settings section are causing an error -- and not initiating dark theme.

the error:
WARN - Warning: JavaScript error: GError: Domain: "WebKitJavascriptError", Code: 699, Message: nyxt:new:1: SyntaxError: Unexpected identifier 'e'. Expected ')' to end an argument list.

Precise recipe to reproduce the issue
try to use dark mode from common settings, or from dark-mode command -- observe the message buffer

Information
Ubuntu
Nyxt 3.6.0
Flatpak installation

@hgluka
Copy link
Contributor

hgluka commented Aug 18, 2023

Both dark-mode from the toggle modes menu and the Use Dark Theme button in the Common Settings section are causing an error -- and not initiating dark theme.

As far as I know, those two are unrelated.

  • dark-mode runs the darken bookmarklet from bookmarklets-mode, which just invokes a piece of JS code to darken a webpage.
  • Dark theme is the Nyxt theme, which handles the UI elements (and internal pages, panel and prompt buffers), but not the webpages themselves.

I can reproduce the error for dark-mode and I think there's an easy fix.
Dark theme works for me, but I have to restart Nyxt for it to be applied. This will hopefully be fixed soon in #3120.

@aadcg
Copy link
Member

aadcg commented Aug 18, 2023

@hgluka correct! Could you please close this issue and open a short issue mentioning the issue relative to dark-mode?

The issue relative to the dark theme is implicit in #2296.

@hgluka
Copy link
Contributor

hgluka commented Aug 18, 2023

@aadcg I was going to just make a PR with the fix, but I can open a new issue as well.

@aadcg
Copy link
Member

aadcg commented Aug 18, 2023

You should always open an issue and assign it to yourself in that case. That communicates to your team members on what you're focusing.

@hgluka
Copy link
Contributor

hgluka commented Aug 18, 2023

Closing in favor of #3128.

@hgluka hgluka closed this as completed Aug 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants