Skip to content

Small option window fixes (Rust, CCPLite and Env)#9265

Merged
mbien merged 1 commit intoapache:masterfrom
mbien:option-window-fixes
Mar 15, 2026
Merged

Small option window fixes (Rust, CCPLite and Env)#9265
mbien merged 1 commit intoapache:masterfrom
mbien:option-window-fixes

Conversation

@mbien
Copy link
Copy Markdown
Member

@mbien mbien commented Mar 12, 2026

  • ccplite icon had the wrong height which caused text alignment issues (now 32px like the other icons)
  • moved rust panel next to other languages
  • fixed Env file hints registration (symptom: layer ordering warning)

check text alignment on C/C++ button (screenshot cut for emphasis)
options_cnd_size_1

fixed:
options_cnd_size_2

(also note that the rust icon is now next to C++ instead of the very end)

@mbien mbien added this to the NB30 milestone Mar 12, 2026
@mbien mbien added C/C++ UI User Interface ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) Rust [ci] enable Rust tests labels Mar 12, 2026
Comment on lines 31 to 35
@OptionsPanelController.SubRegistration(
id = "EnvFileHints",
location = "Env/Hints",
location = "Editor/Hints",
displayName = "#HintsControllerFactory.name"
)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

found via:

WARNING [org.openide.filesystems.Ordering]: Not all children in OptionsDialog/ marked with the position attribute: [Env], but some are: [Actions@0, Advanced@0, Appearance@0, CPlusPlus@0, Editor@0, Html5@0, Java@0, Keywords@0, PreviewExamples@0, Rust@0, Team@0, org-netbeans-modules-php-project-ui-options-PHPOptionsCategory@0, General.instance@100, Editor.instance@200, FontsAndColors.instance@400, Keymaps.instance@600, Java.instance@610, Html5.instance@630, org-netbeans-modules-php-project-ui-options-PHPOptionsCategory.instance@650, CPlusPlus.instance@715, Rust.instance@720, Team.instance@740, Appearance.instance@750, Advanced.instance@800]

lets keep an eye on those warnings, fixing them afterwards can be time consuming (#8916).

Copy link
Copy Markdown
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

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

Only eyeballed this, but looks sane to me. Thank you.

 - ccplite icon had the wrong height which caused text alignment issues
   (now 32px like the other icons)
 - moved rust panel next to other languages
 - fixed Env file hints registration (symptom: layer ordering warning)
@mbien mbien removed the ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) label Mar 15, 2026
@mbien mbien force-pushed the option-window-fixes branch from b680048 to 3d6e38f Compare March 15, 2026 18:20
@mbien mbien merged commit 29421d9 into apache:master Mar 15, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C/C++ Rust [ci] enable Rust tests UI User Interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants