Repository navigation
chore(docs): record what the CSS audit actually found - #765
Merged
Merged
Conversation
Section 5 said the stylesheet was large enough to audit for unused selectors. The audit shows the bulk is generated Tailwind, not hand-written CSS, so the bullet now carries the measurement and the one deletion that followed from it instead of an open suggestion.
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing: 1
✨ Finishing Touches
Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The section 5 CSS finding said the stylesheet is large and suggested auditing unused selectors. It has now been audited, and the answer is that the direction was a dead end: of the 114,177 B the build emits, Tailwind-generated utilities are 94,882 B (83.1%), preflight 4,035 B and @Property 4,057 B, while all hand-written app CSS together is about 6.9 KB. Removing the nine language schemes that no selectable locale can reach (#764) took 3,318 B off, and no hand-written selector is left to cut.
The review's own 115.9 KB figure was measured at a7070d7 and had already stopped holding on main, which the bullet now says instead of repeating it. Further compression needs @source scoping or deleting the components/ui directories nothing imports, which is a separate change.