Initial transpose table (#22) #30
Conversation
|
Note: happy to help with the obsidian plugin development for this as well. I assume we'll need an icon, which would really be the hardest part imo. I tested this through your advanced-tables obsidian plugin, so let me know if you'd like some help with that too. |
There was a problem hiding this comment.
@tgrosinger What additional test cases you'd want to see handled here (e.g. particular chars in header divider, no spacing in cells, etc.)? I wasn't exhaustive for this case to avoid bloat, but happy to add more in. The code is fully working afaik.
|
@tgrosinger as I see that you are still active in this plugin, would you consider to take a look at this PR? |
|
Merged! Thank you. |
|
@ooker777 I do still actively use, but not with my changes side-loaded, as my use case for it has gone done outside of previously company I was working at. Glad to know it sounds like it still works :smile . Hope you guys are able to get a UI option into the actual tables plugin some point soon too! |
|
P.S. just saw the change, and congrats on the new role @tgrosinger - well deserved |
@tgrosinger it's definitely not the prettiest code (due to the need to ignore the delimiter row on both read and write being quite confusing, but I've been a bit strapped for time recently and haven't been able to dive into this code gain. It works well, so let me know what you think.
The main thing that would really help readability for this is if we could (before any computation) just remove the delimiter row flat out. If you have a decent idea of an approach to do that, I'd love to know how. Alternatively, if you wanted to make any changes in my PR, I've granted you full permissions :)