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

Support macOS Mojave dark mode #80

Closed
nrbrook opened this issue Sep 29, 2018 · 13 comments
Closed

Support macOS Mojave dark mode #80

nrbrook opened this issue Sep 29, 2018 · 13 comments

Comments

@nrbrook
Copy link

nrbrook commented Sep 29, 2018

I'd love to see Mojave support, I need it for my App. I will look in to adding support if I get chance

@Kentzo
Copy link
Owner

Kentzo commented Sep 29, 2018

Sounds great. Please take a look at the #70 by @JanX2. It has updated assets. However, I'm not sure at this point whether it will be merged as is.

@nrbrook
Copy link
Author

nrbrook commented Sep 29, 2018

Thanks, I’ve been working on it today. Do you have any tips on creating the background images - are they copies of existing UI elements? What about the snap back and clear icons?

@Kentzo
Copy link
Owner

Kentzo commented Sep 29, 2018

If I recall correctly the assets of the control are based on the button styled as "Textured Rounded". The clear and snap are system assets as is (at least used to be).

I used https://github.com/insidegui/AssetCatalogTinkerer to extract them originally.

@aljungberg may recall more :)

@nrbrook
Copy link
Author

nrbrook commented Sep 30, 2018

When are the 'highlighted' images used and what are they based on?

@Kentzo
Copy link
Owner

Kentzo commented Sep 30, 2018

I think originally they were based on the system assets. Then they were redrawn by @aljungberg and then again by @jwells89 in #33.

The control used to draw different background when mouse pointer was over it. This feature was retained although current assets do not have assets for this state.

@Kentzo
Copy link
Owner

Kentzo commented Sep 30, 2018

I'm thinking of an abstraction to allow simpler control customization (asset-wise). Do not worry about code.

@jwells89
Copy link

@Kentzo I think I still have the Sketch file for this laying around somewhere. Would it be helpful to use them to make dark mode assets?

@nrbrook
Copy link
Author

nrbrook commented Sep 30, 2018

Please see #82

Please improve the assets as you see fit, the photoshop files are in Resources
screenshot 2018-09-30 at 22 50 07

@Kentzo
Copy link
Owner

Kentzo commented Oct 1, 2018

@jwells89 Is it different than https://github.com/Kentzo/ShortcutRecorder/blob/develop/Resources/ShorcutRecorder%20Yosemite.sketch ?

@nrbrook Great Work!

Would it be possible to provide a sketch file instead?
Also please let me know whether new dark mode is based on #82 or current appearance. Would be very helpful, if you could put a button next to the SR control.

@jwells89
Copy link

jwells89 commented Oct 1, 2018

@Kentzo That’s the one, forgot that I committed it.

@nrbrook
Copy link
Author

nrbrook commented Oct 1, 2018

@Kentzo sorry I’ve not used sketch before and did it directly in photoshop. It’s all done with effects though so is editable/transferable.
Dark mode is included in #82. Includes code to load the appropriate images if the appearance changes, as we can’t do it the automatic way using appearance specific assets as we support older macOS (not sure why Apple didn’t seem to consider this possibility and document alternative methods, but I think what I do in #82 is the way it should be done)

@aljungberg
Copy link

If I recall correctly the assets of the control are based on the button styled as "Textured Rounded". The clear and snap are system assets as is (at least used to be).

I used https://github.com/insidegui/AssetCatalogTinkerer to extract them originally.

@aljungberg may recall more :)

Yeah I drew them by hand in Photoshop based on UI widgets. It's just some gradients, strokes, a rounded rectangle. I'd probably have used Sketch if I did it today.

@Kentzo
Copy link
Owner

Kentzo commented Jun 1, 2019

Part of the Shortcut Recorder 3.0

@Kentzo Kentzo closed this as completed Jun 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants