Skip to content

fix: resolve reference category spacing and CSS layer priority - #1613

Merged
doradocodes merged 1 commit into
processing:mainfrom
MishtiGarg250:main
Oct 9, 2026
Merged

doradocodes merged 1 commit into
processing:mainfrom
MishtiGarg250:main

Conversation

@MishtiGarg250

Copy link
Copy Markdown
Contributor

Hi @Nwakaego-Ego ,thanks for that detailed feedback and for those edge cases, I've gone ahead and pushed updates to address everything.
issue: #1538

  • Limit @layer components in global.scss strictly to the h2 bottom margin to prevent Tailwind base styles from overriding other section rules.
  • Refactor reference category spacing in ReferenceDirectoryWithFilter:
    • Apply mb-0 unconditionally to category h2 tags.
    • Shift 40px top margin to unnamed subcategory containers to prevent double margins (e.g., Shape) or missing margins (e.g., Typography).
    • Prevent rendering of empty unnamed subcategories.
  • Restore package-lock.json to undo accidental deletion.

- Limit @layer components in global.scss strictly to the h2 bottom margin
  to prevent Tailwind base styles from overriding other section rules.
- Refactor reference category spacing in ReferenceDirectoryWithFilter:
  - Apply mb-0 unconditionally to category h2 tags.
  - Shift 40px top margin to unnamed subcategory containers to prevent
    double margins (e.g., Shape) or missing margins (e.g., Typography).
  - Prevent rendering of empty unnamed subcategories.
- Restore package-lock.json to undo accidental deletion.
@Nwakaego-Ego

Copy link
Copy Markdown
Contributor

Thank you, @MishtiGarg250, for the fix. I tested the changes locally, and the spacing is now consistent across Typography, Transform, Constants, and Shape without introducing a double gap. The fix also works correctly on mobile.

cc @doradocodes

@doradocodes
doradocodes merged commit 958bc40 into processing:main Oct 9, 2026
4 checks passed
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.

3 participants