fix: Add divider between password and danger zone sections on Profile page#41
fix: Add divider between password and danger zone sections on Profile page#41
Conversation
…in profile Co-authored-by: JustinhSE <84724234+JustinhSE@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR improves the visual clarity of the Profile → Security tab by adding a horizontal divider between the “Update Password” form and the “Danger Zone” section.
Changes:
- Import the shared
SeparatorUI component. - Render the
Separatorbetween the password form and the Danger Zone section.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <Separator className="my-6" /> | ||
|
|
||
| <CardContent> | ||
| <CardTitle>Danger Zone</CardTitle> | ||
| <CardDescription className="mt-2"> |
There was a problem hiding this comment.
CardContent is currently nested inside another CardContent (the outer one opened above the password form). This results in duplicated padding (p-6 pt-0 applied twice) and makes the Card structure harder to reason about. Consider restructuring so the password form and the Danger Zone are sibling sections (e.g., close the first CardContent before the separator and use a second CardContent, or replace the inner CardContent with a plain div/section).
The Security tab on the Profile page lacked visual separation between the "Update Password" form and the "Danger Zone" (delete account) section, reducing clarity.
Changes
src/pages/Profile.tsx: Imported the existingSeparatorcomponent and placed it between the password form and the Danger ZoneCardContent:The
Separatorrenders as a1pxfull-width horizontal rule using thebg-borderCSS variable, which resolves correctly in both light and dark themes.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.