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

display Text instead of URL on sidebar #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nohzafk
Copy link

@nohzafk nohzafk commented May 18, 2024

Display text instead of raw URL on sidebar; I believe it is the desired behavior. like this:

image

@jivimberg
Copy link
Owner

Hi 👋 Thanks for contributing!
I love how clean it looks!

One issue is that currently the index uses the URL as key so if you have multiple URLs with different texts they won't show up on the sidebar. Instead we might want to index by a combination of URL + Text.

We should also show the full URL on hover when displaying the Links by name.

I'd like to let users choose their preferred presentation through the settings page so that users can go back to URLs if they so desire.

@nohzafk
Copy link
Author

nohzafk commented May 21, 2024

i didn't realize the first problem, but for my use case it is ok.

adding hover display and setting customization certainly has some work to do. i'm very new to Obsidian plugin, hopefully i can find some time to investigate.

@jivimberg
Copy link
Owner

Cool! We can collaborate on it!

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.

2 participants