Skip to content

chore(ui): improve footer on small screens#1243

Open
theborakompanioni wants to merge 3 commits into
develfrom
tbk/footer
Open

chore(ui): improve footer on small screens#1243
theborakompanioni wants to merge 3 commits into
develfrom
tbk/footer

Conversation

@theborakompanioni
Copy link
Copy Markdown
Collaborator

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 9, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 76.16% (🎯 70%) 1131 / 1485
🔵 Statements 76.1% (🎯 70%) 1245 / 1636
🔵 Functions 66.86% (🎯 60%) 345 / 516
🔵 Branches 69.23% (🎯 60%) 783 / 1131
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/components/layout/AppFooter.tsx 83.33% 100% 50% 83.33% 94
src/components/layout/footer/BetaWarningDialog.tsx 80% 75% 50% 80% 41
Generated in workflow #3113 for commit 3acc343 by the Vitest Coverage Report Action

@theborakompanioni theborakompanioni requested a review from parrth20 May 9, 2026 21:33
@theborakompanioni theborakompanioni marked this pull request as ready for review May 9, 2026 21:33
@theborakompanioni theborakompanioni self-assigned this May 9, 2026
@theborakompanioni theborakompanioni added the UI/UX Issue related to cosmetics, design, or user experience label May 9, 2026
Copy link
Copy Markdown
Member

@parrth20 parrth20 left a comment

Choose a reason for hiding this comment

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

Comment on lines +87 to +88
<div className="order-last flex inline-block flex-1 text-xs sm:order-first">
<Trans i18nKey="footer.warning">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small UX nit: footer.warning includes a <br/> in the translation string, and this gets rendered through this <Trans>. Since the wrapper is now flex, that line break may not behave like normal text flow. Could we make this wrapper a normal block/text container instead of flex inline-block?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI/UX Issue related to cosmetics, design, or user experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants