Tom Rees-Herdman

Results 525 comments of Tom Rees-Herdman

Adjust the CTA and "New" badge positions for widget areas on mobile and tablet viewports.

Noting that as discussed on [Figma](https://www.figma.com/design/7pSrkEy8t00BcYRAi9LjjH?node-id=1936-22435#843922325), we'll include the alignment of the "New" badge in this issue as well as the widget area CTA.

Margin on top of text of the Change groups CTA should be 32px and not 24px.

Thanks @benbowler. This seems fine for now, we can revisit it more generally if/when we need it in future. IB :white_check_mark:

The AS widgets notification continue to appear under admin settings even after the feature has been disabled.

Thanks for raising this @mohitwp, it is a known issue that's marked as nice-to-have in [Asana](https://app.asana.com/0/1208235052195562/1208499973020585/f). I've labelled it as a P1 and moved it to the Backlog.

The AS widgets notification continue to appear under admin settings even after the feature has been disabled.

Closing, as this will be covered by https://github.com/google/site-kit-wp/issues/9596.

Implement `<PublicationCreate>` component

Hey @hussain-t, the Storybook links in the QAB were pointing to the wrong PR, so I've fixed them. However - once the PR is merged, the Storybook deployment for the...

Console error on view only dashboard.

Thanks for raising this, @mohitwp. I discovered some additional console errors when running through the steps to reproduce, and have updated the issue accordingly.

Console error on view only dashboard.

IB ✅

Console error on view only dashboard.

Thanks for the heads up @mohitwp. The IB should fix this for all secondary users. I've updated the AC to cover secondary admins as well as view-only users to make...

Clear the "temporarily hidden" state for an audience when it's removed from the audience selection

Hi @benbowler, thanks for drafting this IB. A couple of points: > * [ ] Create a new action called `removeDismissedItem`, which takes a slug and resolves using a new...

Clear the "temporarily hidden" state for an audience when it's removed from the audience selection

Thanks @benbowler! IB LGTM. I've bumped the estimate up a level to allow for extra time modelling the scenarios in testing, and because we've generally been underestimating Audience Segmentation issues....