Add UI tooltips and move bbox info box #710
Conversation
|
⏱️ Benchmark run finished in 0m 34s 📈 Compared against baseline: 30s 🟢 Generation time is unchanged. 📅 Last benchmark: 2026-01-10 23:42:14 UTC You can retrigger the benchmark by commenting |
There was a problem hiding this comment.
Pull request overview
This pull request adds informational tooltips to the settings interface and moves the bounding box information display from the map overlay to the main input field. The changes improve user experience by providing contextual help and consolidating bbox information.
Changes:
- Added tooltip icons with hover effects to all settings options (Generation Mode, Roof/Interior/Fill Ground toggles, World Scale, Bounding Box, Map Theme, Language, and Telemetry)
- Modified bbox coordinate handling to auto-populate the input field when map selections are made
- Hidden the redundant bbox info box overlay on the map itself
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/gui/index.html | Added tooltip icons to all settings labels with descriptive help text |
| src/gui/css/styles.css | Implemented tooltip styling with hover effects and positioning |
| src/gui/css/bbox.css | Hidden the bbox coordinates display overlay at bottom of map |
| src/gui/js/main.js | Updated bbox handling logic to sync map selections with input field and restore info display on empty input |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.