Skip to content

fix(docs): center loading button spinner in demos - #13

Merged
Nostromo-618 merged 1 commit into
mainfrom
fix/loading-spinner-centering
Aug 9, 2026
Merged

fix(docs): center loading button spinner in demos#13
Nostromo-618 merged 1 commit into
mainfrom
fix/loading-spinner-centering

Conversation

@Nostromo-618

Copy link
Copy Markdown
Member

Summary

  • Adds a temporary docs.css override so .vd-btn loading spinners are vertically centered in demos (translate: -50% calc(-50% - 1px), margin: 0, line-height: 0).
  • Compensates a known @vanduo-oss/vd3@1.2.2 package CSS issue (line-height inheritance with top/left + negative margin). Missed from the docs: vd3 1.2.2 dogfood — CBUN, Showcase, loading buttons #12 squash merge race.
  • Proper fix still belongs upstream in @vanduo-oss/vd3; drop this override when that ships.

Test plan

  • Open Buttons docs page (loading state demos)
  • Confirm spinner rings look vertically centered on filled, small, and outline loading buttons
  • Confirm rotate animation still runs

Made with Cursor

Compensate vd3 1.2.2 spinner offset (line-height + negative margin) via docs.css until the package ships a proper fix.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Nostromo-618
Nostromo-618 merged commit f2cfc12 into main Aug 9, 2026
1 check passed
@Nostromo-618
Nostromo-618 deleted the fix/loading-spinner-centering branch August 9, 2026 09:48
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.

1 participant