Skip to content

Polish mobile responsive for bounty views#1267

Open
sepulchralvoid666 wants to merge 2 commits into
SolFoundry:mainfrom
sepulchralvoid666:feat/bounty-833-mobile-responsive-polish
Open

Polish mobile responsive for bounty views#1267
sepulchralvoid666 wants to merge 2 commits into
SolFoundry:mainfrom
sepulchralvoid666:feat/bounty-833-mobile-responsive-polish

Conversation

@sepulchralvoid666
Copy link
Copy Markdown

Closes #833

Wallet: 9PmjWFwM2mdnjBNjJyfNMTWGCnGAKFpywsLF9xcRF8PK

Tighter padding at 375px, responsive type scale,
horizontal scroll for filter pills, truncated repo names,
and collapsible status labels on small screens.

Closes SolFoundry#833

Wallet: 9PmjWFwM2mdnjBNjJyfNMTWGCnGAKFpywsLF9xcRF8PK
@nagiexplorer88
Copy link
Copy Markdown

This introduces an unconfigured Tailwind breakpoint. BountyCard.tsx now uses xs:inline and xs:hidden, but frontend/tailwind.config.js does not define an xs screen; it only extends theme values and relies on Tailwind defaults (sm, md, etc.).

That means the generated CSS will not include those xs: variants. In practice, hidden xs:inline stays hidden at every width, so the card loses the PRs label and the status text even on desktop. For #833 this should use an existing breakpoint such as sm: or add an xs screen to the Tailwind config.

Tailwind doesn't include an xs screen by default. The xs:
classes produced no CSS output. Using sm: (640px) instead
ensures the responsive hide/show actually works.
@nagiexplorer88
Copy link
Copy Markdown

Confirmed. The follow-up commit replaces the unconfigured xs: classes in BountyCard.tsx with Tailwind default sm: variants:

<span className="hidden sm:inline">{bounty.submission_count} PRs</span>
<span className="sm:hidden">{bounty.submission_count}</span>
...
<span className="hidden sm:inline">{statusLabel}</span>

That addresses the missing generated CSS problem I pointed out; the hide/show classes now use a configured breakpoint.

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.

🏭 Bounty T1: Mobile Responsive Polish

2 participants