Closed theaubmov closed 5 months ago
@burnettk , yes, i ll handle it, for a bit of context here, bpmn-js is limited in terms of extending the ui, so the one option is to override it partially. my approach was to interven as minimally as possible, putting all logic in propertiesPanelProvider.
@coderabbitai review
The updates encompass a series of enhancements to a web application's user interface, particularly focusing on the BPMN modeler. New layout configurations for the properties panel add functionality and aesthetics, while zoom operations improve the modeler's usability post-import. CSS refinements contribute to a more cohesive visual experience, and JavaScript modifications introduce improved event handling and tab management, enriching the application's interactivity and organization.
File Path | Change Summary |
---|---|
app/app.js |
Added layout config for properties panel and zoom on BPMN import. |
app/css/app.css |
Updated styles for body, container, panel, buttons, and overlays. |
app/css/bpmn-js.css |
Overrode default palette styles and updated label fonts. |
app/fileOperations.js |
Added listeners for buttons, initialized tabs, and tab functions. |
app/index.html |
Updated meta charset, rearranged header, added tabs, and reorganized body layout. |
app/spiffworkflow/DataObject/propertiesPanel/DataObjectPropertiesProvider.js |
Added isDefault: true property to the object returned by the createDataObjectSelector function. |
app/spiffworkflow/DataStoreReference/propertiesPanel/DataStorePropertiesProvider.js |
Added isDefault: true property to the group object within the createCustomDataStoreGroup function. |
app/spiffworkflow/IoPalette.js |
Updated function parameters, added event listener, and palette entries. |
app/spiffworkflow/callActivity/propertiesPanel/CallActivityPropertiesProvider.js |
Added isDefault: true property to the called_element group in the createCalledElementGroup function. |
app/spiffworkflow/conditions/propertiesPanel/ConditionsPropertiesProvider.js |
Added isDefault: true property to mark the group as a high priority group located in the General Tab. |
app/spiffworkflow/eventSelect.js |
Added isDefault: true property to the object returned by the getConfigureGroupForType function. |
app/spiffworkflow/extensions/extensionHelpers.js |
Simplified logic in the getExtension function. |
app/spiffworkflow/extensions/propertiesPanel/ExtensionsPropertiesProvider.js |
Reorganized import statements and alignment adjustments, and added isDefault: true property to certain property groups. |
app/spiffworkflow/index.js |
Introduced a new import statement for PropertiesPanelProvider and added it to the exported object. Additionally, it includes a new entry for paletteProvider in the exported object. |
app/spiffworkflow/messages/propertiesPanel/MessagesPropertiesProvider.js |
Introduced a new property isDefault with the value true to the object returned by the createMessageGroup function. |
app/spiffworkflow/properties/PropertiesPanelProvider.js |
Introduced a module that provides functionality for rendering and managing a properties panel for elements in a user interface. It includes methods for handling the display of property groups, making groups collapsible, updating tab content, and managing tab interactions. |
test/spec/BpmnInputOutputSpec.js |
Updated test expectations for input/output titles. |
"In the warren of code, where the functions hop, 🐇
A new panel blooms, and the errors stop.
With a click and a zoom, the model's in view,
A rabbit's craft, making all things anew." 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@burnettk , the issue of active properties panel is resolved.
Summary by CodeRabbit
New Features
Style Updates
Bug Fixes
Documentation
Refactor
Chores