Skip to content

Optimize mobile listings and depth iconography#246

Merged
kargig merged 2 commits into
mainfrom
feat/icon_theme_changes
Jun 8, 2026
Merged

Optimize mobile listings and depth iconography#246
kargig merged 2 commits into
mainfrom
feat/icon_theme_changes

Conversation

@kargig

@kargig kargig commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Remove text descriptions from dive site and diving center listing views on mobile viewports to increase data density, save screen real estate, and prevent scrolling fatigue.

Introduce a custom thematic 'DepthIcon' component portraying an ocean waves boundary and a downward-pointing arrow to semantically represent all water depth metrics. Standardize this icon across dive cards, grids, maps, and profile charts.

Unify the contact buttons (Website, Email, and Phone) inside diving center cards using the official 'text-divemap-blue' brand color and a consistent subtle background tint. This streamlines the action palette and reduces visual complexity.

kargig added 2 commits June 7, 2026 15:57
Remove text descriptions from dive site and diving center listing views
on mobile viewports to increase data density, save screen real estate,
and prevent scrolling fatigue.

Introduce a custom thematic 'DepthIcon' component portraying an ocean
waves boundary and a downward-pointing arrow to semantically represent
all water depth metrics. Standardize this icon across dive cards,
grids, maps, and profile charts.

Unify the contact buttons (Website, Email, and Phone) inside diving
center cards using the official 'text-divemap-blue' brand color and a
consistent subtle background tint. This streamlines the action palette
and reduces visual complexity.
Replace remaining occurrences of 'TrendingUp' and 'TrendingDown' with
the custom thematic 'DepthIcon' component in log, site detail, trip
detail, and user profile views. This guarantees global visual
consistency for all water depth metrics.

Additionally, remove unused 'TrendingUp' imports from list pages
such as 'DiveSites.jsx' and 'DiveTrips.jsx' to clean up dead code.
@kargig kargig merged commit 8fa2a1c into main Jun 8, 2026
6 checks passed
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