Skip to content

Ui redesign: rounded corners - #110

Merged
technophile-04 merged 6 commits into
mainfrom
remove-rounded
Jun 9, 2026
Merged

Ui redesign: rounded corners#110
technophile-04 merged 6 commits into
mainfrom
remove-rounded

Conversation

@rin-st

@rin-st rin-st commented May 29, 2026

Copy link
Copy Markdown
Member

Most changes are just making borders square

Example of how component looks after update

Screenshot 2026-05-29 at 18 10 02

How to test with se-2 scaffold-eth/scaffold-eth-2#1282 (comment)

@rin-st
rin-st requested review from carletex and technophile-04 May 29, 2026 14:23
@vercel

vercel Bot commented May 29, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
scaffold-ui-docs Ready Ready Preview, Comment Jun 9, 2026 12:05pm
scaffold-ui-example Ready Ready Preview, Comment Jun 9, 2026 12:05pm

Request Review

@technophile-04

Copy link
Copy Markdown
Contributor

Did a pass on the square redesign with Claude code and direction looks great. Found a few rounded leftovers the sweep missed, plus one CSS-level radius that a class search wouldn't catch. Grouping them here so it's one fixup commit:

@sacffold-ui/componets:

CSS-level radius (class sweep can't see this one):

  • .sui-skeleton has border-radius: 0.2rem baked in, so every skeleton stays rounded even after the rounded-* classes were stripped (e.g. the avatar skeletons in Address.tsx). Needs to be set to 0 here:
    border-radius: 0.2rem;

Example app (lower stakes, but it contradicts the squared components):

Unrelated, but while you're in there — the loading spinner width fix uses var(--size-selector, 0.25rem), but --size-selector isn't defined anywhere in the repo, so it always falls back to 0.25rem. Looks copied from daisyUI. Probably cleaner as a plain 1.5rem (or calc(0.25rem * 6)):

width: calc(var(--size-selector, 0.25rem) * 6);

@rin-st

rin-st commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

Ooops, I fixed all the issues 5 days ago but for some reason forgot to sent the message or something went wrong when I sent it. Ready to re-review!

@technophile-04 technophile-04 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Almost everthing looks great! Thanks Rinat, great work!

@technophile-04 technophile-04 changed the title Ui redesign Ui redesign: rounded corners Jun 9, 2026
@technophile-04
technophile-04 merged commit 4945a7f into main Jun 9, 2026
1 of 3 checks passed
@technophile-04
technophile-04 deleted the remove-rounded branch June 9, 2026 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants