mikedilger / gossip

Gossip is a nostr client

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Private chat / Conversation UI

dtonon opened this issue · comments

I propose this quick redesign, primary intended to highlight the new NIP-17 DM:

  • A label identifying the encryption mode, with an information pop-up describing its features/limits; I'm not sure about the "Basic encryption" definition, maybe we need to emphasize that it's weak?

  • Top "Last notes" and "Profile" buttons;

  • Update the icon for the message preferences, using the standard icon used everywhere;

image image

@bu5hm4nn I will create a different issue for the info popover component.

Each message in your DM feed could be either weak or strong. Maybe we should highlight each message.

Right, let's mark the weak messages.
We can apply the info popover here, too.

image

Update:

Screenshot 2024-07-02 at 18 45 59 Screenshot 2024-07-02 at 18 45 06 Screenshot 2024-07-02 at 18 47 35

After fiddling with the note rendering for quite a while I have the following to say
a) currently rendering and parsing the note are not separate, they are inter-twined all the way down to showing shattered content
b) this means that refactoring only a part of the code is virtually impossible / or could have huge hard-to test consequences since many code paths only show on certain note content.

So unfortunately that means I need to re-write note rendering in order to create the design proposed here for DM's. That is not a bad thing, but it means this would be delayed by at least another week (with testing).

I would suggest planning a re-write of note rendering, but combining it with creating automated tests (where we create a folder of json test data for different note scenarios), so that we can start assure good results.

For now I suggest using this simplified note display for DMs:
Screenshot 2024-07-05 at 23 29 28

I would suggest planning a re-write of note rendering

I was planning to redesign it, I will go on.

For now I suggest using this simplified note display for DMs:

Ok.
Is it possibile at least to hide the useless bullet button that point to the current note?

Regarding the tooltip for the encryption strength indicator. I would recommed we write a detailed explanation on how to switch to Giftwrap encryption on the help pages and link to that page in the tooltip. Unless someone writes a text first I will just write one up which can then be improved.

A few small things missing, but please review @dtonon. I also need us all to check that I didn't break main feed note rendering. #807

@bu5hm4nn it's ok, you can just add more space between the elements on bottom-right (indicator, date, seen on).

Some more notes:

image

  • If a DM is a reply the parent overlap the name; we can also simply hide the link for the time being;
  • If a DM uses NIP-44 we should treat it as the default situation, so I would not show anything, or at least not a green dot, it is too prominent; if we want to highlight the new encryption, lets use a dot with a very dark gray;
  • Remove "DIRECT MESSAGE TO: dtonon", it is not necessary;

To do:

  • Add the encryption badge that partially overlap the compose area (let me know if it is too complex, we can find an alternative solution)
  • Add the top right button "Last notes" and "Profile"

About the action menu, I would organizer it this way:


Copy text
- To clipboard
- Via QR Code

Manage
- Delete
- Annotate
- Rebrodcast (X)

Bookmark
- Public
- Private

Copy ID
- Nevent
- Note
- Hex

Debug
- Show Json
- Copy Json
- Rerender
- Dismiss

The annotate doesn't seem to work for me.
Would be nice show the QR Code and the Json in a modal; it is prettier and also easier to revert (close button).