Skip to content

Fix FrequencySelector component on small screens and add "Current subscription" badge #11959

@benbowler

Description

@benbowler

Bug Description

Screenshots

Image Image

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

Acceptance criteria

Implementation Brief

  • Update assets/sass/components/email-reporting/_googlesitekit-frequency-selector.scss to match the layout for mobile - each frequency item taking the full width and falling in vertical columns

Test Coverage

  • Update any failing VRT

QA Brief

  • Known Issue: Currently, if a frequency is already saved, then it takes a couple of seconds for the saved settings to load which causes movement and changes - have already created an issue here for it.
  • Test the side panel in both desktop and mobile viewports.
  • Open the side panel without any frequency being saved.
  • Save a frequency and test both layouts. Test the case when the selected frequency is equal to the save frequency and then try selecting and saving another frequency. The colours and fonts should all match the Figma designs linked in the AC for mobile and large screen viewports.

Changelog entry

  • Fix FrequencySelector component on small screens and add "Current subscription" badge

Metadata

Metadata

Assignees

No one assigned

    Labels

    P0High priorityTeam SIssues for Squad 1Type: BugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions