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

MSC2771: Bookmarks #2771

Closed
wants to merge 4 commits into from
Closed

MSC2771: Bookmarks #2771

wants to merge 4 commits into from

Conversation

MTRNord
Copy link
Contributor

@MTRNord MTRNord commented Sep 10, 2020

Rendered

Signed-Off-By: MTRNord info@nordgedanken.de

@MTRNord MTRNord changed the title MSC2765: Bookmarks MSC2766: Bookmarks Sep 10, 2020
@MTRNord MTRNord changed the title MSC2766: Bookmarks MSC2771: Bookmarks Sep 10, 2020
proposals/2771-bookmarks.md Outdated Show resolved Hide resolved
@turt2live turt2live added kind:feature MSC for not-core and not-maintenance stuff proposal A matrix spec change proposal proposal-in-review labels Sep 10, 2020
proposals/2771-bookmarks.md Outdated Show resolved Hide resolved
proposals/2771-bookmarks.md Outdated Show resolved Hide resolved
@MTRNord
Copy link
Contributor Author

MTRNord commented Nov 8, 2020

I am working on a tui based reference impl "utility" (Not a full MX Client but a todo application focused on just this feature) which implements both approaches but defaults to the main versions when sending.

The unstable prefixes are dev.nordgedanken.bookmarks.room for the main proposal and dev.nordgedanken.bookmarks.global for the alternative.

(I might also build a small flutter app for this allowing synced todos across devices)

https://github.com/MTRNord/mx-bookmarks

Edit: Dev stalled for personal reasons

@turt2live turt2live added the needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. label Jun 8, 2021
important messages without needing to know a search term. This proposal aims to provide browser or TODO list alike
bookmarks.

## What sets this apart from other Proposals that are similar (like Pinning and MSC2437)?
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

link to #2437 please

Copy link
Member

@KitsuneRal KitsuneRal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be honest, I still don't see how this is materially different from #2437, aside from suggesting the ability to store metadata next to the saved event. Perhaps it would be great to work together on advancing #2437 instead of having two mechanisms doing very similar and even overlapping things (out of these two, I see #2437 as more elaborated).

On the other side of similar MSCs is the MSC2437 which allows hashtag alike tagging of events. This allows to search
events by topics or displays them based on their topic. You might think that bookmarks would work with this (and they
might) but this Proposal wants to also cover the specifics of bookmarks and is more aimed to be alike browser bookmarks
and allows not only marking as bookmarks but also aims to give it some bookmark specific metadata.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Browser bookmarks in Firefox have a tags field as well, FWIW (and Chrome bookmarks don't seem to support any metadata at all, or I'm lost in its UI).

{
"event_id": "$12345:example.com",
"title": "Food Choice",
"comment": "I need to buy stuff for this"
Copy link
Member

@KitsuneRal KitsuneRal Jul 11, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that an exhaustive example in terms of metadata, or an indicative one? In other words, can there be something on top of title and comment? If it can, I wonder if the stock set of attributes should come in m. namespace.

@MTRNord
Copy link
Contributor Author

MTRNord commented Aug 2, 2021

Closing due to not having the time to maintain

@MTRNord MTRNord closed this Aug 2, 2021
@turt2live turt2live added abandoned A proposal where the author/shepherd is not responsive and removed proposal-in-review labels Aug 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
abandoned A proposal where the author/shepherd is not responsive kind:feature MSC for not-core and not-maintenance stuff needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. proposal A matrix spec change proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants