Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use ALF for the embed consent modal #3336

Merged
merged 5 commits into from
Apr 8, 2024
Merged

Use ALF for the embed consent modal #3336

merged 5 commits into from
Apr 8, 2024

Conversation

mozzius
Copy link
Member

@mozzius mozzius commented Mar 22, 2024

Self-explanatory. Nothing fancy going on here, although if you don't like the web layout please say 😅 I can't decide if it's good or not

Screenshot 2024-03-22 at 12 17 02

Screenshot 2024-03-22 at 12 17 13

@haileyok
Copy link
Contributor

haileyok commented Apr 4, 2024

If we used Prompt instead of Dialog for this would it look any better? (And then put the buttons ontop of each other instead of side by side) No idea, just a thought. I guess maybe it just looks weird right now because it's so wide.

Copy link

github-actions bot commented Apr 5, 2024

Old size New size Diff
6.36 MB 6.36 MB -638 B (-0.01%)

@mozzius
Copy link
Member Author

mozzius commented Apr 5, 2024

@haileyok how's this?

Screenshot 2024-04-05 at 10 32 26

@haileyok
Copy link
Contributor

haileyok commented Apr 5, 2024

Screenshot 2024-04-05 at 4 17 54 PM

I slightly modified the spacing here. As long as you're good with this, everything else LGTM.

Copy link
Contributor

@haileyok haileyok left a comment

Choose a reason for hiding this comment

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

Ran this through a couple of times on web and native. This looks good to me.

@mozzius mozzius merged commit a49a5a3 into main Apr 8, 2024
6 checks passed
estrattonbailey added a commit that referenced this pull request Apr 12, 2024
* origin/main: (455 commits)
  Use getSuggestions endpoint behind the gate (#3499)
  Added `new_profile_scroll_component` to `Gate` type (#3487)
  Fix useGate lint rule (#3486)
  Make bio area scrollable on iOS (#2931)
  Improve Android haptic, offer toggle for haptics in the app (#3482)
  Search - only enable queries once tab is active (#3471)
  [Statsig] Mark Testflight as staging tier (#3470)
  [Statsig] Typecheck gates (#3467)
  Bump to 1.77 (#3468)
  Search - extra tabs (#3408)
  notify slack on production builds (#3461)
  notify slack on production builds (#3460)
  1.76 release preparations (#3459)
  Update zh-CN translation (#3433)
  Italian Localization (#3388)
  [Statsig] Send prev route name (#3456)
  [Statsig] Instrument feed display (#3455)
  Small logic cleanups (#3449)
  Use ALF for the embed consent modal (#3336)
  Onboarding tweaks (#3447)
  ...
@haileyok haileyok deleted the samuel/alf-embed-consent branch June 8, 2024 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants