damus
damus copied to clipboard
damus-io
Add Bookmarking (Local to device)
| Example |
|---|
 |
UPDATED No settings button anymore, moved to navigation bar in bookmarks view
| Example (Dark) | Empty State | Empty State (Dark) | Settings |
|---|---|---|---|
 |
 |
 |
 |
This just saves them (as JSON) to user defaults so only local to device.
just need a way to delete and then I can merge
Must be a bug but it should show "Un-bookmark" if you've already bookmarked it. I'll check again, would that work? If it worked?
Also we probably shouldn't have "clear bookmarks" in settings, we can just add it as a button on the top-right of the toolbar on the bookmarks view
Also we probably shouldn't have "clear bookmarks" in settings, we can just add it as a button on the top-right of the toolbar on the bookmarks view
Yeah, that would be better. I'll try that
Ok, I think this is all fixed up now, updated the GIF in the description with current behavior @jb55
Looks like one more thing to figure out after rebasing with master:
Cannot convert value of type 'Binding<[NostrEvent]>' to expected argument type 'EventHolder'
Looks like one more thing to figure out after rebasing with master:
Cannot convert value of type 'Binding<[NostrEvent]>' to expected argument type 'EventHolder'
Should be fixed now