SakulFlee/Orbital
1
0
Fork 0
Code Issues 51 Pull requests 24 Projects Releases 1 Packages Wiki Activity

Primer: UI #754

Open
SakulFlee wants to merge 7 commits from ui-primer into main
pull from: ui-primer
merge into: SakulFlee:main
Conversation 1 Commits 7 Files changed 17 +1285 -12
SakulFlee commented 2026-08-06 02:53:58 +02:00
Owner
Copy link
No description provided. 
- Add orbital_egui crate with git dependency on egui (rev 3fcadda for wgpu 30)
- EguiModule implements Module trait, inserts EguiOverlay as RenderOverlayResource
- EguiState ECS resource with toggle system (F2 key)
- RenderOverlay trait extended with on_window_event() for input forwarding
- RenderOverlayContext extended with window reference
- Deferred initialization pattern (winit_state created on first frame)
- PerformancePanel showing viewport size, dt, FPS
- ScenePanel placeholder for future ECS inspection

Workspace compiles cleanly with no warnings.
Press F2 to toggle the egui debug UI overlay.
File restructuring:
- state.rs: EguiState + EguiPanels resource
- module.rs: EguiModule struct, builder, Module impl
- system.rs: sys_egui_toggle
- overlay.rs: EguiOverlay (reads panels from ECS)
- lib.rs: thin re-export module

ECS integration:
- Panels stored as EguiPanels ECS resource instead of on overlay
- Any module can register panels via get_resource_mut::<EguiPanels>()
- EguiModule::with_panel() still works (inserts into resource)
- Removed clone_panel() from Panel trait (no longer needed)

Workspace compiles cleanly.
Added UI Demo
Some checks failed
ci/woodpecker/pr/linting Pipeline failed
Details
ci/woodpecker/pr/test unknown status
Details
5bd534b171
SakulFlee commented 2026-08-06 02:54:13 +02:00
Author
Owner
Copy link

Closes #584

Closes #584
style: auto-format code
Some checks failed
ci/woodpecker/pr/linting Pipeline failed
Details
ci/woodpecker/pr/test unknown status
Details
813856a060
Some checks failed
ci/woodpecker/pr/linting Pipeline failed
Required
Details
ci/woodpecker/pr/test unknown status
Required
Details
Some required checks were not successful.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin ui-primer:ui-primer
git switch ui-primer
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Context: Async
Async specific, be it bug or feature

  
Context: Bug
Something isn't working as intended

  
Context: CI
GitHub Actions / CI related tasks

  
Context: Dependencies
Pull requests that update a dependency file

  
Context: Documentation
Improvements or additions to documentation

  
Context: Enhancement
New feature or request

  
Context: Example
Example related issues

  
Context: Macro

   
Context: Runtime
Runtime related issues

  
Difficulty: Easy

   
Difficulty: Hard

   
Difficulty: Medium

   
Platform: Android
ONLY if this is platform specific!

  
Platform: iOS
ONLY if this is platform specific!

  
Platform: Linux
ONLY if this is platform specific!

  
Platform: macOS
ONLY if this is platform specific!

  
Platform: Web
ONLY if this is platform specific!

  
Platform: Windows
ONLY if this is platform specific!

  
Type: Discussion

   
Type: Evaluation
Further evaluation is needed

  
Type: Tracker
Issues acting as trackers; I.e. a combination of issues of the same overall topic.

No labels
Context: Async
Context: Bug
Context: CI
Context: Dependencies
Context: Documentation
Context: Enhancement
Context: Example
Context: Macro
Context: Runtime
Difficulty: Easy
Difficulty: Hard
Difficulty: Medium
Platform: Android
Platform: iOS
Platform: Linux
Platform: macOS
Platform: Web
Platform: Windows
Type: Discussion
Type: Evaluation
Type: Tracker
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
SakulFlee
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
SakulFlee/Orbital!754
No description provided.