-
-
Notifications
You must be signed in to change notification settings - Fork 764
UI refactor #712
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UI refactor #712
Conversation
|
⏱️ Benchmark run finished in 0m 38s 📈 Compared against baseline: 30s 📅 Last benchmark: 2026-01-11 02:48:36 UTC You can retrigger the benchmark by commenting |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR implements a UI refactor for the Arnis application, updating it from version 2.4.0 to 2.4.1. The changes focus on streamlining the user interface by consolidating UI elements and improving the layout, while also adding better localization support for dynamic status messages.
Changes:
- Restructured the main UI layout to give the map more screen space (63% width) while condensing controls into a sidebar (32% width)
- Introduced a new
setBboxInfohelper function to centrally manage status message localization and color updates - Updated localization files across 15 languages, removing deprecated keys and adding a new
select_area_promptkey - Improved world creation fallback logic in Rust to use the current directory when Minecraft directory is unavailable
Reviewed changes
Copilot reviewed 21 out of 22 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tauri.conf.json | Version bump to 2.4.1 |
| Cargo.toml | Version bump to 2.4.1 |
| Cargo.lock | Lock file update for version 2.4.1 |
| src/gui.rs | Enhanced world selection logic with better fallback to current directory |
| src/gui/index.html | Restructured layout to move map to full-width left panel and consolidate controls |
| src/gui/css/styles.css | Updated styles for new layout with larger map area and revised control panel |
| src/gui/js/main.js | Added setBboxInfo helper for centralized status message management and localization |
| src/gui/locales/*.json | Removed deprecated keys (select_location, zoom_in_and_choose, select_world, progress) and added select_area_prompt across all 15 language files |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 21 out of 22 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.