Add close button, fix layout, refactor #8755
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Files |
There was a problem hiding this comment.
The icon is still there, it just has no fill color by default. This is to make it easier to style.
There was a problem hiding this comment.
I've moved section headers up to the project status modal level, so they're easier to align and make sure they share the same styles etc.
I left the container in the first instance because without it we lose the scrolling when it overflows.
There was a problem hiding this comment.
We use the exact same container three places, so I extracted it.
There was a problem hiding this comment.
This used to be in ProjectStatusModal, but because of how involved it is, I'd rather treat it as "one widget" and extract it into its own file.
There was a problem hiding this comment.
This is mostly just whitespace-change: I've taken the header out and placed it on the status modal level.
This PR consists of cleanup for the project status modal as well as adding the missing button.
I've added inline comments to explain the different steps.
It now looks like this:
