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

Add context around links #90

Merged
merged 1 commit into from
May 13, 2023
Merged

Conversation

ivan-lednev
Copy link
Contributor

@ivan-lednev ivan-lednev commented Apr 4, 2023

First of all, thank you for a great plugin!

This change is inspired by how tools like Roamresearch and Logseq display context around links.

  • If the item has headings above it, show the ancestor headings chained together
  • If the link is in a bullet list, show its ancestors chained together as well as descendants
  • If the link is in a heading, show the section right below it
  • If the link is in a section, just show the complete section

Here is a demo:
image

@TfTHacker
Copy link
Owner

This looks great and thanks you for the submission. Give me a few days to a week to get to this. It's on my todo list :-)

@jancervinka
Copy link

I'm really excited about this as it would significantly improve my workflow. So I'm curious, do you have any updated estimate on when you might be able to review and possibly merge this @TfTHacker?

@TfTHacker TfTHacker self-assigned this May 13, 2023
@TfTHacker TfTHacker added the enhancement New feature or request label May 13, 2023
Copy link
Owner

@TfTHacker TfTHacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I want to apologize for the long delay in implementing your PR. Really, it is a revolutionary step forward. thank you!!! I need to buy you some coffee :-)

@TfTHacker TfTHacker merged commit 641485e into TfTHacker:master May 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants