Radish alpha
r
rad:z4D5UCArafTzTQpDZNQRuqswh3ury
Radicle desktop app
Radicle
Git
Add revision commit list
Merged rudolfs opened 1 year ago
rudolfs opened with revision fd1e0225 on base 557e96f9 +479 -23 1 year ago
rudolfs pushed revision 2 6d7bea3f on base 924130ce +362 -1 1 year ago

Rebase.

rudolfs pushed revision 3 6a24f733 on base 924130ce +359 -0 1 year ago

Fix rebase conflict snafu.

rudolfs pushed revision 4 a80ecbd4 on base 924130ce +376 -152 1 year ago

Fix revision commit listing.

rudolfs pushed revision 5 ac7f37e0 on base f34522b6 +376 -152 1 year ago

Rebase.

rudolfs pushed revision 6 6085fe22 on base f34522b6 +382 -152 1 year ago

Styling.

rudolfs pushed revision 7 2255ab98 on base f34522b6 +381 -152 1 year ago

Move commits into the Changes section

rudolfs pushed revision 8 163386bc on base f34522b6 +383 -154 1 year ago

Use same node version on CI as locally

rudolfs pushed revision 9 bcdbb7f5 on base f34522b6 +452 -154 1 year ago

Make commits stand out from the rest of the files.

rudolfs pushed revision 10 3f4ceb2b on base f34522b6 +450 -154 1 year ago

Remove loading text, commits load very fast and this avoids flickering.

rudolfs pushed revision 11 29a82483 on base f34522b6 +450 -154 1 year ago

Fix CI.

rudolfs pushed revision 12 f7a3cd5a on base f34522b6 +450 -154 1 year ago

Fix rebase snafu.

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

ScreenCapture 2025-02-04 um 08.48.46.png

I feel like there is too much border happening around the Commits dropdown and the chevron border, can we remove one of them?

Can we move this and this inline style into it’s own style declaration?

I think we should avoid writing new Svelte slots and try to use only snippets I would rewrite the HoverPopover component in general

The rest LGTM AFAICT

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

ScreenCapture 2025-02-04 um 08.48.46.png

I feel like there is too much border happening around the Commits dropdown and the chevron border, can we remove one of them?

Can we move this and this inline style into it’s own style declaration?

I think we should avoid writing new Svelte slots and try to use only snippets I would rewrite the HoverPopover component in general

The rest LGTM AFAICT

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

I think its ok if the border shows on hover, its clear you’re hovering. Its not the prettiest, that is true, but it’s how all the buttons work now. I’ll think of something.

But the caret before Changes doesn’t have a hover state. So we either have to make that an icon button too, or its ok if the commits caret also doesn’t have a hover state.

rudolfs pushed revision 13 01644553 on base f34522b6 +480 -154 1 year ago

Make everything idiomatic Svelte 5.

rudolfs pushed revision 14 9b22a642 on base f34522b6 +503 -182 1 year ago

Add hover state to section collapse buttons

rudolfs merged revision 9b22a642 at 8bfb0eb4 1 year ago