obsidian-sortable icon indicating copy to clipboard operation
obsidian-sortable copied to clipboard

Check integration with obsidian-dataview

Open alexandru-dinu opened this issue 3 years ago • 4 comments

  • https://github.com/blacksmithgu/obsidian-dataview

alexandru-dinu avatar Apr 16 '21 08:04 alexandru-dinu

Since the view auto update feature was added to dataview, sortable no longer keeps the table sorted. It gets refreshed along with the view.

arminta7 avatar Apr 23 '21 08:04 arminta7

See discussion thread.

alexandru-dinu avatar Apr 23 '21 14:04 alexandru-dinu

Since obsidian-sortable sorting is generally working for tables generated with obsidian-dataview, and it seems (from the discussion on the other repo) that there is not going to be any merger or dependency, should this issue be closed?

I say "generally" because I have noticed some tiny issues:

  • there is no indicator visible to show the table's non-dynamic sort when the page is opened
  • cycling through column sorting states takes you through ascending, descending, and whatever datatables initial sort state was

I understand that these tiny issues are basically inevitable unless obsidian-sortable specifically reads the dataview query to determine the starting sort state. If you're willing to do that extra work—that's awesome. If not, then I think this can be closed.

Either way, thanks for providing this! 🍻

brandondrew avatar Jun 25 '23 17:06 brandondrew

there is no indicator visible to show the table's non-dynamic sort when the page is opened

Can you please provide a specific example of this?

cycling through column sorting states takes you through ascending, descending, and whatever datatables initial sort state was

Maybe I am misunderstanding, but this is intentional behaviour -- it's how tables on Wikipedia are sorted: https://en.wikipedia.org/wiki/Help:Sortable_tables#Example


The big disclaimer is that I am not a Dataview user myself, so I am not actually familiar with its specifics.

alexandru-dinu avatar Jun 26 '23 20:06 alexandru-dinu