Closed andreamancuso closed 3 months ago
The changes across the dear-imgui
package involve a significant normalization of widget type identifiers, shifting from PascalCase to a consistent kebab-case format. This semantic update enhances readability and aligns with modern naming conventions, improving the overall structure of the widget system. Additionally, some internal logic and rendering adjustments in various components have been implemented to reflect this new identification style while preserving the functionality of existing elements.
Files | Change Summary |
---|---|
.../button.h , .../checkbox.h , .../clipped_multi_line_text_renderer.h , .../combo.h , .../image.h , .../input_text.h , .../map_view.h , .../multi_slider.h , .../separator_text.h , .../slider.h , .../table.h , .../text.h |
Updated m_type initialization from PascalCase to kebab-case (e.g., from "Button" to "di-button" ). |
.../reactimgui.cpp |
Standardized string keys in m_element_init_fn from camel case to kebab case across all widget types. |
.../child.cpp , .../collapsing_header.cpp , .../group.cpp , .../item_tooltip.cpp , .../tabs.cpp , .../text_wrap.cpp , .../tree_node.cpp , .../window.cpp , .../widget.cpp |
Changed m_type member initialization from PascalCase to kebab-case (e.g., "Child" to "child" ). |
.../global.d.ts , .../BulletText.tsx , .../Button.tsx , .../Checkbox.tsx , .../Child.tsx , .../ClippedMultiLineTextRenderer.tsx , .../CollapsingHeader.tsx , .../Combo.tsx , .../DIWindow.tsx , .../DisabledText.tsx , .../Group.tsx , .../Image.tsx , .../InputText.tsx , .../ItemTooltip.tsx , .../MapView.tsx , .../MultiSlider.tsx , .../PlotView.tsx , .../Separator.tsx , .../SeparatorText.tsx , .../Slider.tsx , .../TabBar.tsx , .../TabItem.tsx , .../Table.tsx , .../TextWrap.tsx , .../TreeNode.tsx , .../UnformattedText.tsx |
Updated JSX elements from <widget> to specific components (e.g., <button> , <checkbox> , etc.). |
.../components.tsx |
Removed Unknown from the exported components. |
.../types.ts |
Modified WidgetPropsMap type definition by adding and removing various widget types and their properties. |
.../ReactNativePrivateInterface.js , .../nativeFabricUiManager.ts |
Updated internal logic related to widget types and handling within the React Native interface. |
sequenceDiagram
participant User
participant App
participant Component
User->>App: Interacts with UI
App->>Component: Render component
Component->>Component: Initialize m_type
Component->>App: Return rendered UI
App->>User: Display updated UI
🐰 "In the land where widgets play,
We’ve changed their names in a merry way,
From Caps to kebabs, oh what a sight,
Our code now dances, gleaming bright!
Hoppity hop, let’s celebrate,
New names, new style—it feels so great!"
🐇✨
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?
@coderabbitai please review
@andreamancuso: I will review the changes in the pull request.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
Summary by CodeRabbit
New Features
Bug Fixes
Chores