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

note links (bug/enhancement) #169

Closed
olheadinternet opened this issue Dec 28, 2022 · 3 comments
Closed

note links (bug/enhancement) #169

olheadinternet opened this issue Dec 28, 2022 · 3 comments
Labels
bug Something is not working, or not working as intended

Comments

@olheadinternet
Copy link
Contributor

olheadinternet commented Dec 28, 2022

There are some bugs mentioned below, but my general proposal is:
notes with more than 1 linked note should not have a preview, rather tapping on the note should lead to a thread with just the linked notes, and a button to "expand" from there.


3rd note missing in damus

Test 1: note1ad643jqnm4wygfn0afdtgea9a56dqc78e7pjgllc2x5eauulunysjuuzjl

and replicated with text between notes: (see test 1.2 in next section)

however when test 1 is rendered as a quote note… it does show all 3 notes as seen in test 2, but that goes away if you enter thread view

test 2: note1yw7tdjrmvt9gnvvjsqtr59tx46stantzwfytd8gmz45aygphzj4qr99n0u

Jra7gsw

KAx6QzM


I also tried to surround the links with text...

test 1.2: note187tpva2pdey6kgj4mcd52eu3z9rrl9vu5kx98x036qcd9e3x5vrq7ph9mk

not only is the 3rd link missing but the text above it "test c:" is also missing.

kuO7DSJ


in my mind what should happen when I click the following note is i am first presented with the notes that are linked, they are older because they have to exist first, and should render above this note:

note10ewyvqpwscqe3c6ctarjeyj8enxrtv7wmpzl58dg3rf2q3qgyuuqm8apav

that note was sent prior to 0.1.8(5) with the thread update… that also has 3 note links. In this instance none of the note links appear, text of note isn’t shown in full, the first link does render as a quote.

I’m seeing boosts and mentions of this note below, which I’m ok with, but sequentially I want to see the notes mentioned in the note first, then expand the conversation if I want to see conversation around that note.


vishalxl replied to one of my tests with these screen shots of what they look like in console & astral
vishalxl/Nostr-Clients-Features-List#3 (comment)

There isn't the space to do that in damus, notes with more than 1 linked note should not have a preview, rather tapping on the note should lead to a thread with just the linked notes, and a button to "expand" from there.


@olheadinternet
Copy link
Contributor Author

olheadinternet commented Dec 28, 2022

here is a test with an invoice followed by a note link, note link disappears

note18ywu835z6ny6hhhsytpungxt7m43zp0ae24j64s3zjxu44cyy34qmxzjnr

IMG_0380

Update with wider invoice design... still can't see the note link

note13eue56zmkd8wnxctflhhjn8d260hxzws90c8zrmztq22tu3g5rks6gcu8n

IMG_0388


here is a test with a note link followed by a invoice, works fine.
the text doesn't render fully in portrait... the text does fully render in landscape.

note1630vnckghc8mad4gjdm9h3shmt7sqk2zuhtlks5msapznvxsn9gs5hykzm

IMG_0381

@olheadinternet

This comment was marked as outdated.

@bebopkid bebopkid added the bug Something is not working, or not working as intended label Jan 3, 2023
@olheadinternet
Copy link
Contributor Author

as of 1.0.0 (3) note quote preview has been removed, I assume temporarily... but my proposal still stands when their gui is updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working, or not working as intended
Projects
None yet
Development

No branches or pull requests

3 participants