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

Flat and more unified look for icons and menu items #19

Merged
merged 5 commits into from Nov 8, 2015

Conversation

bonsak
Copy link

@bonsak bonsak commented Nov 7, 2015

Updated the toolbar icons and included the menu items. Included a
plugin icon as well.

Updated the toolbar icons and included the menu items. Included a
plugin icon as well.
@bonsak
Copy link
Author

bonsak commented Nov 7, 2015

All icons are now 64x64 which seems to be standard in the native c4d icons

@bonsak
Copy link
Author

bonsak commented Nov 7, 2015

Icon for Combo Button is missing. Some work still to be done regarding readability. Just did a quick mockup in PS:
readability

@bonsak
Copy link
Author

bonsak commented Nov 7, 2015

Mockup of menu. Still needs some tweaking.
menu-mockup

@NiklasRosenstein
Copy link
Collaborator

Looking good! Yes the Combo Button was also missing in the original icon set. Would be nice to have a button for it as well.

Afaik, you can't have an icon for a menu "folder" though.

@NiklasRosenstein NiklasRosenstein added this to the v1.3 milestone Nov 7, 2015
@NiklasRosenstein NiklasRosenstein self-assigned this Nov 7, 2015
@bonsak
Copy link
Author

bonsak commented Nov 8, 2015

Your right. The folder will just get a default grey.

Fixed readability issues with small sizes.
No longer needed
Changed these icons to match the others
@NiklasRosenstein
Copy link
Collaborator

Very nice, thanks!

Is it possible to have different icons for Dialog, Icons and String tables in the browser window? Or shall i leave it like it is?

I'm thinking about removing the icons from the Browser window completely, it makes no sense since they all have the same icon.

@bonsak
Copy link
Author

bonsak commented Nov 8, 2015

Makes sense.

@NiklasRosenstein
Copy link
Collaborator

Only the "Source Wizard" is a bit hard to recognize.

image
image

@NiklasRosenstein NiklasRosenstein merged commit bb3ff8c into nrosenstein-c4d:master Nov 8, 2015
@NiklasRosenstein
Copy link
Collaborator

Not a biggie IMHO, doesn't need to be changed right now.

@bonsak
Copy link
Author

bonsak commented Nov 8, 2015

Agree. Diff and Compare also needs a little bit more love. Looks ok in general though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants