GitHawk icon indicating copy to clipboard operation
GitHawk copied to clipboard

Dark mode / night theme

Open BasThomas opened this issue 6 years ago • 24 comments

Would love this. Definitely not too important though.

BasThomas avatar Jul 28 '17 20:07 BasThomas

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

shaps80 avatar Jul 29 '17 09:07 shaps80

This could go together with a dark app icon as well

BasThomas avatar Jul 29 '17 13:07 BasThomas

@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:

Sherlouk avatar Oct 10 '18 18:10 Sherlouk

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

kocheck avatar Nov 12 '18 18:11 kocheck

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!

Sherlouk avatar Nov 12 '18 19:11 Sherlouk

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

rnystrom avatar Nov 12 '18 23:11 rnystrom

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

ggliv avatar May 26 '19 03:05 ggliv

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.

BasThomas avatar May 28 '19 18:05 BasThomas

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

Sent with GitHawk

Huddie avatar Jun 04 '19 16:06 Huddie

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

jdisho avatar Jun 04 '19 20:06 jdisho

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.

Sherlouk avatar Jun 04 '19 23:06 Sherlouk

Would love to see one too :)

Sent with GitHawk

ghost avatar Jul 19 '19 19:07 ghost

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...)

ideologysec avatar Jul 26 '19 04:07 ideologysec

@BasThomas Any news?

ggliv avatar Jul 26 '19 22:07 ggliv

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 avatar Jul 27 '19 11:07 BasThomas

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.

flowinho avatar Aug 01 '19 08:08 flowinho

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."

ideologysec avatar Aug 01 '19 09:08 ideologysec

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

ijm8710 avatar Aug 01 '19 13:08 ijm8710

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 avatar Aug 20 '19 17:08 kocheck

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

kocheck avatar Aug 21 '19 02:08 kocheck

Any update on this @kocheck ?

raygervais avatar Oct 23 '19 23:10 raygervais

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

Huddie avatar Nov 14 '19 00:11 Huddie

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.

raygervais avatar Nov 14 '19 18:11 raygervais

Go for it 😊

BasThomas avatar Nov 18 '19 21:11 BasThomas