Skip to content

Conversation

Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Oct 8, 2025

Purpose

Improve the visibility of the emoji picker in the BlockNote editor by forcing its position to fixed. This prevents it from being visually hidden under the left sidebar, especially when zooming in or out.

issue : 1221

In the screenshot, I intentionally forced the tranform / translate to the left, to demonstrate the behavior.

Capture d'écran 2025-10-08 130717

Proposal

  • Add global CSS rule via createGlobalStyle to target emoji suggestion container
  • Force position: fixed on emoji picker using a data attribute selector
  • Prevent clipping behind sidebar or other layout containers
  • Verified via manual testing and visual confirmation on zoomed layout

@Ovgodd Ovgodd requested a review from AntoLC October 8, 2025 11:27
@Ovgodd Ovgodd self-assigned this Oct 8, 2025
prevents it from hiding under sidebar by applying global style override

Signed-off-by: Cyril <[email protected]>
@Ovgodd Ovgodd force-pushed the fix/1221-emoji-dropdown-idx branch from d9d9615 to 10eabd0 Compare October 8, 2025 11:28
@Ovgodd Ovgodd added bug Something isn't working frontend labels Oct 8, 2025
@Ovgodd Ovgodd marked this pull request as ready for review October 8, 2025 11:31
Copy link
Collaborator

@AntoLC AntoLC left a comment

Choose a reason for hiding this comment

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

Are we sure there is a bug ?
I don't reproduce it on the production environment, neither with the zoom:

-.Docs.3.webm

@Ovgodd
Copy link
Collaborator Author

Ovgodd commented Oct 9, 2025

Are we sure there is a bug ? I don't reproduce it on the production environment, neither with the zoom:

-.Docs.3.webm

hm indeed, On my side I wasn't able to reproduce the problem too, the "only way" to simulate the virgil's issue was to modify manually the transform to watch what happened IF the bug may come back,

wdyt ?

@AntoLC
Copy link
Collaborator

AntoLC commented Oct 13, 2025

hm indeed, On my side I wasn't able to reproduce the problem too, the "only way" to simulate the virgil's issue was to modify manually the transform to watch what happened IF the bug may come back,

wdyt ?

If you don't reproduce the bug, you don't know if your fix works.
We should try to see with Virgile how he gets this bug to be able to reproduce it, if Virgile cannot reproduce the bug, we should close the issue, and the PR.

@Ovgodd
Copy link
Collaborator Author

Ovgodd commented Oct 13, 2025

hm indeed, On my side I wasn't able to reproduce the problem too, the "only way" to simulate the virgil's issue was to modify manually the transform to watch what happened IF the bug may come back,
wdyt ?

If you don't reproduce the bug, you don't know if your fix works. We should try to see with Virgile how he gets this bug to be able to reproduce it, if Virgile cannot reproduce the bug, we should close the issue, and the PR.

Yep that makes sense

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

Labels

bug Something isn't working frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants