Skip to content
This repository has been archived by the owner on Feb 6, 2022. It is now read-only.

Updated Icon #102

Closed
wants to merge 3 commits into from
Closed

Updated Icon #102

wants to merge 3 commits into from

Conversation

kgn
Copy link

@kgn kgn commented Oct 25, 2015

I'm loving Helium! It's awesome, so I wanted to make a new icon for it so it fit in the dock better with the mac icon style. I hope you like it :)

screen shot 2015-10-25 at 5 29 40 pm

@JadenGeller
Copy link
Owner

I appreciate the contribution, but I going to stick with the current icon for now. People already know the app by the current colorful icon, so I think such a drastic change would hurt recognition.

@net
Copy link
Contributor

net commented Oct 27, 2015

@JadenGeller You should consider option to move the app to the menubar. The current icon does just a little bit out of place in El Capitan.

@JadenGeller
Copy link
Owner

That's definitely a reasonable thing to consider.

@JadenGeller
Copy link
Owner

@net Thinking a bit about it, I'm hesitant to do that. The app does not always float on top—it can be temporarily disabled, or the app can be hidden—without a dock icon, it is difficult for users to find the window. I think that windowed apps really ought to show in the dock.

Is the motivation for menu bar support simply hiding the icon?

@net
Copy link
Contributor

net commented Oct 28, 2015

@JadenGeller Think of how much functionality could be moved to the menu bar. You could move all the menus there. You could have a hide/show toggle in the menu bar menu. Then you wouldn't have to select the helium window before accessing menu actions.

@JadenGeller
Copy link
Owner

But all apps require you to select the window before accessing the menu. Why should Helium act differently in this regard?

@net
Copy link
Contributor

net commented Oct 28, 2015

I'm talking about this type of menu.

screen shot 2015-10-27 at 8 03 30 pm

@JadenGeller
Copy link
Owner

Transitioning to #107

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

Successfully merging this pull request may close these issues.

None yet

3 participants