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

v1.7.9 update causes multiple files under git version control to change to the modified color when only one file is edited #352

Closed
AriaFallah opened this issue May 6, 2016 · 14 comments
Assignees
Labels
bug Confirmed defect in package logic, deprecation notices, and PRs which fix them.

Comments

@AriaFallah
Copy link

Previous discussion here: #341

@Alhadis
Copy link
Member

Alhadis commented May 6, 2016

Alright, could you open your installed version of File-Icons and make this change?

I just want to make doubly sure this is the catalyst, because I'm still at a loss as to why I can't replicate it.

@AriaFallah
Copy link
Author

AriaFallah commented May 6, 2016

@Alhadis that did it. Let me see if I can reproduce in another project except this one.

@AriaFallah
Copy link
Author

AriaFallah commented May 6, 2016

It happens if I modify a file in the src directory of this project: https://github.com/AriaFallah/mobx-store

Could you try cloning it and see if you can reproduce there?

@Alhadis
Copy link
Member

Alhadis commented May 6, 2016

Heh, nope... nothing. :|

At least my hunch about e93cbf5 was right. Will address this by individually targeting every icon that uses the .medium-yellow swatch and assigning the darker colour there.

You'll notice we can't simply do .status-modified .medium-yellow, because .medium-yellow is a Less mixin, and not a class that's actually applied to the file's tree-view entry. :D I LUV PRECOMPILERZ <3 <3 #noNotReally

This is another reason I'm looking forward to releasing v2.

@AriaFallah
Copy link
Author

Ah shucks. Well good luck! I'm not sure why you can't reproduce 😕

@Alhadis
Copy link
Member

Alhadis commented May 6, 2016

Neither am I. When I'm done, I'll need you to help verify the changes work. So, don't go too far. :p

@Alhadis Alhadis self-assigned this May 6, 2016
@Alhadis Alhadis added the bug Confirmed defect in package logic, deprecation notices, and PRs which fix them. label May 6, 2016
@Alhadis
Copy link
Member

Alhadis commented May 6, 2016

Alright... I've pushed what I hope is a fix. Run the following in your shell, please:

git clone https://github.com/DanBrooker/file-icons.git ~/.atom/dev/packages/file-icons
atom -d your/project/path/here

That'll open the folder in Atom using development mode.

Let me know if this fixes the issue. It should, but I'd prefer to be sure.

@AriaFallah
Copy link
Author

AriaFallah commented May 6, 2016

I also did rm -rf ~/.atom/packages/file-icons to be doubly sure that I was using the git cloned one, but it still is not working 😢

Here is a sad short story documenting the issue: https://streamable.com/0laq. As you can see, I'm terrible at making sad faces, and all the files still change color when I save the changes to just one of them.

@Alhadis
Copy link
Member

Alhadis commented May 6, 2016

The hell. Does this happen with any other repository, or just this one...?

@AriaFallah
Copy link
Author

@Alhadis All repos unfortunately. For the time being, I'm fine with using 1.7.8 forever 😅 don't worry about it too much.

@Alhadis
Copy link
Member

Alhadis commented May 6, 2016

Well, when v2 gets released (hopefully sooner rather than later), you'll definitely want to update. :p And other users might be affected by this, not just you (so I'd prefer this does get fixed).

@Alhadis
Copy link
Member

Alhadis commented May 6, 2016

Honestly, I think the easiest thing to do is simply revert the fix for #341 after all. All of this trouble's being caused just by adjusting a certain shade of orange to be more noticeable when it's modified.

@Alhadis Alhadis closed this as completed in f6b2f05 May 6, 2016
@Alhadis
Copy link
Member

Alhadis commented May 6, 2016

@AriaFallah, I've published a patch release. Please let me know if this happens again.

@AriaFallah
Copy link
Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed defect in package logic, deprecation notices, and PRs which fix them.
Projects
None yet
Development

No branches or pull requests

2 participants