Skip to content

Conversation

lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Sep 23, 2025

The main thing this fixes is the MongoDB Assistant heading:

Screenshot 2025-09-23 at 14 54 21

I left the other known tags that aren't properly styled alone until we get designs.

Also fixing the welcome message and learn more links to match the latest styling:
Screenshot 2025-09-23 at 16 29 16

@lerouxb lerouxb requested a review from a team as a code owner September 23, 2025 13:55
@Copilot Copilot AI review requested due to automatic review settings September 23, 2025 13:55
@lerouxb lerouxb added the feature flagged PRs labeled with this label will not be included in the release notes of the next release label Sep 23, 2025
@github-actions github-actions bot added the fix label Sep 23, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Fixes inconsistent chat markdown heading styles between Compass and DE by adding explicit font weight styling to headings in the MongoDB Assistant chat interface.

  • Adds explicit font-weight: 700 to heading styles to ensure consistency across environments
  • Includes comment explaining the need for this override due to DE's CSS reset

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@lerouxb lerouxb requested a review from gagik September 23, 2025 13:55
lineHeight: '15px',
marginTop: '4px',
// DE has reset css that sets all font weights to 400
fontWeight: 700,
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we also tackle blockquotes while we're at it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can't get them to work. They are calculated as height: 60px no matter what I do. Will probably address in a separate PR.

@gagik
Copy link
Contributor

gagik commented Sep 23, 2025

I left the other known tags that aren't properly styled alone until we get designs.

can we follow this for the time being?
billede

@lerouxb lerouxb merged commit 5842961 into main Sep 23, 2025
56 of 58 checks passed
@lerouxb lerouxb deleted the markdown-fixes-de branch September 23, 2025 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature flagged PRs labeled with this label will not be included in the release notes of the next release fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants