Skip to content

Fix emoji picker position#7146

Merged
estrattonbailey merged 1 commit into
mainfrom
emoji-picker-fix
Dec 18, 2024
Merged

Fix emoji picker position#7146
estrattonbailey merged 1 commit into
mainfrom
emoji-picker-fix

Conversation

@estrattonbailey
Copy link
Copy Markdown
Member

@estrattonbailey estrattonbailey commented Dec 18, 2024

Just needed to be wrapped in a Portal, otherwise it was being positioned relative to the center column. Floating elements like this should always be moved out to the root so they never conflict with other UI.

Test: chat and composer on web

After, before
CleanShot 2024-12-17 at 18 18 27@2x
CleanShot 2024-12-17 at 18 18 43@2x

@arcalinea arcalinea temporarily deployed to emoji-picker-fix - social-app PR #7146 December 18, 2024 00:17 — with Render Destroyed
@github-actions
Copy link
Copy Markdown
Contributor

Old size New size Diff
6.8 MB 6.8 MB 26 B (0.00%)

@gaearon
Copy link
Copy Markdown
Contributor

gaearon commented Dec 18, 2024

tested in composer too?

@estrattonbailey estrattonbailey merged commit a07949e into main Dec 18, 2024
drash-course added a commit to drash-course/social-app that referenced this pull request Dec 18, 2024
* main: (58 commits)
  Fix tests
  Layout tweaks (bluesky-social#7150)
  Trending (Beta) (bluesky-social#7144)
  Fix emoji picker position (bluesky-social#7146)
  Tweak Follow dialog Search placeholder (bluesky-social#7147)
  New progress guide - 10 follows (bluesky-social#7128)
  Pipe statsig events to logger (bluesky-social#7141)
  Fix notifications borders (bluesky-social#7140)
  Refetch empty feed on focus (bluesky-social#7139)
  Read storage on window.onstorage (bluesky-social#7137)
  [ELI5] Tweak wording on the signup screen (bluesky-social#7136)
  alf error screen (bluesky-social#7135)
  add safe area view to profile error screen (bluesky-social#7134)
  Adjust gates (bluesky-social#7132)
  disable automaticallAdjustsScrollIndicatorInsets (bluesky-social#7131)
  Bump more native deps (bluesky-social#7129)
  Update more Expo packages (bluesky-social#7127)
  feat: widen recent search profile link for mobile devices (bluesky-social#7119)
  Fix video uploads on native (bluesky-social#7126)
  Fix post time localization on Android (bluesky-social#6742)
  ...

# Conflicts:
#	src/view/com/profile/ProfileSubpageHeader.tsx
#	src/view/screens/ProfileList.tsx
Signez pushed a commit to Signez/bsky-social-app that referenced this pull request Dec 26, 2024
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.

4 participants