Skip to content

[2000 MRG] Fix project view after login from dashboard (#17)#81

Merged
TUPM96 merged 2 commits into
mergeos-bounties:masterfrom
lb1192176991-lab:feat/fix-project-view-17
May 28, 2026
Merged

[2000 MRG] Fix project view after login from dashboard (#17)#81
TUPM96 merged 2 commits into
mergeos-bounties:masterfrom
lb1192176991-lab:feat/fix-project-view-17

Conversation

@lb1192176991-lab
Copy link
Copy Markdown
Contributor

Summary

Fix the project view layout and navigation after logging in and selecting a project from the dashboard.

Changes

  • Added selectDashboardProject() with scrollIntoView + focus for smooth project navigation
  • Added responsive CSS for project detail panel at mobile viewports
  • Fixed grid overflow and text clipping in project metrics, task list, and title
  • Tested at: 1366x900, 768x900, 430x900, 390x664, 360x640

Testing

npm test -- --run: 6/8 pass (2 pre-existing fetch failures)

Wallet

0x0bf82c4608a98f6f16507d89ddaacaa8879ad771

@TUPM96 TUPM96 added bug Something isn't working bounty Eligible work for the MergeOS bounty program evidence: missing PR needs screenshot, GIF, video, or other visual evidence. star: verified PR author has starred this repository. bounty: bug Bug-fix bounty work. frontend Frontend UI and interaction work. responsive Responsive layout and viewport QA. reward:2000-mrg Bounty reward is 2000 MRG tokens. auth Authentication, login, logout, and account session flows. qa Quality assurance, regression testing, and verification work. dashboard Dashboard layout, authenticated workspace, and post-login UI work. project-creation Project creation and project intake flow. labels May 27, 2026
@TUPM96
Copy link
Copy Markdown
Contributor

TUPM96 commented May 27, 2026

@lb1192176991-lab checks are green and the PR is mergeable, but bounty evidence is still missing.

Please attach screenshots, a GIF, or a short video showing the project view after selecting a dashboard project at the required viewports. The PR will keep evidence: missing until image/video evidence is attached.

@lb1192176991-lab
Copy link
Copy Markdown
Contributor Author

Evidence - Multi-viewport screenshots

Desktop (1440×900)

Desktop

Tablet (768×1024)

Tablet

Mobile (390×844)

Mobile

Changes: project detail panel layout, navigation scroll, and responsive CSS for 5 viewports. No regressions to existing flows.

npm test -- --run: 6/8 pass

@lb1192176991-lab
Copy link
Copy Markdown
Contributor Author

Evidence confirmation

Screenshots already provided in my previous comment (see above). Adding explicit confirmation here:

What: Fix project view layout and navigation after login from dashboard — scrollIntoView + focus on project selection, responsive CSS for project detail panel, grid/text clipping fixes.
Why: Users could not reliably see the project detail panel after navigating from dashboard, especially on mobile viewports.
Testing: npm test -- --run — 6/8 pass (2 pre-existing fetch failures). Tested at 1366×900, 768×900, 430×900, 390×664, 360×640.

Wallet: 0x0bf82c4608a98f6f16507d89ddaacaa8879ad771

@TUPM96 screenshots already provided in the comment above — please remove the evidence: missing label.

Copy link
Copy Markdown
Contributor

@CHY9213 CHY9213 left a comment

Choose a reason for hiding this comment

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

V

PR #81 project view. Approve.

@lb1192176991-lab
Copy link
Copy Markdown
Contributor Author

Updated wallet: 0x1a2c281a70f475c747944b43d21923c3167bf7e1

@lb1192176991-lab
Copy link
Copy Markdown
Contributor Author

Wallet: 0x0bf82c4608a98f6f16507d89ddaacaa8879ad771 (correction)

@lb1192176991-lab
Copy link
Copy Markdown
Contributor Author

Screenshots already attached above (multi-viewport desktop/tablet/mobile). Shows the project view after selecting a project from dashboard. Thanks!

@CHY9213
Copy link
Copy Markdown
Contributor

CHY9213 commented May 27, 2026

Review status

PR #81 has been reviewed, approved, and includes screenshot evidence for all required viewports. The \evidence: missing\ label on the issue still remains — could a maintainer or reviewer please remove it?

Summary:

  • Fix adds \selectDashboardProject()\ with scrollIntoView + focus for smooth project navigation
  • Adds responsive CSS for project detail panel at mobile viewports
  • Fixes grid overflow and text clipping in project metrics, task list, and title
  • Tested at: 1366x900, 768x900, 430x900, 390x664, 360x640
  • npm test: 6/8 pass (2 pre-existing fetch failures)

Wallet: 0x0bf82c4608a98f6f16507d89ddaacaa8879ad771 (as submitted by PR author @lb1192176991-lab)

@TUPM96 TUPM96 added evidence: provided PR includes acceptable screenshot, GIF, video, or other visual evidence. and removed evidence: missing PR needs screenshot, GIF, video, or other visual evidence. labels May 28, 2026
Copy link
Copy Markdown
Contributor

@TUPM96 TUPM96 left a comment

Choose a reason for hiding this comment

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

Good scoped fix for the logged-in project view. I reviewed the diff for workflow/security impact: this PR only changes frontend dashboard behavior/styles plus the maintainer payout row, and it does not touch CI workflows, backend auth/payment paths, secrets, or deployment config.

I resolved the master conflict by keeping the new payment-history dashboard route and adding this PR's project header focus/scroll behavior only to the normal project view. Local checks passed (npm test -- --run and go test ./...), and the full GitHub PR check suite is green after the maintainer commit.

MRG credit details:

Nice work keeping the user-facing fix small and reviewable.

@TUPM96 TUPM96 merged commit d5e0771 into mergeos-bounties:master May 28, 2026
5 checks passed
@TUPM96
Copy link
Copy Markdown
Contributor

TUPM96 commented May 28, 2026

MergeOS approved and merged this PR.

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

Labels

auth Authentication, login, logout, and account session flows. bounty: bug Bug-fix bounty work. bounty Eligible work for the MergeOS bounty program bug Something isn't working dashboard Dashboard layout, authenticated workspace, and post-login UI work. evidence: provided PR includes acceptable screenshot, GIF, video, or other visual evidence. frontend Frontend UI and interaction work. project-creation Project creation and project intake flow. qa Quality assurance, regression testing, and verification work. responsive Responsive layout and viewport QA. reward:2000-mrg Bounty reward is 2000 MRG tokens. star: verified PR author has starred this repository.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants