Skip to content

Remove metric icons from stats overview tab bar#25508

Draft
jkmassel wants to merge 1 commit intotrunkfrom
jkmassel/remove-overview-tab-icons
Draft

Remove metric icons from stats overview tab bar#25508
jkmassel wants to merge 1 commit intotrunkfrom
jkmassel/remove-overview-tab-icons

Conversation

@jkmassel
Copy link
Copy Markdown
Contributor

Summary

  • Drop the SF Symbol leading each metric tab (Views, Visitors, Likes, Comments, Posts) in MetricsOverviewTabView.
  • Flatten the HStack wrapper in headerView now that only the label remains.

The icon added visual noise without carrying information beyond the uppercase label. showTrend still gates the trend badge below; only the header icon is removed.

Test plan

  • Stats → Traffic tab: metric tab bar shows only the uppercase labels, no icons
  • Selected metric underline + animation still works
  • MetricsOverviewTabView preview renders correctly in Xcode

The SF Symbol leading each metric label (Views, Visitors, Likes,
Comments, Posts) added visual noise without carrying information not
already in the uppercase label.
@dangermattic
Copy link
Copy Markdown
Collaborator

1 Error
🚫 PR requires at least one label.
2 Warnings
⚠️ View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.
⚠️ PR is not assigned to a milestone.

Generated by 🚫 Danger

@jkmassel jkmassel marked this pull request as draft April 23, 2026 19:02
@wpmobilebot
Copy link
Copy Markdown
Contributor

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number32059
VersionPR #25508
Bundle IDorg.wordpress.alpha
Commit97270ef
Installation URL5fbnpk81ienpo
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Copy Markdown
Contributor

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number32059
VersionPR #25508
Bundle IDcom.jetpack.alpha
Commit97270ef
Installation URL6o1g8god2lfcg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

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