Radish alpha
r
rad:z4D5UCArafTzTQpDZNQRuqswh3ury
Radicle desktop app
Radicle
Git
Improve Sidebars to make them more equal
Draft did:key:z6MkkfM3...sVz5 opened 1 year ago
did:key:z6MkkfM3...sVz5 opened with revision 6c039998 on base 53d49e0b +21 -39 1 year ago
rudolfs commented on revision 1 1 year ago

Hm, I don’t like how it looks now, multiple words kinda look ugly in a list. How about this?

Repositories
Delegated
Contributed
Private
rudolfs commented on revision 1 1 year ago

And if you’re going with “All Issues” and “All Patches”, then you should also adjust the filters in the second column in the single issue/patch views. Although, not sure it makes full sense there either as we would still just want to say “Open” instead of “Open issues”, so idk.

Screenshot 2025-02-10 at 09.45.04.png

rudolfs commented on revision 1 1 year ago

Also noticed that we’re now opening patches on “All” by default. I think it makes more sense to default to just “Open” patches, because most of the time people don’t care about merged or draft patches.

did:key:z6MkkfM3...sVz5 commented on revision 1 1 year ago

We could also change the sidebar to have All, Open and Closed labels when clicking on Issues from e.g. patches but I think it’s important to still show which type of cob we are listing.

The other option would be to go with a non-clickable Issues section header and then have All, Open, Closed this would be more similar to the filters.

EDIT: Kept on thinking, and there is also the option to keep the Issues and Patches sidebar the same, with the additional filter for All and make the Repositories sidebar equal to patches and issues

did:key:z6MkfgZK...5YMm commented on revision 1 1 year ago

Delegated sounds like I delegated it to someone, doesn’t it? NVM I see its like that now.

Repositories
Delegate of
Contributed to
Private to me
did:key:z6MkfgZK...5YMm commented on revision 1 1 year ago

I think its great as it is. If you landed on “Open” it would be even clearer how it works. For the filter on the single patch view, I’d just add “All Patches” instead of “All” and leave the others as they are.

did:key:z6MkfgZK...5YMm commented on revision 1 1 year ago

I might be missing something but they seem pretty similar to me, except

  • the filter on the single issue/patch view shoudl say All patches at the top
  • the order or items listed

And if we desaturate them in the main sidebar we should probably also desaturate them here.

Screenshot 2025-02-11 at 11.27.15.png Screenshot 2025-02-11 at 11.27.33.png

did:key:z6MkfgZK...5YMm commented on revision 1 1 year ago

Actually, we could also simplify this filter save some space if we make the breadcrumb the handle like so:

Screenshot 2025-02-11 at 11.36.58.png