Skip to content

feat: add additional resets to normalize.css#838

Merged
rohanchkrabrty merged 1 commit into
mainfrom
fix-normalize
Jun 10, 2026
Merged

feat: add additional resets to normalize.css#838
rohanchkrabrty merged 1 commit into
mainfrom
fix-normalize

Conversation

@rohanchkrabrty

@rohanchkrabrty rohanchkrabrty commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add additional resets in normalize.css such as: visible hr, list-style removal, heading/link/abbr resets, border-collapse on tables, font-property inheritance on form controls, Firefox focus/invalid fixes, textarea resize, placeholder styling, button cursors, block-level replaced elements, and [hidden]
  • Move the button appearance/background reset from styles/reset.css into normalize.css
  • Rename styles/reset.css to styles/global.css (now only holds global scrollbar styling) and update the import in styles/index.css

@vercel

vercel Bot commented Jun 10, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
apsara Ready Ready Preview, Comment Jun 10, 2026 10:42am

@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This PR refines baseline CSS normalization across the Raystack style system. The normalize.css file receives substantial updates to element normalization rules: base typography formatting is reformatted, table normalization gains explicit border and text-indent properties, form controls are enhanced with inherited font feature and variation settings plus comprehensive typography inheritance, a new textarea resize restriction is added, and replaced elements (img, svg, video, etc.) receive consolidated display and sizing normalization. Corresponding stylesheet changes remove button reset styles from global.css (now delegated to normalize.css) and reorganize imports in index.css to include global.css, remove reset.css, and standardize import path quoting.

Suggested reviewers

  • rsbh
  • paanSinghCoder
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title 'feat: add additional resets to normalize.css' accurately captures the main objective of adding resets to normalize.css, aligning with the PR's primary goal of incorporating missing preflight resets.
Description check ✅ Passed The PR description accurately describes the changeset, covering the addition of resets to normalize.css, the movement of button styles, and the rename of reset.css to global.css.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rohanchkrabrty rohanchkrabrty changed the title feat: add missing preflight resets to normalize.css feat: add additional resets to normalize.css Jun 10, 2026
@rohanchkrabrty rohanchkrabrty merged commit 5d4ee32 into main Jun 10, 2026
5 checks passed
@rohanchkrabrty rohanchkrabrty deleted the fix-normalize branch June 10, 2026 10: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.

2 participants