Skip to content

Redesign Add favorite UI Review#1856

Merged
alisa911 merged 6 commits into
mainfrom
070726
Jul 7, 2026
Merged

Redesign Add favorite UI Review#1856
alisa911 merged 6 commits into
mainfrom
070726

Conversation

@alisa911

@alisa911 alisa911 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@alisa911

alisa911 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

#543

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 standardizes “icon button” and “loading progress” UI across the map web app by introducing shared wrapper components (IconBtn, OverlayLinearProgress) and updating multiple menus/panels to use them, along with a new shared hover background CSS variable.

Changes:

  • Introduces reusable UI components: IconBtn (standard icon-button styling + active state) and OverlayLinearProgress (overlay-positioned progress bar).
  • Replaces many direct IconButton / LinearProgress usages with the new shared components to unify styling/behavior.
  • Refactors/adjusts favorite folder selection and description editing panels to improve layout/UX consistency.

Reviewed changes

Copilot reviewed 38 out of 39 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
map/src/variables.css Adds shared CSS variable for secondary button hover background.
map/src/menu/weather/WeatherHeader.jsx Uses IconBtn for the close/back control.
map/src/menu/tracks/TracksMenu.jsx Replaces LinearProgress with OverlayLinearProgress.
map/src/menu/tracks/CloudTrackItem.jsx Replaces LinearProgress with OverlayLinearProgress.
map/src/menu/trackfavmenu.module.css Uses shared CSS variable for hover background.
map/src/menu/settings/SettingsMenu.jsx Uses IconBtn for the close button.
map/src/menu/settings/settings.module.css Removes close icon color/hover styling (relies on IconBtn).
map/src/menu/settings/CloudTrash.jsx Uses IconBtn for the back button.
map/src/menu/settings/CloudChanges.jsx Uses IconBtn for the back button.
map/src/menu/search/explore/WikiPlacesItem.jsx Replaces LinearProgress with OverlayLinearProgress.
map/src/menu/search/explore/ExploreMenu.jsx Uses IconBtn and OverlayLinearProgress in the header.
map/src/menu/MainMenu.js Formats SearchManager imports.
map/src/menu/favorite/FavoritesMenu.jsx Replaces LinearProgress with OverlayLinearProgress.
map/src/menu/configuremap/TerrainConfig.jsx Uses IconBtn for the back button.
map/src/menu/configuremap/PoiCategoriesConfig.jsx Uses IconBtn for the back button.
map/src/menu/configuremap/configuremap.module.css Removes close icon color/hover styling (intended to rely on IconBtn).
map/src/map/layers/MvtLayerConfig.js Fixes missing semicolon formatting.
map/src/login/login.module.css Removes close icon color/hover styling (relies on IconBtn).
map/src/login/Login.jsx Uses IconBtn for the close button.
map/src/login/CreateAccount.jsx Uses IconBtn for the close button.
map/src/login/ChangeResetPwd.jsx Uses IconBtn for the close button.
map/src/infoblock/infoblock.module.css Removes close icon color/hover styling (relies on IconBtn).
map/src/infoblock/components/wpt/WptPhotoList.jsx Uses IconBtn for the back button.
map/src/infoblock/components/tabs/PointsTab.jsx Replaces LinearProgress with OverlayLinearProgress.
map/src/infoblock/components/InformationBlock.jsx Replaces LinearProgress with OverlayLinearProgress.
map/src/infoblock/components/favorite/WptEditPanel.jsx Replaces LinearProgress with OverlayLinearProgress.
map/src/infoblock/components/favorite/structure/folderSelectionPanel.module.css Adjusts folder-selection icon styling to match new button behavior.
map/src/infoblock/components/favorite/structure/FolderSelectionPanel.jsx Uses IconBtn, new arrow icons, and divider layout updates.
map/src/infoblock/components/favorite/structure/DescriptionPanel.jsx Refactors rich text editor wiring to avoid re-init and save-on-close behavior.
map/src/frame/pro/ProFeatures.jsx Uses IconBtn for the close button.
map/src/frame/pro/pro.module.css Removes close icon color/hover styling (relies on IconBtn).
map/src/frame/components/progress/progress.module.css Adds positioning styles for overlay progress bar.
map/src/frame/components/progress/OverlayLinearProgress.jsx New overlay wrapper around MUI LinearProgress.
map/src/frame/components/items/SmallRadio.jsx Simplifies rendering.
map/src/frame/components/items/items.module.css Updates chevron hover/shape styling using shared hover background variable.
map/src/frame/components/header/AppBarWithBtns.jsx Uses IconBtn for back/close controls.
map/src/frame/components/btns/IconBtn.jsx New shared icon button wrapper with optional active state styling.
map/src/frame/components/btns/buttons.module.css Adds shared .iconBtn and .iconBtnActive styles.
map/src/assets/icons/ic_action_arrow_down.svg Adds an arrow-down icon asset used by folder selection expand/collapse.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread map/src/menu/configuremap/configuremap.module.css
Comment thread map/src/infoblock/components/InformationBlock.jsx Outdated
@alisa911 alisa911 requested a review from RZR-UA July 7, 2026 09:40
@alisa911 alisa911 merged commit 009a96b into main Jul 7, 2026
@alisa911 alisa911 deleted the 070726 branch July 7, 2026 10:29
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.

3 participants