Skip to content

Conversation

@jagodek
Copy link
Contributor

@jagodek jagodek commented Jan 11, 2025

This pull request introduces changes to the createDifferentialConfigurationRow function in src/util/Ui/PropertiesOutliner.js to improve the layout and functionality of the UI elements. The most important changes include the addition of a grid layout, the creation of a helper function for row creation, and the reorganization of UI elements into a more structured format.

Layout and UI improvements:

  • Added a grid container UIDiv to organize the UI elements in a grid layout.
  • Introduced a helper function createRow to streamline the process of creating and adding rows to the grid.
  • Updated the grid template columns for better alignment and spacing of UI elements.
  • Reorganized the addition of UI elements (UISelect, UINumber, UICheckbox, UIButton) into the grid using the new createRow function.

These changes enhance the readability and maintainability of the code while improving the user interface layout.

added labels for options in differential modifier

@jagodek jagodek linked an issue Jan 11, 2025 that may be closed by this pull request
@jagodek jagodek self-assigned this Jan 11, 2025
@jagodek jagodek marked this pull request as ready for review January 11, 2025 16:55
@jagodek jagodek requested a review from grzanka January 11, 2025 17:16
Co-authored-by: Leszek Grzanka <grzanka@agh.edu.pl>
@grzanka grzanka added this pull request to the merge queue Jan 11, 2025
Merged via the queue into master with commit a014ad7 Jan 11, 2025
9 checks passed
@grzanka grzanka deleted the 1945-refactor-of-property-fields-related-to-modifiers branch January 11, 2025 18:20
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.

Refactor of property fields related to modifiers

3 participants