fix: prevent PR status chart text overflow at lg breakpoint#144
Open
jamesrayammons wants to merge 7 commits intoentrius:testfrom
Open
fix: prevent PR status chart text overflow at lg breakpoint#144jamesrayammons wants to merge 7 commits intoentrius:testfrom
jamesrayammons wants to merge 7 commits intoentrius:testfrom
Conversation
…igible - Dashboard models: remove tier, currentTier, qualifiedUniqueReposCount, rawCredibility, credibilityScalar, repositoryUniquenessMultiplier, all bronze/silver/gold tier stats fields - Add isEligible and credibility to MinerEvaluation - Delete Predictions model and API - Remove TierConfig from Configurations model and API - Update leaderboard MinerStats type
Strip Bronze/Silver/Gold tier system across 51 files (~4950 lines removed). Replace with flat isEligible boolean, update dynamic PR threshold from 500 to 300, revamp onboard Getting Started with 7-step inline setup guide matching docs, remove Roadmap and FAQ tabs from onboard page.
🤖 Automated formatting by GitHub Actions
…ting Replace dangerouslySetInnerHTML with ReactMarkdown + rehypeRaw for safe rendering of GitHub-provided HTML in issue bodies and PR conversations. Add missing CSS for lists, paragraphs, tables, and other elements that MUI CssBaseline resets. - Use ReactMarkdown with remarkGfm and rehypeRaw (same as ReadmeViewer) - Add styles for ul/ol, li, p, table, hr, task checkboxes - Fix pre/code nesting so inline code inside blocks renders correctly - Left-align conversation panels to match GitHub layout - Fix fallback text rendering (HTML em instead of markdown asterisks)
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Add minWidth: 0 to flex containers and noWrap to stat text in the dashboard ACTIVE/UNRANKED section to prevent MERGED, OPEN, CLOSED labels and values from overflowing the card boundary. Fixes entrius#138
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
3 participants
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.
Add minWidth: 0 to flex containers and noWrap to stat text in the dashboard ACTIVE/UNRANKED section to prevent MERGED, OPEN, CLOSED labels and values from overflowing the card boundary.
Summary
Fix text overflow in the PR Status Chart (ACTIVE/UNRANKED section) on the Dashboard at the
lgbreakpoint (~1280px). The MERGED, OPEN, CLOSED labels and values on the UNRANKED side were spilling past the card's right edge due to flex items defaulting tomin-width: auto.Related Issues
Fixes #138
Type of Change
Screenshots
Checklist
npm run formatandnpm run lint:fixhave been runnpm run buildpasses