Skip to content

Comments

Maked Project Health Dashboard responsive#3495

Merged
kasya merged 3 commits intoOWASP:mainfrom
anurag2787:fix/responsive-project-health-dashboard
Jan 24, 2026
Merged

Maked Project Health Dashboard responsive#3495
kasya merged 3 commits intoOWASP:mainfrom
anurag2787:fix/responsive-project-health-dashboard

Conversation

@anurag2787
Copy link
Contributor

@anurag2787 anurag2787 commented Jan 23, 2026

Proposed change

This PR fixes layout issues on small screens in the Project Health Dashboard and Metrics pages.

Resolves #3445

Before FIx

Image Image

After Fix

image image

Checklist

  • Required: I followed the contributing workflow
  • Required: I verified that my code works as intended and resolves the issue as described
  • Required: I ran make check-test locally: all warnings addressed, tests passed
  • I used AI for code, documentation, tests, or communication related to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 23, 2026

Summary by CodeRabbit

Style

  • Improved responsive design across the dashboard with adjusted layout breakpoints for better alignment with screen sizes.
  • Optimized component sizing and typography scaling to provide an enhanced experience on mobile and larger devices.
  • Fine-tuned UI proportions for improved visual consistency across viewport sizes.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

Delayed horizontal layout breakpoints from md to lg across dashboard/nav, adjusted metrics page header to be responsive, and reduced several component/icon base sizes with small-screen responsive overrides.

Changes

Cohort / File(s) Change Summary
Dashboard layout & navbar
frontend/src/app/projects/dashboard/layout.tsx, frontend/src/components/ProjectsDashboardNavBar.tsx
Changed responsive breakpoint usage from mdlg, altering when containers switch between flex-col and flex-row and adjusting alignment on medium viewports.
Metrics page header
frontend/src/app/projects/dashboard/metrics/[projectKey]/page.tsx
Replaced a single-row header container with a responsive flex layout that stacks on small screens and spaces on larger screens; made project name typography responsive (text-lg default, text-2xl on larger screens).
Compact component sizing
frontend/src/components/GeneralCompliantComponent.tsx, frontend/src/components/MetricsScoreCircle.tsx
Reduced base container and icon sizes (e.g., h-14 w-14h-10 w-10) and added sm: overrides; decreased default text sizes with sm: increases; preserved existing behavior and conditional rendering.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • arkid15r
  • aramattamara
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title contains a typo ('Maked' instead of 'Made') but accurately describes the main change of making the Project Health Dashboard responsive.
Description check ✅ Passed The description clearly explains the purpose of the PR, references the linked issue (#3445), includes before/after screenshots, and verifies the fix works as intended.
Linked Issues check ✅ Passed The code changes address all requirements from issue #3445: responsive layout adjustments across dashboard and metrics pages with updated breakpoints and responsive sizing for both desktop and small screens.
Out of Scope Changes check ✅ Passed All changes are focused on responsive layout adjustments (breakpoint changes, sizing updates) directly addressing the issue; no unrelated refactoring or feature additions detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@frontend/src/app/projects/dashboard/metrics/`[projectKey]/page.tsx:
- Line 75: In the JSX div in page.tsx that currently uses className "flex gap-2
overflow-x-auto pb-1 sm:w-auto sm:w-full sm:overflow-visible", remove the
redundant/conflicting small-screen width class (either remove "sm:w-auto" or
"sm:w-full") so Tailwind behavior is explicit; decide whether the small
breakpoint should be auto width or full width, keep the chosen "sm:w-*" and
update the className on that div accordingly.

coderabbitai[bot]
coderabbitai bot previously approved these changes Jan 23, 2026
@anurag2787 anurag2787 marked this pull request as ready for review January 23, 2026 10:15
kasya
kasya previously approved these changes Jan 24, 2026
Copy link
Collaborator

@kasya kasya left a comment

Choose a reason for hiding this comment

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

@anurag2787 this looks good 👍🏼
Thank you!

@kasya kasya enabled auto-merge January 24, 2026 03:07
@codecov
Copy link

codecov bot commented Jan 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.57%. Comparing base (b177dfe) to head (a6ee022).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3495   +/-   ##
=======================================
  Coverage   85.57%   85.57%           
=======================================
  Files         461      461           
  Lines       14236    14236           
  Branches     1897     1897           
=======================================
  Hits        12182    12182           
  Misses       1679     1679           
  Partials      375      375           
Flag Coverage Δ
backend 84.47% <ø> (ø)
frontend 88.62% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...p/projects/dashboard/metrics/[projectKey]/page.tsx 95.45% <ø> (ø)
...ntend/src/components/GeneralCompliantComponent.tsx 100.00% <ø> (ø)
frontend/src/components/MetricsScoreCircle.tsx 100.00% <100.00%> (ø)
...rontend/src/components/ProjectsDashboardNavBar.tsx 100.00% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b177dfe...a6ee022. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kasya kasya dismissed stale reviews from coderabbitai[bot] and themself via a6ee022 January 24, 2026 03:24
@sonarqubecloud
Copy link

@kasya kasya added this pull request to the merge queue Jan 24, 2026
Merged via the queue into OWASP:main with commit fd10a28 Jan 24, 2026
36 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make Project Health Dashboard responsive

2 participants