Skip to content

Comments

test(dashboard): strengthen fontWeightStrong assertion for markdown bold#35872

Merged
sadpandajoe merged 1 commit intomasterfrom
fix-markdown-bold
Oct 29, 2025
Merged

test(dashboard): strengthen fontWeightStrong assertion for markdown bold#35872
sadpandajoe merged 1 commit intomasterfrom
fix-markdown-bold

Conversation

@sadpandajoe
Copy link
Member

SUMMARY

Follow up to #35821, where the test was looking for anything greater than 400. Update test to require fontWeightStrong >= 600 (semibold) instead of > 400. This ensures the theme token has adequate weight to show a true visual difference when bold text is rendered.

Values < 600 (like 500/medium) are too subtle and appear similar to normal weight (400), making bold text barely distinguishable. The stricter assertion catches this inadequate configuration.

🤖 Generated with Claude Code

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

Update test to require fontWeightStrong >= 600 (semibold) instead of > 400.
This ensures the theme token has adequate weight to show a true visual
difference when bold text is rendered.

Values < 600 (like 500/medium) are too subtle and appear similar to normal
weight (400), making bold text barely distinguishable. The stricter assertion
catches this inadequate configuration.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@bito-code-review
Copy link
Contributor

bito-code-review bot commented Oct 28, 2025

Code Review Agent Run #7a7f43

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: b8ec243..b8ec243
    • superset-frontend/src/dashboard/components/gridComponents/Markdown/Markdown.test.tsx
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by Bito Logo

Copy link

@korbit-ai korbit-ai bot left a comment

Choose a reason for hiding this comment

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

I've completed my review and didn't find any issues.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

@dosubot dosubot bot added dashboard Namespace | Anything related to the Dashboard dashboard:markdown Related to the markdown component of the Dashboard labels Oct 28, 2025
@bito-code-review
Copy link
Contributor

Interaction Diagram by Bito
sequenceDiagram
participant Config as Backend Config<br/>🔄 Updated | ●●● High
participant Theme as supersetTheme
participant GlobalStyles as GlobalStyles.tsx<br/>🟩 Added | ●●○ Medium
participant Markdown as Markdown Component
participant SafeMarkdown as SafeMarkdown
participant Test as Markdown.test.tsx<br/>🔄 Updated | ●●● High
Config->>Theme: fontWeightStrong: 600
Theme->>GlobalStyles: Apply font-weight to strong tags
GlobalStyles->>Markdown: CSS: strong &#123; font-weight: 600; &#125;
Markdown->>SafeMarkdown: Render markdown with **bold**
SafeMarkdown-->>Markdown: Render <strong> with 600 weight
Test->>Theme: Verify fontWeightStrong >= 600
Theme-->>Test: Assertion passes
Loading

Critical path: Backend Config->supersetTheme->GlobalStyles.tsx->Markdown Component->SafeMarkdown; Test validates theme value >= 600

Note: The test assertion was tightened to require fontWeightStrong >= 600 (semibold) instead of > 400, with clarified rationale that lower values lack visual distinction. This drives a configuration update where the backend fontWeightStrong must be increased from 500 to 600 to ensure bold markdown text renders with adequate visual contrast.

@rusackas rusackas added the 🎪 ⚡ showtime-trigger-start Create new ephemeral environment for this PR label Oct 28, 2025
@github-actions github-actions bot added 🎪 b8ec243 🚦 building Environment b8ec243 status: building 🎪 b8ec243 📅 2025-10-28T23-28 Environment b8ec243 created at 2025-10-28T23-28 🎪 b8ec243 ⌛ 48h Environment b8ec243 expires after 48h 🎪 b8ec243 🤡 rusackas Environment b8ec243 requested by rusackas and removed 🎪 ⚡ showtime-trigger-start Create new ephemeral environment for this PR labels Oct 28, 2025
@github-actions
Copy link
Contributor

🎪 Showtime is building environment on GHA for b8ec243

@github-actions github-actions bot added 🎪 b8ec243 🚦 deploying Environment b8ec243 status: deploying 🎪 b8ec243 🚦 running Environment b8ec243 status: running 🎪 🎯 b8ec243 Active environment pointer - b8ec243 is receiving traffic 🎪 b8ec243 🌐 54.245.149.242:8080 Environment b8ec243 URL: http://54.245.149.242:8080 (click to visit) and removed 🎪 b8ec243 🚦 building Environment b8ec243 status: building 🎪 b8ec243 🚦 deploying Environment b8ec243 status: deploying 🎪 b8ec243 🚦 running Environment b8ec243 status: running 🎪 🎯 b8ec243 Active environment pointer - b8ec243 is receiving traffic labels Oct 28, 2025
@github-actions
Copy link
Contributor

🎪 Showtime deployed environment on GHA for b8ec243

Environment: http://54.245.149.242:8080 (admin/admin)
Lifetime: 48h auto-cleanup
Updates: New commits create fresh environments automatically

@sadpandajoe sadpandajoe merged commit 2db1900 into master Oct 29, 2025
81 of 83 checks passed
@sadpandajoe sadpandajoe deleted the fix-markdown-bold branch October 29, 2025 00:23
@sadpandajoe sadpandajoe added v6.0 Label added by the release manager to track PRs to be included in the 6.0 branch and removed 🎪 b8ec243 🚦 running Environment b8ec243 status: running 🎪 b8ec243 📅 2025-10-28T23-28 Environment b8ec243 created at 2025-10-28T23-28 labels Oct 29, 2025
@sadpandajoe sadpandajoe removed 🎪 b8ec243 ⌛ 48h Environment b8ec243 expires after 48h 🎪 b8ec243 🤡 rusackas Environment b8ec243 requested by rusackas 🎪 b8ec243 🌐 54.245.149.242:8080 Environment b8ec243 URL: http://54.245.149.242:8080 (click to visit) labels Oct 29, 2025
sadpandajoe added a commit that referenced this pull request Oct 29, 2025
…old (#35872)

Co-authored-by: Claude <noreply@anthropic.com>
(cherry picked from commit 2db1900)
@sadpandajoe sadpandajoe removed the v6.0 Label added by the release manager to track PRs to be included in the 6.0 branch label Oct 30, 2025
sadpandajoe added a commit that referenced this pull request Dec 4, 2025
…old (#35872)

Co-authored-by: Claude <noreply@anthropic.com>
(cherry picked from commit 2db1900)
sadpandajoe added a commit that referenced this pull request Dec 4, 2025
…old (#35872)

Co-authored-by: Claude <noreply@anthropic.com>
(cherry picked from commit 2db1900)
sadpandajoe added a commit that referenced this pull request Dec 16, 2025
…old (#35872)

Co-authored-by: Claude <noreply@anthropic.com>
(cherry picked from commit 2db1900)
aminghadersohi pushed a commit to aminghadersohi/superset that referenced this pull request Jan 17, 2026
…old (apache#35872)

Co-authored-by: Claude <noreply@anthropic.com>
(cherry picked from commit 2db1900)
aminghadersohi pushed a commit to aminghadersohi/superset that referenced this pull request Jan 24, 2026
…old (apache#35872)

Co-authored-by: Claude <noreply@anthropic.com>
(cherry picked from commit 2db1900)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dashboard:markdown Related to the markdown component of the Dashboard dashboard Namespace | Anything related to the Dashboard size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants