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 toggle hover and transparency issue #241

Closed
bgorfain opened this issue Apr 22, 2020 · 10 comments · Fixed by #285
Closed

Dark mode toggle hover and transparency issue #241

bgorfain opened this issue Apr 22, 2020 · 10 comments · Fixed by #285
Labels
bug Something isn't working

Comments

@bgorfain
Copy link

When you hover over the dark mode button when in light mode, there are two very slightly different colors: #210d26 and #000000. The dots on the light mode background are slightly lighter than the background of the button. Also, the white part of the button that sits on top of the black background that expands into dark mode when the button is pressed is slightly misaligned. I spent some time trying to fix this but I don't know anything about html or css.

@theRTC204
Copy link
Collaborator

I'm not following, what exactly is the issue? Perhaps providing a screenshot of what you think is wrong would help.

@bgorfain
Copy link
Author

button
The dots on the button are a slightly different color that the background of the rest of the button. It's pretty insignificant, but I don't think it's meant to look like that.

Also:
Screenshot from 2020-04-22 16-22-42
There is a bit of black on the bottom right edge of the button which makes it look misaligned.

@theRTC204
Copy link
Collaborator

Ah, some of this appears to be the result of #237 combined with some issues from the Darkmode.js 3rd party library.

@theRTC204 theRTC204 added the bug Something isn't working label Apr 23, 2020
@DevSplash
Copy link
Contributor

Could you provide your browser version? I can't reproduce it.

@DevSplash
Copy link
Contributor

Ah, some of this appears to be the result of #237 combined with some issues from the Darkmode.js 3rd party library.

It seems unrelated, but I can fix it.

@theRTC204
Copy link
Collaborator

It seems unrelated, but I can fix it.

It's directly related. That PR is where the opacity was introduced, and the opacity is exactly what is causing the button color to change as you scroll the page.

@DevSplash
Copy link
Contributor

It seems unrelated, but I can fix it.

It's directly related. That PR is where the opacity was introduced, and the opacity is exactly what is causing the button color to change as you scroll the page.

I tried to remove the opacity, but the problem persists. The button on the official website also have the problem when using a light background.

0

@theRTC204
Copy link
Collaborator

I'm not refering to the black edge that's peaking out. If you read my original message that you quoted, I do say the reported issues are a combination of that PR and some existing issues in the library itself.

The concerns mentioned about the background are the result of the opacity changes.

The black edge poking out is from the library.

@DevSplash
Copy link
Contributor

I'm not refering to the black edge that's peaking out. If you read my original message that you quoted, I do say the reported issues are a combination of that PR and some existing issues in the library itself.

The concerns mentioned about the background are the result of the opacity changes.

The black edge poking out is from the library.

Yes, you are right. Adding background-color instead of the opacity might fix it.

DevSplash added a commit to DevSplash/ac-nh-turnip-prices that referenced this issue Apr 27, 2020
@DevSplash DevSplash mentioned this issue Apr 27, 2020
mikebryant added a commit that referenced this issue Apr 27, 2020
@bgorfain
Copy link
Author

Awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants