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

[REQUEST] Better icon for git plus output console #613

Closed
schrej opened this issue Feb 2, 2017 · 6 comments
Closed

[REQUEST] Better icon for git plus output console #613

schrej opened this issue Feb 2, 2017 · 6 comments

Comments

@schrej
Copy link

schrej commented Feb 2, 2017

The little pin icon in the bottom right is not obviously related to git-plus at all. If not hovered over out of interest one does not know what it does at all.

I would propose to replace it with another icon or just text to make it more meaningful.

The fontawesome git icon would be quite a good candidate, it is not included within atom though.

Replacing it with git+ is also a good alternative in my opinion if including the whole fontawesome font is to much overhead:
screenshot

@cbcoutinho
Copy link

I second this idea - I think it would be great to change the logo to git+ instead of a pin as mentioned by @schrej.

Also, right now the little pin is always visible and doesn't seem to be related to version control. If the open file is not under version control, the pin just sits there. Is it possible to make it visible only when the file is in a version-controlled project? That, or maybe it goes from a darker less visible color to a more highlighted color?

@akonwi
Copy link
Owner

akonwi commented Feb 7, 2017

This is a good idea. I'll change it to 'git+'. I think with an icon that just says 'git' then it could be mistaken as something else.

@cbcoutinho I'll also change it to only show the icon when the package is fully activated so it's only visible when a git repository is in the project.

@akonwi akonwi closed this as completed in bd489a9 Feb 7, 2017
@cbcoutinho
Copy link

Hey @akonwi, icon looks great,

Sorry to re-open the issue, but I don't think the package activation is working correctly, or at least how I expect it should:

  • Removing all git projects from the tree view does not remove the git+ icon, requires atom restart
  • Opening atom without any previous git projects has no git+ icon, but opening a project into the tree view, or any of its files, doesn't open up the git+ icon. Also the short cuts don't work - only git init. Also requires atom restart

I'm was envisioning the package being more dynamic, but all these restarts are unnecessary. I think it's an question of either/or, just keep it always present. That's better than having the restart atom to get the package to work.

If you want me to open a new issue, let me know

@akonwi
Copy link
Owner

akonwi commented Feb 8, 2017

I can remove the icon when projects are removed. I'm not sure that I can add the icon when a project is added though because I'm consuming the status-bar package's service to create the icon, so I'll need to figure out if it's possible to manually invoke the service consumption. And even if it is, I probably shouldn't do that because those API's change.

Maybe I will just always leave the pin there and disable it when there isn't a git repository in the workspace.

If you open a blank atom workspace. No git commands will be available. If you add a project which is not a repository, the init command will be available in the regular command palette. Once you add a git repository, then all the git-plus commands will be activated.

@akonwi akonwi reopened this Feb 8, 2017
@akonwi
Copy link
Owner

akonwi commented Feb 8, 2017

Update: I've managed to re-add the icon when repositories are introduced into the workspace

@akonwi akonwi closed this as completed in eba7fc9 Feb 8, 2017
@cbcoutinho
Copy link

cbcoutinho commented Feb 8, 2017 via email

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

No branches or pull requests

3 participants