Skip to content

fix(feedback): Render AiPrivacyNotice inline with spam detection description#114149

Closed
chromy wants to merge 1 commit intomasterfrom
hector.dearman/fix-ui-feedback-ai-privacy-notice-inline
Closed

fix(feedback): Render AiPrivacyNotice inline with spam detection description#114149
chromy wants to merge 1 commit intomasterfrom
hector.dearman/fix-ui-feedback-ai-privacy-notice-inline

Conversation

@chromy
Copy link
Copy Markdown
Contributor

@chromy chromy commented Apr 28, 2026

Move AiPrivacyNotice inside the muted description Text for the "Enable Spam Detection" toggle so the notice flows inline with the surrounding copy instead of rendering as a separate block beneath it.


Before:

CleanShot 2026-04-28 at 13 30 35@2x

After:

CleanShot 2026-04-28 at 13 30 24@2x

Agent transcript: https://claudescope.sentry.dev/share/QVleynSd_XXIfd9VA9-rjxlOiXheNactszNG3txqLzA

…ription

Place AiPrivacyNotice inside the muted description Text so the notice flows
inline with the surrounding copy instead of appearing as a separate block
below it.

Agent transcript: https://claudescope.sentry.dev/share/iM5aerIoOt9nHkrsInoSykRZv1tYLVKEdzsB_gQN6E4
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 28, 2026
@chromy chromy marked this pull request as ready for review April 28, 2026 12:35
@chromy chromy requested a review from a team as a code owner April 28, 2026 12:35
@TkDodo
Copy link
Copy Markdown
Collaborator

TkDodo commented Apr 28, 2026

why are the checkboxes so misalinged 😮 ?

@TkDodo
Copy link
Copy Markdown
Collaborator

TkDodo commented Apr 28, 2026

I would say this is the layout we’d want? For some reason, we’re not using Row layout for all rows but Stack layout with custom Texts for the last two rows.

Screenshot 2026-04-28 at 14 44 05

@TkDodo TkDodo closed this Apr 28, 2026
@TkDodo
Copy link
Copy Markdown
Collaborator

TkDodo commented Apr 28, 2026

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

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants