[JEWEL-1189] Fix ComboBox Popup Clipping Items Vertically #3375
+32
−18
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DanielSouzaBertoldi
force-pushed
the
dsb/JEWEL-1189
branch
2 times, most recently
from
f45594e to
faa3ec0
Compare
DanielSouzaBertoldi
force-pushed
the
dsb/JEWEL-1189
branch
from
faa3ec0 to
277643b
Compare
faogustavo
approved these changes
Jan 8, 2026
Collaborator
|
Ready to merge |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
Continuing on our efforts to improve user experience all across
the globeJewel, I present to you the latest groundbreaking UI improvement since sometime last year: a sweetListComboBoxpopup that doesn't clip item names when vertically scrolling through the options.This bug led to countless reports by angry users demanding the situation to be solved ASAP, with the utmost quality and urgency. Since we are here to serve the people and we have a deep aversion to disappointing users, we not only obliged, we made sure to cook up this solution with keen eagle eyes, careful not to inadvertently break anything else.
Changes
contentPaddingproperty was applied as a modifier ofSelectableLazyColumnusing.padding(contentPadding), which caused it to be fixed at the edges and clip the content. Now, we just passcontentPaddingas the parameter toSelectableLazyColumn, which will make it scroll with the content instead of staying fixed.comboBoxesRow()to make it easier to know which view is rendered by Swing or Compose. This is basically unnecessary, so yeah, I can revert it if it's not really helpful/worth it.Evidences
Swing Behavior
Screen.Recording.2026-01-07.at.16.08.00.mov
Screen.Recording.2026-01-07.at.16.10.42.mov
Swing ComboBox Panel Enhanced