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

Fix icons #731

Merged
merged 5 commits into from
Nov 25, 2015
Merged

Fix icons #731

merged 5 commits into from
Nov 25, 2015

Conversation

daxgames
Copy link
Member

Added Git for Windows icon to mintty tasks to differentiate them from 100% supported Cmder tasks

Get 100% supported Cmder task icons from icons\Cmder.ico so tabs have an ico if launched from the cmder.bat file

@Stanzilla
Copy link
Member

Was the current way of trying to get the icon from the exe not working?

@daxgames
Copy link
Member Author

It would probably work fine if I could build the cmder.exe but I can't because I do not have the tools. I, and anyone else trying/developing on the dev branch that cannot build the exe, use the cmder.bat to start cmder hence no icons.

I just did what I did so it would work all the time with or without the cmder.exe. To me working all the time was better than working some of the time even if the cmder.bat is deprecated.

@Stanzilla
Copy link
Member

Makes sense

@MartiUK
Copy link
Member

MartiUK commented Nov 24, 2015

Any possibility of moving that last commit to a separate pull request? It has nothing to do with fixing the icons 😄

@daxgames
Copy link
Member Author

Sure. Will do later today.

@daxgames
Copy link
Member Author

Done

MartiUK added a commit that referenced this pull request Nov 25, 2015
@MartiUK MartiUK merged commit 182d0c1 into cmderdev:development Nov 25, 2015
@MartiUK
Copy link
Member

MartiUK commented Nov 25, 2015

Thanks.

@daxgames daxgames deleted the fix_icons branch February 29, 2016 13:45
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.

3 participants