nips icon indicating copy to clipboard operation
nips copied to clipboard

Quoted highlights

Open jb55 opened this issue 1 year ago • 7 comments

This is in preparation for Damus' highlight implementation. We have a new quote highlight variation. This is to prevent the creation and multiple notes (highlight + kind 1) for a single highlight action that wishes to add a comment along side a highlight, which looks bad in micro-blogging clients where these notes may appear in succession.

This is backwards compatible with existing highlights, except for the situation where clients might not properly handle the mention attribute.

Since @pablof7z has already mentioned he does not like this approach, we will create a DIP if/when this is not accepted.

jb55 avatar Aug 22 '24 06:08 jb55

The tricky part is determining whether Highlights should be on a kind1 feed or not. Amethyst has always shown them, but it hasn't been super well received. Highlights are seen as personal notes + notes for others following the original page/post. It's not about letting my followers know all the highlights of the book I am reading as I mark them. Users might be highlighting lots of things for their future selves and those just pollute their feed.

Derek posted 20+ highlights from an article he was reading last week. All highlights showed up to his followers. It was all fine, but it felt very weird. It looks like the user forgot that each highlight is a note and they are just spamming their followers with notes.

Otherwise, this looks ok to me.

vitorpamplona avatar Aug 22 '24 12:08 vitorpamplona

This is backwards compatible with existing highlights, except for the situation where clients might not properly handle the mention attribute.

This seems like the main problem to me. I agree with Vitor's UX critique as well. Maybe a new kind would be better? I do agree that having a comment go with a highlight would be an improvement.

staab avatar Aug 22 '24 16:08 staab

I do agree that having a comment go with a highlight would be an improvement.

Agree as well. I just don't know if this is a comment for future me, for public view in this site or for my followers.

Maybe a new kind would be better?

To me, it's either this or a regular kind1 quote with the highlight's nevent inside of it. I am not sure if there is a need for a new kind.

vitorpamplona avatar Aug 22 '24 16:08 vitorpamplona

To me, it's either this or a regular kind1 quote with the highlight's nevent inside of it. I am not sure if there is a need for a new kind.

The problem with this is that you get duplicate notes, which is the original problem Will is pointing out. A new kind that is "highlight with comment for public viewing" vs the current "highlight without comment for private use" has pretty different semantics.

staab avatar Aug 22 '24 16:08 staab

The problem with this is that you get duplicate notes, which is the original problem Will is pointing out.

Yeah, but if we all agree that the Highlight is supposed to be quoted by a kind1 to display on Kind1 feeds then we would never show the highlight event itself (or only show it as a "reply")

vitorpamplona avatar Aug 22 '24 16:08 vitorpamplona

I'm not vehemently against it, I mainly don't see the issue this addresses as a big problem, as clients already need to be aware of the other stuff they're rendering in a feed (i.e. reposts/quotes).

The drawback I see is that if you repost, are you reposting the comment? the highlight? I mean -- you're reposting both, but you can't repost just the highlight.

pablof7z avatar Aug 23 '24 16:08 pablof7z

Coracle now supports rendering comment (in next release)

staab avatar Sep 16 '24 19:09 staab

published DIP04 to document damus' implementation of this:

https://github.com/damus-io/dips/blob/master/04.md

jb55 avatar Nov 11 '24 20:11 jb55

I'm curious to know solution @pablof7z had in mind with clients already being aware of stuff to render? If a highlight event and a quote of that highlight event come in within 1 second its a guess that they should be linked together for rendering? DIP04 seems like a simple solution, I'll probably implement it also.

fabianfabian avatar Apr 11 '25 08:04 fabianfabian

The feedback I got was that you were supposed to filter the highlight if there was a quote repost of the highlight. I prefer something more self contained vs a spec that requires you to filter your feed.

jb55 avatar Apr 11 '25 21:04 jb55

https://git.fiatjaf.com/lantern implements this. It fits well with the code inherited from hypothes.is

Originally I went out of my way to create a cumbersome solution to do the kind:1 combo, but after chasing way too many bugs I've decided to just implement this proposal.

It's somewhat sad that the comment is in a tag though, it would be better as the main content while the highlight quote would go in a tag, but it's too late now.

fiatjaf avatar Apr 12 '25 19:04 fiatjaf