Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Dark mode / night theme #198

Open
BasThomas opened this issue Jul 28, 2017 · 24 comments
Open

Dark mode / night theme #198

BasThomas opened this issue Jul 28, 2017 · 24 comments
Labels
👍 feature approved Features approved for implementation
Milestone

Comments

@BasThomas
Copy link
Collaborator

Would love this. Definitely not too important though.

@shaps80
Copy link
Contributor

shaps80 commented Jul 29, 2017

Haha I wondered how long it would take for someone to request this.

@BasThomas
Copy link
Collaborator Author

This could go together with a dark app icon as well

@Sherlouk
Copy link
Member

@Huddie @BrianLitwin @rnystrom

Anybody got a mockup design or palette they think would work nice as a dark mode? Need some inspiration before I go rainbow city on the app :trollface:

@kocheck
Copy link
Collaborator

kocheck commented Nov 12, 2018

@Sherlouk This is a huge interest of mine 👍 I might spin something up this weekend to play with.

@Sherlouk
Copy link
Member

I spent a week turning my Slack into dark mode for a reason 😂Dark mode should be a requirement for all apps :trollface:

Can't wait to see what you make!

@rnystrom
Copy link
Member

Very very interested. The hardest part for us will be text colors. Might have to update StyledTextKit to support themes or something so we can regen attributed strings without rebuilding.

Sent with GitHawk

@ggliv
Copy link

ggliv commented May 26, 2019

I understand that this is an ancient thread but what’s the status on this? I’d love to see this happen.

@BasThomas
Copy link
Collaborator Author

Hey @ggliv, thanks for bringing it up! Let's wait and see what Apple announces on Monday... and see where to go from there.

There's also still a work-in-progress pull request for a dark mode here.

@Huddie
Copy link
Collaborator

Huddie commented Jun 4, 2019

Seems like dark mode will kinda come for free with iOS 13.

Sent with GitHawk

@jdisho
Copy link
Collaborator

jdisho commented Jun 4, 2019

I guess, we still need to support dark mode for older iOS versions, right? 🤔

@Sherlouk
Copy link
Member

Sherlouk commented Jun 4, 2019

Didn't take you folks long to start updating this 😅 Personally I don't think we should actively try to support dark mode on iOS 12-

Dark mode exclusive to iOS 13+ using OS systems sounds like a simpler solution and less maintenance longer term.

@ghost
Copy link

ghost commented Jul 19, 2019

Would love to see one too :)

Sent with GitHawk

@ideologysec
Copy link

Definitely throwing my vote in for a dark mode in iOS 13+ (previous versions not so much; let's aim for the MVP of getting it implemented, and if there's enough demand it can maybe be backported...)

@ggliv
Copy link

ggliv commented Jul 26, 2019

@BasThomas Any news?

@BasThomas
Copy link
Collaborator Author

Hi! I'd love to see this, but am waiting for an update on this issue.

In the meantime, anyone's free to start on this!

@BasThomas BasThomas added the 👍 feature approved Features approved for implementation label Jul 27, 2019
@flowinho
Copy link

flowinho commented Aug 1, 2019

Seems like dark mode will kinda come for free with iOS 13.

Only for UI that is using SwiftUI. As this is not applicable for GitHawk, it will not automatically transition to supporting DarkMode.

Definitely throwing my vote in for a dark mode in iOS 13+ (previous versions not so much; let's aim for the MVP of getting it implemented, and if there's enough demand it can maybe be backported...)

This would mean re-implementing everything either in SwiftUI (which is imho not yet production ready) or implementing it in one of the old-fashioned ways.

Dark mode exclusive to iOS 13+ using OS systems sounds like a simpler solution and less maintenance longer term.

I definitely second this. However this would require a significant amout of refactoring.

@ideologysec
Copy link

ideologysec commented Aug 1, 2019

Only for UI that is using SwiftUI. As this is not applicable for GitHawk, it will not automatically transition to supporting DarkMode.

Uhhh.... no. SwiftUI automatically supports Dark Mode, but it is in no way a requirement for it. Have you read the API and design docs? (https://developer.apple.com/documentation/appkit/supporting_dark_mode_in_your_interface)

@flowinho I'd respond to the rest of your quotes but it's as wrong as this - seems like your post is based on the WWDC keynote and extrapolations?

Yes, there's more work required to create a dark theme, but it doesn't require a reimplementation in SwiftUI, and it's a little easier than "the old-fashioned ways."

As the above linked article says, "The system does a lot of the work for you, and if your app uses standard views and controls, you might not need to make many changes. Standard views and controls automatically update their appearance to match the current interface style. If you already use color and image assets, you can add dark variants without changing your code."

@ijm8710
Copy link

ijm8710 commented Aug 1, 2019

This is above my head but just be aware of #2553; not sure if the WIP helps at all, especially since I think that was done before it was announced that dark mode would be more natively supported, but just didn’t want it to fall completely on the wayside if it did have value

@kocheck
Copy link
Collaborator

kocheck commented Aug 20, 2019

I have been deep into the dark mode docs for another design project this past month. I think I am going to take a crack at using the default apple dark mode stuff here if you all are still interested. I know a lot of refactoring is required.

Started working on this design again here.

@kocheck
Copy link
Collaborator

kocheck commented Aug 21, 2019

I started working out the color styles tonight, trying to use as much of what apple gives us as possible. Looks like the designs files (overall) are a little behind production and could use some love ❤️.

I should have the color guide done first. Following the screen designs. I think I am going to recreate the screens from what I can see in production right now.

Light and Dark Mode colors:

image

Going to keep chipping away at this 👍

image

@raygervais
Copy link

Any update on this @kocheck ?

@Huddie
Copy link
Collaborator

Huddie commented Nov 14, 2019

I think the GitHub Mobile app takes care of this issue 😀

@raygervais
Copy link

You’re not wrong haha! Still, diversity in the app space doesn’t hurt anyone. Would be happy to try my hand at this (very new to iOS Open Source) still.

@BasThomas
Copy link
Collaborator Author

Go for it 😊

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
👍 feature approved Features approved for implementation
Projects
None yet
Development

No branches or pull requests