-
Notifications
You must be signed in to change notification settings - Fork 9
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
Merge into main repo? #9
Comments
Should be doable - would be good to have it confirmed working flawlessly on win+linux first though. And once that's done you're probably a better person to suggest it on the main repo - I can't even get a fix for the menu arrows through ;) |
Best to ship and iterate in my opinion :) I've asked @frank-trampe to focus on helping you get your designs in the next release that is due in about a week's time, but as a volunteer what he does it up to him :) |
Sounds great - there are a couple of controversial choices if the design is to be complete such as updating the icon for all platforms and removing all menu icons (I feel quite strongly about this - from a UX perspective most of them are very bad and the rest are superfluous) |
I want to follow your leadership on this :)
|
ok, I'll do a short writeup + add the necessary pr's later today and let's see what happens :) |
@davelab6 is it possible for you to test the theme on win/linux ? My virtual machine is very buggy right now:) |
@kosalaya Can you please test the theme on Win and report :) |
@kosalaya I've just pushed an update in the |
I wonder if we could merge this in to the main repo, or otherwise see it included in the mac package?
The text was updated successfully, but these errors were encountered: