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

Make.md breaks plugin #5

Open
ashworksco opened this issue Jan 1, 2024 · 4 comments
Open

Make.md breaks plugin #5

ashworksco opened this issue Jan 1, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@ashworksco
Copy link

What happened?

Files do not get pinned or hidden, but the plugin thinks they have.

What did you expect to happen?

I expect files to either be pinned or hidden based on right clicking and selecting from the pop-up menu.

Can you reproduce the bug, if so how? As detailed but minimal as possible.

Yes, and I have done so several times now with installing/uninstalling.

Other?

No response

Plugin version

1.1.3

Obsidian version

1.5.3

OS

macOS

@ashworksco ashworksco added the bug Something isn't working label Jan 1, 2024
@kelszo
Copy link
Owner

kelszo commented Jan 1, 2024

Can you provide some more details? Currently working as expected on both my devices, cannot replicate

@kelszo
Copy link
Owner

kelszo commented Jan 1, 2024

According to the latest Changelog for Obsidian:

File Explorer and Tags pane both received a small code refactor. This might affect some plugins that rely on the internal JS structure. Themes should not be affected.

However, I am currently also running Obsidian v1.5.3 and it works. Do you have any other plugins that affect the File Explorer? Has the plugin worked before?

@zenus260
Copy link

zenus260 commented Jan 4, 2024

Can you provide some more details? Currently working as expected on both my devices, cannot replicate

I have the same issue and I'm on v1.5.3 too. I tested it on a new vault and it worked fine so it most likely is caused by a conflicting plugin. The only plugin I have that affects the File Explorer is make.md.

What ashworksco is probably referring to by the plugin thinking that files are hidden or pinned is that if you pin a note using the right click menu and then open that menu again, the option is now to unpin the note, however the note has not been pinned.

@kelszo
Copy link
Owner

kelszo commented Jan 4, 2024

Thank you for the vital information @zenus260!

I tested installing make.md and I found that File Explorer++ stopped working on reload. If @ashworksco can confirm they are also running make.md I think we can confirm the cause.

Make.md seems to be a really cool plugin, wouldn't mind testing myself. However, looking through their code they have some aggressive code practice. Seems like the cause is that they override the base Obsidian function getLeavesOfType so when trying to fetch the File Explorer leaf it returns their leaf instead. See: patches.ts in the make.md code.

TLDR: To be honest, nothing we can do on our side. They make it impossible to fetch the file-explorer leaf and thus break our integration. The make.md team should consider a less invasive approach to their patching method for interoperability with other plugins.

UPDATE: Actually found a fix, make.md has a Compatibility Mode in it's settings. Activating that seems to make File Explorer++ work as expected.

@kelszo kelszo changed the title Plugin not working as expected Make.md breaks plugin Jan 4, 2024
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
None yet
Development

No branches or pull requests

3 participants