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

Icon update #1119

Merged
merged 10 commits into from
Jan 22, 2017
Merged

Icon update #1119

merged 10 commits into from
Jan 22, 2017

Conversation

daschuer
Copy link
Member

I have updated some icons to be more significant and not ambigious for future feature enhancements.
See commits.
I have also renamed the Library treeview entry to Tracks since all is finally in the library.

I hope that this will help to avoid the confusion between Library and Browse.

I would also like to rename the browse feature to something more significant since Browse can be anything. Do you have an idea?

@daschuer
Copy link
Member Author

unfortunately this removes the color transition from the icons. @esbrandt: what is the receipt for the transition?

@daschuer daschuer mentioned this pull request Jan 13, 2017
11 tasks
@Be-ing
Copy link
Contributor

Be-ing commented Jan 13, 2017

LGTM!

These icons should be vectorized. That does not need to be fixed right now. I have opened a bug to track it.

@rryan
Copy link
Member

rryan commented Jan 13, 2017

Nice, @daschuer did you draw these by hand? If not, is any of the source material copyright?

@ferranpujolcamins
Copy link
Contributor

ferranpujolcamins commented Jan 13, 2017 via email

@daschuer
Copy link
Member Author

The musical note and the computer are from the gnome high contrast theme with LGPL 2.1
The Interface icon is my own work.
The iTunes and tractor icons are redrawn from original icons found on their websites.

@Be-ing
Copy link
Contributor

Be-ing commented Jan 14, 2017

New name for Browse: what about "HDD", "system folders", or something like this, that references the fact that you are looking at whats in the harddrive as is?

I think "Files" or "File system" would work better.

@daschuer
Copy link
Member Author

daschuer commented Jan 14, 2017

New name for Browse: what about "HDD", "system folders", or something like this, that references the fact that you are looking at whats in the harddrive as is?

"System Folders" sounds nice. Does this cover also implies jump drives?
"HDD" might be too trecky, maybe unroll HDD to "Drives"

Gnome used the term "Places" but hat will probably only be understandable by Linux users.

Any other votes?

@daschuer
Copy link
Member Author

Yes "File system", but that might confuse windows users who are thinking in "Drive Letters".

@Be-ing
Copy link
Contributor

Be-ing commented Jan 14, 2017

Yes "File system", but that might confuse windows users who are thinking in "Drive Letters".

I'm confused how it would be confusing. How does the feature behave on Windows? I'm guessing it shows the C drive as the root of the file system tree?

@daschuer
Copy link
Member Author

On Linux we have

  • Quick links
  • Jump Drives
  • / (root folder of file system)

On windows we have

  • Quick links
  • Devices
    ** C:
    ** D:

If we agree "File system" is not confusing we can go for it.

@Be-ing
Copy link
Contributor

Be-ing commented Jan 14, 2017

I think "File system" would work for Windows too. It wouldn't be quite as clear for Windows, but I think it's the best option proposed so far. Any other opinions?

@rryan
Copy link
Member

rryan commented Jan 14, 2017

"File system" seems too technical to me, what do you think of:

  • File Explorer
  • Explorer
  • My Computer
  • Computer

"Files" also sounds good to me, and is short and parallel to "Tracks".

@Be-ing
Copy link
Contributor

Be-ing commented Jan 14, 2017

"Computer" is clear and works well with Windows terminology. It also matches the new icon.

@Be-ing
Copy link
Contributor

Be-ing commented Jan 14, 2017

The musical note and the computer are from the gnome high contrast theme with LGPL 2.1

This should be acknowledged somewhere.

@Be-ing
Copy link
Contributor

Be-ing commented Jan 14, 2017

Rename res/images/library/ic_library_library.png to res/images/library/ic_library_tracks.png. Also rename the browse icon file for whatever new name we decide.

@daschuer
Copy link
Member Author

Windows 7 uses "Computer" Windows 10 uses "This PC" Mac OS, just "MAC", Linux has the "Computer://" place where the Icon is taken from.
How about just hit the Middle of windows 10 and 7: "This Computer" since this computer is a MAC on a MAC, it fits also there ;-)

@daschuer
Copy link
Member Author

OK, now we have "This Computer" and polished icons with color gradients.

One day, we should rename the C++ classes as well, but I like to postpone this to avoid conflicts and hard to track diffs, when merging the the new library layout.

@Be-ing
Copy link
Contributor

Be-ing commented Jan 14, 2017

How about just hit the Middle of windows 10 and 7

IMO "This Computer" feels kinda weird because it is a middle ground that no other system uses exactly. I think just "Computer" works better.

@rryan
Copy link
Member

rryan commented Jan 14, 2017

👍 to "Computer" as well

@Be-ing
Copy link
Contributor

Be-ing commented Jan 14, 2017

Can we distribute LGPL icons in the Mac App Store? I think the copyright should be acknowledged in the LICENSE file. It has been a long time since I read the LGPL 2.1. Do we need to distribute the full text of that license?

@timrae
Copy link
Contributor

timrae commented Jan 15, 2017

@daschuer
Do have svg's for all of these icons? If you already have them it definitely make sense to add them to the source tree now, rather than hunting for them later.

@daschuer
Copy link
Member Author

Sorry, no. That was just pixel shifting in Gimp

@daschuer
Copy link
Member Author

LGPL should be no problem for Apple. I have just found that:
https://forums.developer.apple.com/thread/27709
I think if this would be not allowed, we have a problem with the whole Qt library under LGPL

@ferranpujolcamins
Copy link
Contributor

ferranpujolcamins commented Jan 15, 2017 via email

@esbrandt
Copy link
Contributor

esbrandt commented Jan 15, 2017

unfortunately this removes the color transition from the icons. @esbrandt: what is the receipt for the transition?

@daschuer
sorry for being late. Moved recently and RL happened.
The icon templates are as always in https://github.com/mixxxdj/mixxx/blob/master/res/images/templates/ic_template_library_and_preferences.svg

Multi layered inkscape template, incl. filter settings for the overlays used. If the icons change , i´d love to see the template updated as well.

EDIT: The icons are public domain, as stated in the templates metadata.

@daschuer
Copy link
Member Author

Sorry, for now I have no svg versions of the new icons so I cannot update ic_template_library_and_preferences.svg

@daschuer
Copy link
Member Author

I think we can merge now. Icon tweaking towards svg can be done in a separate PR.

@daschuer daschuer merged commit 43efad8 into mixxxdj:master Jan 22, 2017
@daschuer daschuer deleted the icon_update branch September 7, 2021 21:06
@mixxxbot mixxxbot mentioned this pull request Aug 23, 2022
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.

6 participants