Closed pauloconnor closed 3 years ago
I love that you took a stab at this already. Awesome stuff. However, I feel that nightmode should really be a global per-user setting; that is to say, there should be a button somewhere that a user clicks to enable this mode and then all pages should be styled accordingly. It shouldn't be something that you click on the graphs page and then lose by clicking over to the dashboards page. That's a pretty jarring experience imho.
A few other random thoughts:
GRAPH_TEMPLATE
setting for the default view. We need to add a GRAPH_TEMPLATE_DARK
or similar for nightmode.:inverse
on activation.I finally got around to playing around with this today (sorry!) and I like the initial work here. I think there are some issues that would need to be resolved before this gets merged in, but it's a good start. I won't blame you if you don't feel like hacking on this any more, but I intend to use it as a starting point whenever I get back around to this feature.
Some notes to self:
I had a properly working version of this written at Monitorama EU but I lost the changes :( I'll try and get back to this soon.
NOOOOOOOOOOO!!!
This is what I'm thinking for nightmode.
Comments?
That looks really good. My only change would be to try and make the toolbar more subtle.
Agreed. That will be non-trivial though - the icons are white so I'll have a think about that. Let me push this code so you can have a play and see what you think.
Check out some of the stuff you can do with css transforms, I bet that would help.
Jason
On Fri, Nov 15, 2013 at 10:05:23AM -0800, pauloconnor wrote:
Agreed. That will be non-trivial though - the icons are white so I'll have a think about that. Let me push this code so you can have a play and see what you think.
Reply to this email directly or view it on GitHub: https://github.com/obfuscurity/descartes/pull/135#issuecomment-28590069
Jason Dixon http://obfuscurity.com/ https://twitter.com/obfuscurity
Better. I'd still go a bit darker. They're easily the most prominent thing on the page. I mean, it's not the worst thing in the world if it gets merged as-is. I can always tweak it in the tree. :smile_cat:
I've made the text match the header links. I've also darkened the background for the well.
unrelated to anything I feel like I know you since 2012 @obfuscurity (dont think you ever changed your profile picture 😂)
Blast from the past to even get a notification on issues related to graphite for me.
There are a few elements that don't change for nightmode, in particular buttons and drop downs but it's designed more for fullscreen of dashboard. Nightmode will persist while selecting Fullscreen.
Addresses https://github.com/obfuscurity/descartes/issues/109