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
bookmark endpoints #96
bookmark endpoints #96
Conversation
this feature allows the user to bookmark endpoints, displaying those endpoints on top of all other endpoints. the status of bookmarked is persisted on localstorage.
|
||
export function setLocalStorageBookmarks(names: string[]) { | ||
localStorage.setItem(BOOKMARK_LOCALSTORAGE_KEY, JSON.stringify(names)); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will have some issues with browsers limiting local storage, such as lack of space of Safari's icognito mode. Better check that localStorage is actually working and guard it with a try catch, falling back to memory: https://michalzalecki.com/why-using-localStorage-directly-is-a-bad-idea/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
solved
This branch now has conflicts. Can you take a look? |
for sure! |
…avorite-endpoints
this feature allows the user to bookmark endpoints, displaying those endpoints on top of all other endpoints. the status of bookmarked is persisted on localStorage.
Screenshots
no bookmarks
with bookmarks