search

mdelobelle / metadatamenu

For data management enthusiasts : type and manage the metadata of your notes.
https://mdelobelle.github.io/metadatamenu
MIT License
478 stars 27 forks source link

Mobile app crashes when you try to select a link from metadata menu table #576

Closed cameronj86 closed 4 months ago

cameronj86 commented 4 months ago

https://streamable.com/u3yund

To replicate, create a table and then try to access one of the links through the mobile app by clicking it

RaviVanam commented 4 months ago

I'm also facing the same issue on my Xiaomi Redmi Note 10 pro

mdelobelle commented 4 months ago

Hi I don't have an Android to try to reproduce but U have notice another behavior since the last mobile version update on IOS. I've made a modification that fixes this behavior on IOS, maybe this will also fix this issue on Android. I will push it in the next beta release

mdelobelle commented 4 months ago

I've release the version 0.8.0 can you test on your android mobile if this fixes your issue?

RaviVanam commented 4 months ago

Thanks for the release! it's working now! 😀

RaviVanam commented 4 months ago

Sorting doesn't seem to work properly after version 0.8.0. Can you look into it. I was sorting based on date.

cameronj86 commented 4 months ago

Just gave it a shot myself and it works for me!

Sort functionality works for me. I did think it was broken for a beat, but it turns out I was sorting by a different field first and it was throwing me off. The beta appears to fix the numerical sorting as well, so it's worth noting that sorting by any numerical columns may not have worked before but are currently working now.

There is a slight visual bug that might be new where the unique value is off kilter when you scroll: Screenshot_2024-02-17-10-08-32-586_md.obsidian.jpg

My uneducated guess is that the unique values count might be throwing off the functionality for the column. Personally this is a much lower priority for me because I can still complete my workflow but let me know if you want me to load a separate ticket

RaviVanam commented 4 months ago

hi @cameronj86 are u sorting using filename? can u try sorting a field that is set as date in metaclassfields.

cameronj86 commented 4 months ago

hi @cameronj86 are u sorting using filename? can u try sorting a field that is set as date in metaclassfields.

Hey.

I don't have any fields that are set as date. If I recall correctly, there's a potential conflict between obsidian's date and metadata menu plugin's date, especially if time is used, so all of my date columns are technically just text fields

RaviVanam commented 4 months ago

I was sorting obsidian's date (time is used) before. It worked fine. But started to break after the new release. Tried it again on another vault that was running on older version. it was working fine, updated to 0.8.0 and it started to break there too.

RaviVanam commented 4 months ago

I opened an issue for the sorting issue. This thread can be closed.