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

Add voila dark mode #669

Closed
wants to merge 1 commit into from
Closed

Add voila dark mode #669

wants to merge 1 commit into from

Conversation

eteq
Copy link
Contributor

@eteq eteq commented Jun 8, 2021

This was an attempt to do #415 , but not it does not work in that no dark mode is activated.

I'm just guessing this is because vue is not hooked up to whatever voila uses to signal dark mode? But it's curious it's not consistent with #665...

Opening as a draft because it doesn't work, but might be a good starting point for a later implementation

@pllim
Copy link
Contributor

pllim commented Jun 8, 2021

Pretty sure it used to work. In fact, I enabled it in one of the early Imviz PR that was never merged because the white background was hurting my eyes as I did R&D.

@pllim
Copy link
Contributor

pllim commented Jun 8, 2021

You are right. It no longer works. Maybe @maartenbreddels know why.

@pllim pllim added bug Something isn't working UI/UX😍 labels Jun 8, 2021
@maartenbreddels
Copy link
Collaborator

@mariobuikhuizen could it be the template of jdaviz, I think the vue(tify) templates support it already right?

@mariobuikhuizen
Copy link
Collaborator

Yeah, I will take a look.

@pllim
Copy link
Contributor

pllim commented Jun 15, 2021

I'll move this patch to #682

@pllim pllim closed this Jun 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working UI/UX😍
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants