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

Sort by last accessed #739

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

Conversation

vslinko
Copy link
Contributor

@vslinko vslinko commented May 19, 2024

@sissbruecker Hi! What do you think of such a feature in the mainstream?

@vslinko vslinko marked this pull request as draft May 19, 2024 20:59
@vslinko
Copy link
Contributor Author

vslinko commented May 29, 2024

@sissbruecker waiting for your feedback

@garymoon
Copy link

@vslinko They might not be seeing this because it's a draft, I'd suggest marking it ready.

@sissbruecker
Copy link
Owner

Thanks for the contribution, but I'm not sure if I want to proceed with this as I don't see a lot of utility. I'd suggest to create an issue first and get some feedback if others are interested in this and what their use cases would be. I'll still keep this open for now.

Regarding the implementation, I'm not sure if this approach works reliably when opening bookmarks in the same page, though that would require some testing. The alternative might be to intercept link clicks, make the API request first, and then navigate programmatically.

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.

3 participants