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

Link Labels Feature #102

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

zyansheep
Copy link

Closes #100

This PR is a little messy (changed a few things to make webpack building easier)

Also the code is not very DRY, I basically just copy-pasted the node filter scheme to make it work with links.

Also there is a small bug caused by implementation details with the global link / link label hide / show features (showing all the links will show all links instead of showing all links not filtered).

Lmk what you think!

@infologie
Copy link
Contributor

infologie commented Jan 8, 2024

@zyansheep Thanks for submitting a pull request. Some of my colleagues have requested this feature recently, so I'm on board. This one will probably involve @Myllaume more than I, so I will ask him to check the code. edit I don't have a specific timeframe to give you because we haven't yet planned the next development cycle, so I'll get back to you on that. (More info on our "Now" page: https://cosma.arthurperret.fr/now.html)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement] Display link type on links.
2 participants