Skip to content

Fix circular profile photo with pastel trans pride flag border - #38

Merged
ZulTheDev merged 1 commit into
mainfrom
fix/circular-trans-flag-profile-border
Oct 8, 2026
Merged

ZulTheDev merged 1 commit into
mainfrom
fix/circular-trans-flag-profile-border

Conversation

@ZulTheDev

Copy link
Copy Markdown
Owner

Keep the portrait, mask and scan overlay circular at all screen sizes. Replace the segmented conic gradient with five equal horizontal pastel stripes in the exact order cyan → pink → white → pink → cyan. No profile image or content changes.

@vercel

vercel Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
zul-portfolio-api Ready Ready Preview Oct 8, 2026 4:57pm UTC

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-08T16:58:22.373137Z 7b54755 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7b54755c7e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread client/src/style.css
Comment on lines +6372 to +6373
aspect-ratio:1 / 1;
border-radius:50%;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Override the cozy profile radii

In the rendered portfolio hero, the container always has cozy-hero-art, whose earlier .cozy-hero-art .hero-photo-frame and .cozy-hero-art .hero-photo-cutout rules set border-radius: 28px!important and 24px!important. Those declarations win over this non-important 50%, so the visible frame and clipping mask remain rounded rectangles at every viewport rather than becoming circular. Remove or override the cozy-specific important radii.

Useful? React with 👍 / 👎.

Comment thread client/src/style.css
Comment on lines +1622 to +1624
linear-gradient(
180deg,
#5BCEFA 0% 20%,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Apply the flag stripes to the active cozy border

When the profile is rendered inside .cozy-hero-art, the later, more-specific .cozy-hero-art .hero-photo-cutout { background: #201a2d; } replaces this entire layered background. The only visible colored frame is therefore the separate diagonal, uneven gradient on .hero-photo-frame, so users never see the five equal horizontal cyan/pink/white/pink/cyan bands introduced here. The stripes need to be applied through the active cozy-specific frame/cutout rule.

Useful? React with 👍 / 👎.

@ZulTheDev
ZulTheDev merged commit 615ca06 into main Oct 8, 2026
3 checks passed

This branch was successfully deployed

1 active deployment
Preview — 7b54755c Deployed Oct 8, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant