-
-
Notifications
You must be signed in to change notification settings - Fork 625
feat(renderer): allow placing git icons after filename #1203
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
Conversation
i guess, see my comment before :) this is not perfect yet, the algorithms are very clunky as it is. |
Wasn't complaining, just reporting, PR is in progress, of course. :) |
6bd0637
to
4379ea1
Compare
@gegoune could you give it another try ? i've fixed multiple issues and also added icons for symlinks :) |
Highlighting issue seems to be fixed now. Unrelated, and I think there already is an issue for it, but why not put those markers in signcolumn? |
Nice. I'll give this a test / review on the weekend. |
the signcolumn is another issue, i'll implement this after :) one thing at a time ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This definitely works. Tested files and folders: before, after, foo
RE symlink git icons: They are not really part of this change. They can be made to work with some tweaking of Raised #1217. I've pushed a WIP branch that I'll get back to later. |
@alex-courtis do you want me to remove the code that adds the git icons for symlinks ? It works fine on my end though. |
6d13b8f
to
5d25057
Compare
No! I've tested it. Once we resolve #1217 everything will display correctly. |
2f250c9
to
f2aba7f
Compare
- fix last issues with git placement highlighting - create a formatting function for simplifying code - add git icon in symlink
allows for better consistency
f2aba7f
to
206ee78
Compare
This feature allows placing git icons after the filename.
This is a first draft, i'd like to improve the code a bit before merging because it's not pretty as it is, but usable.
This also change the behavior for special icons, which will display icons normally instead of the default file icon.
TODO:
implements #246, #1069 and #72, #1217