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

Iconize 2.11.9 conflicts with windows icons and Metabind #438

Closed
Davve37 opened this issue Apr 8, 2024 · 35 comments
Closed

Iconize 2.11.9 conflicts with windows icons and Metabind #438

Davve37 opened this issue Apr 8, 2024 · 35 comments
Assignees
Labels
bug Something isn't working

Comments

@Davve37
Copy link

Davve37 commented Apr 8, 2024

Describe the bug
Iconize conflicts with windows 11 icons and cause lines where they exist to fail. It also disrupt some Metabind fields.

To Reproduce
Steps to reproduce the behavior:

  1. Enable iconize 2.11.9
  2. Go to note with windows 11 icons
  3. ???
  4. See error

Expected behavior
What I expected:
https://i.imgur.com/wwLGZTF.png
https://i.imgur.com/oaD8AGD.png

Screenshots
What I got:
https://i.imgur.com/ry8o1tC.png
https://i.imgur.com/goSQEgg.png

Additional context
Add any other context about the problem here.

@mayeenulislam
Copy link

I'm having a similar issue.
Any icon that is not within Iconize's radar is getting omitted from view in Editing mode.
But they are working in the Reading mode.

For example, my note has a hand emoji like: 👆, which was added using the Mac Emoji picker, which is behaving like this.
But if I want to pick the same emoji using : notation, it's not within the list of Icons.

I'm using Font Awesome Solid and Lucide Icons and for Emoji Style my choice is "Native"

@FlorianWoelki
Copy link
Owner

is it only for emojis? if so, could you check if changing the emoji mode in the settings to "native" fixes this.

@FlorianWoelki FlorianWoelki added the question Further information is requested label Apr 8, 2024
@mayeenulislam
Copy link

It's only for Emojis. When the cursor is on that line, it's visible, like below

image

But when the cursor is out:

image

I tried changing Emoji Style to "Twimoji" and Force Reload Obsidian

Then "None" and Force Reload.

The issue persists.

The Emojis only works fine when the Iconize is inactive (even without Force Reload)

@mayeenulislam
Copy link

I'm afraid it's affecting the Bulleted List as well. Iconize v2.11.9.

When Iconize is active:
image

When inactive:
image

@FlorianWoelki
Copy link
Owner

It's only for Emojis. When the cursor is on that line, it's visible, like below

image

But when the cursor is out:

image

I tried changing Emoji Style to "Twimoji" and Force Reload Obsidian

Then "None" and Force Reload.

The issue persists.

The Emojis only works fine when the Iconize is inactive (even without Force Reload)

what about having emojis set to native and then force reload?

@Davve37
Copy link
Author

Davve37 commented Apr 8, 2024

what about having emojis set to native and then force reload?

That helps a bit however it still destroys dataview tables when windows icons appear inside the dataview table.

https://i.imgur.com/kGOKCmW.png

@mayeenulislam
Copy link

My Emoji Style is "Native", it's the same.

However found a new bug using :FasAnglesUp: into the note:
image

When Iconize is active, opening the note by clicking on the title from Explorer giving an error:

🔴 Failed to load "MyNote.md". Ranges must be added sorted by 'from' position and 'startSide'

@I-Pch
Copy link

I-Pch commented Apr 8, 2024

I'm having a similar issue. Any icon that is not within Iconize's radar is getting omitted from view in Editing mode. But they are working in the Reading mode.

For example, my note has a hand emoji like: 👆, which was added using the Mac Emoji picker, which is behaving like this. But if I want to pick the same emoji using : notation, it's not within the list of Icons.

I'm using Font Awesome Solid and Lucide Icons and for Emoji Style my choice is "Native"

Makes me think about was I was seeing with Iconize 2.11.7, yesterday -> #412 (comment) (only tested with emojis/twemojis though) which I think I still get with Iconize 2.11.9 (but I didn't got the opportunity to properly test this yet)

Edit: Selected the wrong reply to quote 🙈

Edit-2: Setting the emojis style to any of the options available, then Reload app without saving also doesn't have any effect on the "ghost" emojis: they stay invisible 🤷‍♀️

Edit-3: I'm also having this issue 👇 ...

When Iconize is active, opening the note by clicking on the title from Explorer giving an error:

🔴 Failed to load "MyNote.md". Ranges must be added sorted by 'from' position and 'startSide'

@schemen
Copy link

schemen commented Apr 8, 2024

Same error here: bullet points are being lost

@FlorianWoelki
Copy link
Owner

would it be possible for someone to give me a minimal reproduction example. especially if it is affecting dataview + iconize the specific dataview query.

@primarist
Copy link

I'm also experiencing this but with the Tasks plugin. Tasks uses emojis to indicate different due dates, priorities, etc. The emojis are only visible when the cursor is placed in the line containing the emoji, but when you select a different line, the emoji disappears. Additionally, when that task is queried using Dataview, the emoji is replaced by the text "null".

@dkadev
Copy link

dkadev commented Apr 8, 2024

I'm experiencing this with inline emojis on macOS, I can see them in source mode but as soon as I change to read mode emoji is replaced with "null".
Disabling iconize plugin stops this behaviour. Happening since I updated today.
I have another vault with outdated version 2.11.4 and bug does not exist.

@I-Pch
Copy link

I-Pch commented Apr 8, 2024

would it be possible for someone to give me a minimal reproduction example. especially if it is affecting dataview + iconize the specific dataview query.

As a very minimal repro, I have this query :

> [!NOTE] Title
> ✨
> ```dataview
> TABLE file.ctime AS "Created"
> FROM "Test"
> SORT file.ctime DESC
> ```

This 👆 doesn't work and in fact breaks as soon as I add the :sparkles: emoji 😊
The issue seem to be linked to having a query within a callout + at least one emoji/icon (didn't test with icons)

While this query still works as expected :

> [!NOTE] Title
> 
> ```dataview
> TABLE file.ctime AS "Created"
> FROM "Test"
> SORT file.ctime DESC
> ```

In the screenshot below, the 3 Dataview queries are the same as the ones I shared here, just placed in different context 😊 .
Dataview Iconize

Edit: Same result using just :LiSparkles: (from Lucide)

Iconize - Dataview - Lucide Icon

I have only 14 notes in the vault I'm using to test the emojis issues 😊 and only 2 plugins (Dataview and Iconize 2.11.9).
I'm on Mac (Big Sur 11.7.10) + Obsidian 1.5.12 and my vault is stored on and sync through iCloud.
I only use the default Obsidian Dark Theme and have no CSS snippets.

Edit: Adding a very small screen-recording where all I do to break the Dataview query is adding an emoji ...

Iconize.Dataview.-.Callout.-.Emoji.mov

I hope this helps 😊

@ollecello
Copy link

ollecello commented Apr 9, 2024

I've the same massive problems on all levels since the latest update.

@FlorianWoelki
Copy link
Owner

would it be possible for someone to give me a minimal reproduction example. especially if it is affecting dataview + iconize the specific dataview query.

As a very minimal repro, I have this query :

> [!NOTE] Title
> ✨
> ```dataview
> TABLE file.ctime AS "Created"
> FROM "Test"
> SORT file.ctime DESC
> ```

This 👆 doesn't work and in fact breaks as soon as I add the :sparkles: emoji 😊 The issue seem to be linked to having a query within a callout + at least one emoji/icon (didn't test with icons)

While this query still works as expected :

> [!NOTE] Title
> 
> ```dataview
> TABLE file.ctime AS "Created"
> FROM "Test"
> SORT file.ctime DESC
> ```

In the screenshot below, the 3 Dataview queries are the same as the ones I shared here, just placed in different context 😊 . Dataview Iconize

Edit: Same result using just :LiSparkles: (from Lucide)

Iconize - Dataview - Lucide Icon I have only 14 notes in the vault I'm using to test the emojis issues 😊 and only 2 plugins (Dataview and Iconize 2.11.9). I'm on Mac (Big Sur 11.7.10) + Obsidian 1.5.12 and my vault is stored on and sync through iCloud. I only use the default Obsidian Dark Theme and have no CSS snippets.

Edit: Adding a very small screen-recording where all I do to break the Dataview query is adding an emoji ...

Iconize.Dataview.-.Callout.-.Emoji.mov
I hope this helps 😊

thank you! this really helped and i was able to reproduce it.

@FlorianWoelki FlorianWoelki added the bug Something isn't working label Apr 9, 2024
@FlorianWoelki FlorianWoelki self-assigned this Apr 9, 2024
@FlorianWoelki FlorianWoelki removed the question Further information is requested label Apr 9, 2024
@FlorianWoelki FlorianWoelki moved this to In Progress in Iconize Board Apr 9, 2024
@FlorianWoelki
Copy link
Owner

just fyi: i have a working solution, just need to adjust the code a bit and adapt it to icons as well. i'll keep you updated here.

@FlorianWoelki
Copy link
Owner

hey there,

I've pushed a commit to fix the latest issues. at least I am not able to reproduce any issue related to that anymore. could someone please check out the latest code and see if it works for them?

@I-Pch
Copy link

I-Pch commented Apr 10, 2024

hey there,

I've pushed a commit to fix the latest issues. at least I am not able to reproduce any issue related to that anymore. could someone please check out the latest code and see if it works for them?

Sorry for my potential ignorant comment but don't you need to release a new version for us to test ? 🙈
... or is there a specific process to follow ? (manual install maybe ?)

@FlorianWoelki
Copy link
Owner

hey there,

I've pushed a commit to fix the latest issues. at least I am not able to reproduce any issue related to that anymore. could someone please check out the latest code and see if it works for them?

Sorry for my potential ignorant comment but don't you need to release a new version for us to test ? 🙈

... or is there a specific process to follow ? (manual install maybe ?)

no worries. yes but i want to make sure that everything works before releasing it.

you can check the development guide in the readme to get some insights on how to test things locally. if it sounds too complicated i can also upload to built files to GitHub and you can download them.

@mProjectsCode
Copy link

You could setup beta releases with BRAT for this. That would allow users to download beta versions via BRAT, without pushing out beta releases to all users. I do that for almost all of my plugins.

@CoachKidd
Copy link

I've got this issue too... Thanks for taking a look at it... Is there a release date for it yet?

@FlorianWoelki
Copy link
Owner

probably this weekend.

@CoachKidd
Copy link

probably this weekend.

That's awesome news, thanks!

As an end user of Obsidian (my days of being a competent coder are long gone) are there any user tests I can do that would be helpful for you?

@FlorianWoelki
Copy link
Owner

You could setup beta releases with BRAT for this. That would allow users to download beta versions via BRAT, without pushing out beta releases to all users. I do that for almost all of my plugins.

yeah so I should setup BRAT in the future, thank you for the suggestion!

probably this weekend.

That's awesome news, thanks!

As an end user of Obsidian (my days of being a competent coder are long gone) are there any user tests I can do that would be helpful for you?

sure! because BRAT (or something similar) is not embedded yet, you can copy and paste the main.js file to the Iconize plugin (sorry I cannot upload the file directly). This is the link.

These are the steps you need to follow:

  1. Quit Obsidian entirely
  2. Go to your Vault's directory (where you see the .obsidian directory)
  3. Go to: /.obsidian/plugins/obsidian-iconize (or obsidian-icon-folder)/
  4. Paste the downloaded main.js file into this directory

Then, hopefully, things will work.

@CoachKidd
Copy link

CoachKidd commented Apr 11, 2024

Thanks @FlorianWoelki ... (obsidian-icon-folder)/ works perfectly for me.

Do you have a "Buy me a coffee" link?

@FlorianWoelki
Copy link
Owner

Thanks @FlorianWoelki ... It works perfectly for me.

Do you have a "Buy me a coffee" link?

that's really lovely!

I do, but obviously it's not required. always appreciate the support!

@I-Pch
Copy link

I-Pch commented Apr 11, 2024

You could setup beta releases with BRAT for this. That would allow users to download beta versions via BRAT, without pushing out beta releases to all users. I do that for almost all of my plugins.

yeah so I should setup BRAT in the future, thank you for the suggestion!

probably this weekend.

That's awesome news, thanks!
As an end user of Obsidian (my days of being a competent coder are long gone) are there any user tests I can do that would be helpful for you?

sure! because BRAT (or something similar) is not embedded yet, you can copy and paste the main.js file to the Iconize plugin (sorry I cannot upload the file directly). This is the link.

These are the steps you need to follow:

  1. Quit Obsidian entirely
  2. Go to your Vault's directory (where you see the .obsidian directory)
  3. Go to: /.obsidian/plugins/obsidian-iconize (or obsidian-icon-folder)/
  4. Paste the downloaded main.js file into this directory

Then, hopefully, things will work.

Thank you for sharing the file 😄 !

I just ran some tests (in the same vault I used before) and it's way better from my side of the screen 😊 :

