Skip to content

Conversation

@AyushCodes160
Copy link

@AyushCodes160 AyushCodes160 commented Nov 19, 2025

Updated the documentation site's CSS (docs/css/style.css) to improve visual design and modernize styling. Added a gradient background, updated the homepage container with rounded corners and shadows, and refreshed code block styling with a dark theme and improved spacing. Replaced -webkit- prefixes with standard CSS transitions, changed list bullets to custom arrow bullets, and refined spacing, padding, and visual hierarchy. The changes include 318 additions and 50 deletions, improving readability and visual consistency across the documentation site.

@hasezoey hasezoey changed the title Update style.css Update homepage style Dec 1, 2025
Copy link
Collaborator

@hasezoey hasezoey left a comment

Choose a reason for hiding this comment

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

Overall i would say this looks OK, though due to the new li::before, there are some unwanted things.

For anyone that does not want to run it locally:

Screenshot of current state

Without advertisements, as those are not build on local deployments:

Image
For comparison, the current homepage style Image

Comment on lines 147 to 148
-webkit-background-clip: text;
-webkit-text-fill-color: transparent;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does there not exist non-webkit specific properties for this?

Comment on lines +61 to +67
li:before {
content: "▸";
position: absolute;
left: 0;
color: #8B0000;
font-weight: bold;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Due to this now being a thing, it also needs to be disabled for #links > ul > li::before and #follow > ul > li::before (see screenshot)

@hasezoey hasezoey added the docs This issue is due to a mistake or omission in the mongoosejs.com documentation label Dec 1, 2025
- Replace webkit-specific text-fill-color with standard color property
- Remove bullet points from #links and #follow sections
@AyushCodes160
Copy link
Author

Changes Made:

  • Replaced webkit-specific -webkit-text-fill-color with standard color property for better cross-browser compatibility
  • Added background-clip: text as the standard property while maintaining -webkit-background-clip for Safari
  • Removed bullet points from #links and #follow sections to improve navigation appearance

Testing:

  • Verified the changes in multiple browsers (Chrome, Firefox, Safari)
  • Ensured the gradient text effect works as expected
  • Confirmed navigation links display correctly without bullet points

Related Issues:
Fixes: #15775

Additional Notes:

  • This change maintains the existing visual design while improving code quality and browser compatibility
  • The CSS is now more maintainable and follows better practices

@hasezoey Could you please review this when you get a chance?

Copy link
Collaborator

@hasezoey hasezoey left a comment

Choose a reason for hiding this comment

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

Looks better now. Though i only became aware after first reviewing this PR, there already exists a (slight) redesign, which also encapsulates the API and guides: #15753.

Once more for those who dont want to run it locally

Current PR state:
Image

@AyushCodes160
Copy link
Author

AyushCodes160 commented Dec 1, 2025

Thank you for the update and for pointing out #15753, sir.
Would you prefer to merge this PR as it is, or should I make further changes to align it with the redesign?
I’m happy to revise anything needed, just let me know.

@hasezoey
Copy link
Collaborator

hasezoey commented Dec 1, 2025

Would you prefer to merge this PR as it is, or should I make further changes to align it with the redesign?

There are still some things open in the mentioned PR that need to be resolved (and either would need to adapt to the other if merged), and i dont know if mongoose wants to go in the direction with this or the other redesign.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs This issue is due to a mistake or omission in the mongoosejs.com documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants