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

[Bug] Colour glitches on start since last update #308

Closed
ZiClaud opened this issue Dec 10, 2024 · 7 comments · Fixed by #371
Closed

[Bug] Colour glitches on start since last update #308

ZiClaud opened this issue Dec 10, 2024 · 7 comments · Fixed by #371

Comments

@ZiClaud
Copy link
Contributor

ZiClaud commented Dec 10, 2024

Screencast.from.2024-12-10.09-23-44.mp4

It only happens when I have the page open

Screencast.from.2024-12-10.09-28-56.mp4

It also happens with other colours

@giwih
Copy link
Contributor

giwih commented Dec 10, 2024

I think for some reason the theme you selected is not saved, and color glitches are caused by another bug I fixed in - #304 , do you have any problems with saving other settings?

@ZiClaud
Copy link
Contributor Author

ZiClaud commented Dec 11, 2024

I think for some reason the theme you selected is not saved, and color glitches are caused by another bug I fixed in - #304 , do you have any problems with saving other settings?

Nope, no problems with saves, just with the colour, and just on Firefox.
There's still the same problem while using https://xengshi.github.io/materialYouNewTab/, so I think that it's either because of some extension I use, or something like that.

Also, the same exact thing happens on mobile (I use the Firefox app).

@prem-k-r
Copy link
Collaborator

Is it fixed with recent PRs merging?
https://github.com/XengShi/materialYouNewTab/archive/refs/heads/main.zip

@ZiClaud
Copy link
Contributor Author

ZiClaud commented Dec 13, 2024

Is it fixed with recent PRs merging? refs/heads/main.zip

I mean, sort of.
Now it won't appear the glitched theme, but it still resets the theme to the red one.
Also, the second attachment shows the console.

Screencast.from.2024-12-13.15-38-56.mp4
Screencast.from.2024-12-13.15-43-49.mp4

@itz-rj-here
Copy link
Collaborator

Is it fixed with recent PRs merging? refs/heads/main.zip

I mean, sort of. Now it won't appear the glitched theme, but it still resets the theme to the red one. Also, the second attachment shows the console.

Screencast.from.2024-12-13.15-38-56.mp4
Screencast.from.2024-12-13.15-43-49.mp4

Well it's also causing in the chrome browser too. :(

@prem-k-r
Copy link
Collaborator

That's odd..
It seems some issue with custom color

@giwih
Copy link
Contributor

giwih commented Dec 13, 2024

firefox has a different browser engine, firefox restores the last value of color picker, and when duplicate/open a new tab triggers EventListener, which updates the theme, the default color picker is #ea1010, if you put for example some purple, then when you open a new tab will be purple.

all chromium browsers:
image
firefox:
image
In general, firefox keeps triggering EventListener and that's why the theme is updated. If you remove these lines, everything will work (color picker will not work, of course).

colorPicker.addEventListener('input', handleColorPickerChange);

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