I was able to:

  • Have Dataview queries in a callout + icon/emoji but only if I used Native emojis (using Twemojis, apparently lead to the freezing of the whole vault 🤷‍♀️ ... where all I could do was scroll through the specific note I used for testing purpose here > Iconize 2.11.9 conflicts with windows icons and Metabind #438 (comment).
    I had to delete the workspace.json to solve the issue (because the vault would open on the note that broke it 😅 , so rebooting wasn't enough 😇 )
  • Open notes in a split view 🎉 !!!

Emojis still seems to render when "they want to" in Live Preview 😊 (#438 (comment))

Edit: Regarding the emojis 😊 : Setting them to None doesn't seems to render any emojis in Live Preview while setting them to Native render them "once in a while" (depending on the emoji) in Live Preview...

@FlorianWoelki
Copy link
Owner

You could setup beta releases with BRAT for this. That would allow users to download beta versions via BRAT, without pushing out beta releases to all users. I do that for almost all of my plugins.

yeah so I should setup BRAT in the future, thank you for the suggestion!

probably this weekend.

That's awesome news, thanks!
As an end user of Obsidian (my days of being a competent coder are long gone) are there any user tests I can do that would be helpful for you?

sure! because BRAT (or something similar) is not embedded yet, you can copy and paste the main.js file to the Iconize plugin (sorry I cannot upload the file directly). This is the link.
These are the steps you need to follow:

  1. Quit Obsidian entirely
  2. Go to your Vault's directory (where you see the .obsidian directory)
  3. Go to: /.obsidian/plugins/obsidian-iconize (or obsidian-icon-folder)/
  4. Paste the downloaded main.js file into this directory

Then, hopefully, things will work.

Thank you for sharing the file 😄 !

I just ran some tests (in the same vault I used before) and it's way better from my side of the screen 😊 :

I was able to:

  • Have Dataview queries in a callout + icon/emoji but only if I used Native emojis (using Twemojis, apparently lead to the freezing of the whole vault 🤷‍♀️ ... where all I could do was scroll through the specific note I used for testing purpose here > Iconize 2.11.9 conflicts with windows icons and Metabind #438 (comment).
    I had to delete the workspace.json to solve the issue (because the vault would open on the note that broke it 😅 , so rebooting wasn't enough 😇 )
  • Open notes in a split view 🎉 !!!

Emojis still seems to render when "they want to" in Live Preview 😊 (#438 (comment))

Edit: Regarding the emojis 😊 : Setting them to None doesn't seems to render any emojis in Live Preview while setting them to Native render them "once in a while" (depending on the emoji) in Live Preview...

really awesome to hear that. I am going to check out the native emoji issue. could you maybe give me some text example where it sometimes fails?

@I-Pch
Copy link

I-Pch commented Apr 11, 2024

@FlorianWoelki 😊 : Regarding the emojis, this is a very minimal sample I used to test the rendering 😊

There's really nothing fancy 😄 (there's a mix of emojis and Lucide icons ) :
(I re-wrote the "emoji names" (:herb:)

:herb:
:sparkles: :LiRedo:

:LiSparkles:

> [!note]
> 
> :LiSparkles: Something 

- :purple_heart:
- :LiSparkles:


:herb: 

I did stumbled upon on something I think could be interesting: The emojis which can be rendered in Live Preview (:sparkles: works, for example) appear twice in Source Mode ... But the ones that don't get rendered in Live Preview appear only once in Source Mode

And here's a screenshot, in the 3 views, of the result I get with the small sample I shared above 😊 :
In it, you'll see that :purple_heart: and :herb: fails to render in Live Preview while :sparkles: appears without issue 😊 ...

Iconize - Emojis - 3 modes

For :purple_heart: and :herb:, they seems to only appear if my cursor is on the line and if I don't have the same note opened in split views 🤷‍♀️ ...
(That's new for me, sorry 😅 )

In the screen-recording below, instead of the emojis being rendered when I place my cursor on the line (in the note at the center), the cursor gets "longer" and there seems to be a really tiny space in place of the emoji...

Iconize.-.Emojis.-.Lines.mov

But if the note is in Live Preview and "alone", this is what I see 😊 ...

Iconize.-.Emojis.-.Line.-.Alone.mov

I do get various mixed results depending on the number of split views open and the mode the notes are in (Source, LP, Reading)

But at least, I can now have split views 🥳 !!!

@blooest
Copy link

blooest commented Apr 11, 2024

Seems like there's two issues in one in this thread.

Is the code in #438 (comment) meant to fix this?

#438

When Iconize is active, opening the note by clicking on the title from Explorer giving an error:

🔴 Failed to load "MyNote.md". Ranges must be added sorted by 'from' position and 'startSide'

Because I'm still having that issue when trying to load any notes with inlines. Oddly, one that opens on startup works perfectly, with icons, until I click off of it.

@FlorianWoelki
Copy link
Owner

Seems like there's two issues in one in this thread.

Is the code in #438 (comment) meant to fix this?

#438
When Iconize is active, opening the note by clicking on the title from Explorer giving an error:

🔴 Failed to load "MyNote.md". Ranges must be added sorted by 'from' position and 'startSide'

Because I'm still having that issue when trying to load any notes with inlines. Oddly, one that opens on startup works perfectly, with icons, until I click off of it.

could you give me something that I can reproduce?

@claremacrae
Copy link

In case a simple reproduction of this issue with the Tasks plugin is useful, please see obsidian-tasks-group/obsidian-tasks#2766 (comment)

@FlorianWoelki
Copy link
Owner

In case a simple reproduction of this issue with the Tasks plugin is useful, please see obsidian-tasks-group/obsidian-tasks#2766 (comment)

thanks. it's already fixed and I'll make a release tomorrow.

@FlorianWoelki FlorianWoelki moved this from In Progress to Done in Iconize Board Apr 12, 2024
@FlorianWoelki
Copy link
Owner

i've released the version with the fix. hopefully it is fixed now.

@m-primo
Copy link

m-primo commented Apr 13, 2024

The issue still persists after the update

image

image

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests