Skip to content
This repository was archived by the owner on Apr 22, 2025. It is now read-only.

Rewrite/cleanup/simplify ListComboBox code #715

Closed
wants to merge 1 commit into from

Conversation

rock3r
Copy link
Collaborator

@rock3r rock3r commented Dec 5, 2024

  • Split the editable and non-editable list combo boxes
  • Fix up item state management as it was utter chaos
  • Use the appropriate names for style items
  • Add iconTextGap to list item metrics
  • Only do preview selection when mouse-moving over an item, not when hovering, to reduce glitches when scrolling
  • Fix margins and colours
  • Create PopupManager to track and change a popup's visibility and use it in (List)(Editable)ComboBox
  • More!

Fixes #707
Fixes #710

@rock3r rock3r added bug Something isn't working cleanup Improve the project/tooling and/or get rid of tech debt labels Dec 5, 2024
@rock3r rock3r requested a review from hamen December 5, 2024 17:08
@rock3r rock3r self-assigned this Dec 5, 2024
* Split the editable and non-editable list combo boxes
* Fix up item state management as it was utter chaos
* Use the appropriate names for style items
* Add iconTextGap to list item metrics
* Only do preview selection when mouse-moving over an item, not when
  hovering, to reduce glitches when scrolling
* Fix margins and colours
@rock3r rock3r force-pushed the fix-simple-list-item branch from 8b4ad2a to 1bcd3ba Compare December 10, 2024 12:06
@rock3r rock3r changed the base branch from main to fix-slc-excessive-recomposition December 10, 2024 12:14
@rock3r
Copy link
Collaborator Author

rock3r commented Dec 10, 2024

The CI is red here but it should be fine when #724 is merged

@rock3r rock3r linked an issue Dec 11, 2024 that may be closed by this pull request
hamen added a commit to rock3r/intellij-community that referenced this pull request Jan 15, 2025
Merge after JetBrains#2905

Reference JetBrains/jewel#715

Signed-off-by: Ivan Morgillo <imorgillo@gmail.com>
@hamen
Copy link
Contributor

hamen commented Jan 15, 2025

Migrated to IJP as JetBrains/intellij-community#2912

@hamen hamen closed this Jan 15, 2025
intellij-monorepo-bot pushed a commit to JetBrains/intellij-community that referenced this pull request Jan 20, 2025
Merge after #2905

Reference JetBrains/jewel#715

Signed-off-by: Ivan Morgillo <imorgillo@gmail.com>

closes #2912

GitOrigin-RevId: d0b2a0cb59088752dde6f53e8ec00bdb0169adc8
@rock3r rock3r deleted the fix-simple-list-item branch January 23, 2025 13:59
intellij-monorepo-bot pushed a commit to JetBrains/intellij-community that referenced this pull request Mar 12, 2025
Merge after #2905

Reference JetBrains/jewel#715

Signed-off-by: Ivan Morgillo <imorgillo@gmail.com>

closes #2912

(cherry picked from commit d0b2a0cb59088752dde6f53e8ec00bdb0169adc8)


(cherry picked from commit dde7fb4970b3cda5c819b78d6bd6cc20508b9c11)

IJ-MR-155570

GitOrigin-RevId: c0b4da708ff0936fb25e8b7624e78e691ddf6fa6
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working cleanup Improve the project/tooling and/or get rid of tech debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Editable ListComboBox: item padding is different than the one in Swing Expose icon and text modifiers in SimpleListItem
2 participants