Skip to content

feat: add tooltips to Map Manager Import and Restyle buttons#408

Merged
fank merged 1 commit into
mainfrom
feat/map-manager-button-tooltips
Apr 30, 2026
Merged

feat: add tooltips to Map Manager Import and Restyle buttons#408
fank merged 1 commit into
mainfrom
feat/map-manager-button-tooltips

Conversation

@fank
Copy link
Copy Markdown
Member

@fank fank commented Apr 30, 2026

Summary

  • Adds title tooltips to the Import Map and Restyle All buttons in the Map Manager header (and the empty-state Import button).
  • New i18n keys mm_import_map_tooltip and mm_restyle_all_tooltip, translated across all 8 supported locales.

The descriptions reflect what the handlers actually do:

  • Import Map: accepts a grad_meh ZIP export (internal/server/handler_maptool.go requires .zip) and runs the full processing pipeline.
  • Restyle All: regenerates only the MapLibre style JSON files (topo.json, topo-dark.json, topo-relief.json, color-relief.json) from existing tiles via internal/maptool/restyle.go — no tiles are rebuilt.

Test plan

  • Hover the Import Map button — tooltip appears with grad_meh ZIP description
  • Hover the Restyle All button — tooltip explains it regenerates style files only
  • Switch UI language and verify translations render

Explains what each action does: Import accepts a grad_meh ZIP export
and runs the full processing pipeline; Restyle All only regenerates
MapLibre style JSON files from existing tiles.
@github-actions
Copy link
Copy Markdown

Coverage Report for ui

Status Category Percentage Covered / Total
🔵 Lines 99%
⬆️ +0.01%
4852 / 4901
🔵 Statements 98.4%
🟰 ±0%
6839 / 6950
🔵 Functions 98.02%
🟰 ±0%
1885 / 1923
🔵 Branches 89.2%
⬆️ +0.02%
2240 / 2511
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
ui/src/i18n/locales.ts 100%
🟰 ±0%
100%
🟰 ±0%
100%
🟰 ±0%
100%
🟰 ±0%
ui/src/pages/map-manager/MapManager.tsx 97.1%
⬆️ +0.04%
82.81%
⬆️ +1.78%
97.91%
🟰 ±0%
96.82%
⬆️ +0.07%
52, 62, 83, 95-96, 130, 140, 164
Generated in workflow #561 for commit 1dc5b60 by the Vitest Coverage Report Action

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds descriptive tooltips to the map import and restyle buttons in the Map Manager UI to improve user guidance. It includes the necessary translation keys for these tooltips across all supported languages in the localization configuration. I have no feedback to provide.

@fank fank merged commit 607e040 into main Apr 30, 2026
3 checks passed
@fank fank deleted the feat/map-manager-button-tooltips branch April 30, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant