Fix dashboard layout regression after login#143
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates dashboard-related CSS to improve layout behavior (shrinking/truncation in grid/flex contexts), adjust responsive profile display, and refine amber styling.
Changes:
- Added
min-width: 0and new grid rules to prevent overflow and allow proper text truncation in dashboard headers/titles. - Tweaked responsive
.dash-profilelayout to a compact avatar-only presentation at smaller widths. - Updated the amber icon background color and removed an
overflow-x: cliprule.
Comment on lines
4004
to
4008
| .how-step-icon.amber, | ||
| .why-icon.amber { | ||
| color: #d97706; | ||
| background: #fff7ed; | ||
| background: #fffbeb; | ||
| } |
Comment on lines
6638
to
6644
| .dash-profile { | ||
| grid-template-columns: 32px 14px; | ||
| max-width: 50px; | ||
| } | ||
| .dash-profile > span:not(.profile-avatar) { | ||
| display: none; | ||
| } |
Comment on lines
5412
to
5415
| grid-template-columns: minmax(0, 1fr) 286px; | ||
| gap: 16px; | ||
| padding: 18px; | ||
| overflow-x: clip; | ||
| } |
| font-size: 12px; | ||
| } | ||
| } | ||
|
|
TUPM96
added
bug
Contributor
|
Maintainer check: tagged this PR with the bounty/reward/evidence/star structure. Star is verified, but this PR is blocked because it has a conflict against current master in frontend/src/styles.css and it does not include visual evidence yet. Please rebase/resolve the stylesheet conflict and attach before/after dashboard evidence before requesting another merge pass. |
Contributor
Author
TUPM96
pushed a commit
that referenced
this pull request
May 29, 2026
Maintainer merge: reviewed the updated notification count/read-all changes after the author response. Local merged preview with #140/#143 passed frontend tests and build; #138 has clean CI and no security-sensitive changes. Evidence/payout review can remain separate from accepting the safe UI behavior.
Contributor
|
Maintainer follow-up: merged after rechecking the updated branch against current master. Validation:
I noticed a non-functional trailing blank-line warning locally, but it does not affect runtime behavior, so I did not block merge on that alone. |
This was referenced May 29, 2026
Contributor
|
MRG credit issued after maintainer merge.
|
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
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.
Fixes #16