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

Tabler icons not found after updating plugin #239

Closed
rben01 opened this issue Sep 16, 2023 · 4 comments
Closed

Tabler icons not found after updating plugin #239

rben01 opened this issue Sep 16, 2023 · 4 comments
Labels
question Further information is requested

Comments

@rben01
Copy link

rben01 commented Sep 16, 2023

Describe the bug
All of my folders’ icons are Tabler icons. Despite having the Tabler icon pack installed, and its ZIP present in .obsidian/plugins/obsidian-icon-folder/icons, when I start up Obsidian, I get a million notifications saying “icon not found”, and icons are not shown for my vault’s folders.

To Reproduce
Steps to reproduce the behavior:

  1. Install the Tabler icon pack
  2. Set many folders’ icons to Tabler icons
  3. Fast forward several months
  4. Update the plugin after not updating it for a few months
  5. Restart Obsidian
  6. Bug: a million “icon missing” notifications, and icons aren't visible

Expected behavior
No issue; folder icons remain visible

Screenshots
It is almost certainly related to this, the contents of .obsidian/plugins/obsidian-icon-folder:

    - .
10.0k ├── data.json
    - ├── icons
    - │  ├── tabler-icons
  569 │  │  └── Template.svg
  28M │  └── tabler-icons.zip
 803k ├── main.js
  309 ├── manifest.json
 2.3k └── styles.css

After updating, it looks like the set of icons was replaced with a ZIP, but the icons in use were not extracted into a folder correctly. The file tabler-icons/Template.svg was only created when I set the icon of a folder after updating.

Additional context
The data in data.json is still intact, so in theory this could be fixed by running through that file and extracting all the necessary icons.

@rben01 rben01 changed the title Tabler icons not found Tabler icons not found after updating plugin Sep 16, 2023
@FlorianWoelki
Copy link
Owner

I don't quite understand it. The plugin never removes or deletes any icons when they are updated. Normally, after the recent update, you should've just downloaded the tabler icons pack as a zip and everything would have worked.

What you could do to resolve the issue, is just extracting all the icons into the tabler icons directory.

@FlorianWoelki FlorianWoelki added the question Further information is requested label Sep 16, 2023
@rben01
Copy link
Author

rben01 commented Sep 16, 2023

Looks like it may be related to the change here (#153) not taking into account the icons that are needed in the vault before cleaning things up.

@FlorianWoelki
Copy link
Owner

Got it. I think I could make something asynchronous in the background that checks in the beginning, if all used icons are downloaded/added correctly. That would fix some things.

I'll create a new issue for that and close this for now!

@FlorianWoelki
Copy link
Owner

Keeping track of it at #242

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants