Skip to content
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.

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

Improve UX/UI when setting up Analytics from the Site Kit WP dashboard widget #8024

Closed
wpdarren opened this issue Jan 3, 2024 · 3 comments
Closed
Labels
Module: Analytics Google Analytics module related issues P1 Medium priority Type: Enhancement Improvement of an existing feature

Comments

@wpdarren
Copy link
Collaborator

wpdarren commented Jan 3, 2024

Bug Description

On the WP Dashboard, when you click on the Set up Analytics (or Complete set up) CTA button on the Site Kit widget, the behaviour is not the same as other buttons. The button is still active and the loading spinner does not appear. It can take a couple of seconds for the set up to commence, so would be an enhancement to show the user that process has started, by making the button inactive and showing the spinner.

WP-dashboard.mp4

Steps to reproduce

  1. Set up Site Kit without Analytics
  2. Go to the WP Dashboard and view the Site Kit widget
  3. Click on the Set up Analytics CTA button and observe behaviour.

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

  • The visual appearance of the "Set up Google Analytics" button on the WP dashboard widget should be consistent with other spinner buttons on Site Kit screens
    • Specifically, the styling of the loading/saving/in-progress state is inconsistent as mentioned below
    • E.g. the in-progress state on the SK dashboard
      image

Implementation Brief

Context: The CircularProgress (spinner) is rendered by the SpinnerButton component. However, the issue arises because the stroke color of the spinner is not correctly applied to the svg.circle element within the WP Dashboard.

In assets/sass/components/wp-dashboard/_googlesitekit-wp-dashboard.scss:

Test Coverage

  • No new tests are required for this change.

QA Brief

  • Follow the steps form description, verify that spinner is showing properly like in the picture in AC

Changelog entry

  • Show the loading spinner in the CTA when setting up Analytics from the Site Kit WP dashboard widget.
@wpdarren wpdarren added Module: Analytics Google Analytics module related issues Type: Enhancement Improvement of an existing feature labels Jan 3, 2024
@aaemnnosttv
Copy link
Collaborator

Thanks @wpdarren – it looks like the spinner is present, just not particularly visible.
image

Thanks for raising 👍

@aaemnnosttv aaemnnosttv added the P1 Medium priority label Jan 3, 2024
@hussain-t hussain-t assigned hussain-t and unassigned hussain-t Jan 4, 2024
@eugene-manuilov eugene-manuilov self-assigned this Jan 4, 2024
@eugene-manuilov
Copy link
Collaborator

IB ✔️

@eugene-manuilov eugene-manuilov removed their assignment Jan 4, 2024
@zutigrm zutigrm self-assigned this Jan 5, 2024
@zutigrm zutigrm mentioned this issue Jan 5, 2024
18 tasks
@zutigrm zutigrm removed their assignment Jan 5, 2024
@hussain-t hussain-t assigned hussain-t and unassigned hussain-t Jan 8, 2024
@techanvil techanvil assigned techanvil and unassigned techanvil Jan 8, 2024
@wpdarren wpdarren self-assigned this Jan 9, 2024
@wpdarren
Copy link
Collaborator Author

wpdarren commented Jan 9, 2024

QA Update: ✅

Verified:

  • The visual appearance of the "Set up Google Analytics" button on the WP dashboard widget is now consistent with other spinner buttons on Site Kit screens.
  • Tested on desktop and smaller viewports.
wp.mp4

@wpdarren wpdarren removed their assignment Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Module: Analytics Google Analytics module related issues P1 Medium priority Type: Enhancement Improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

7 participants