New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update arrow icons to follow same pattern #920

Open

sejas wants to merge 5 commits into trunk from update/arrow-icons

+17 −26

Member

sejas commented Feb 11, 2025 •

edited

Loading

Related issues

Fixes https://github.com/Automattic/dotcom-forge/issues/10438

Proposed Changes

Let's use the same arrow to identify external links ↗

Testing Instructions

Run npm start
Make sure your are logged out in Studio
Navigate to the follow screens and confirm the icon is correct
Search for icon={ external } and observe there are zero occurrences

Header main buttons

Before	After

Overview

Before	After

Previews

Before	After

Assistant

Before	After

Pre-merge Checklist

Have you checked for TypeScript, React or other console errors?

sejas added 5 commits

February 11, 2025 18:37


          Replace arrow icon on header

571e50a


          tab assistant

9a231d1


          tab previews

61d4355


          tab snapshots

f6c641c


          tab overview

9861e9e

sejas self-assigned this

sejas requested a review from a team

February 11, 2025 19:00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet