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

Period/full-stop is pushed onto a newline in survey footer #9075

Closed
3 tasks
tofumatt opened this issue Jul 25, 2024 · 2 comments
Closed
3 tasks

Period/full-stop is pushed onto a newline in survey footer #9075

tofumatt opened this issue Jul 25, 2024 · 2 comments
Labels
javascript Pull requests that update Javascript code P1 Medium priority Team S Issues for Squad 1 Type: Bug Something isn't working

Comments

@tofumatt
Copy link
Collaborator

tofumatt commented Jul 25, 2024

Bug Description

The survey footer has a bug, likely because of the link component used, that causes the period after the "Terms" link to appear on a new line.

Steps to reproduce

  1. Trigger a user survey
  2. Observe footer has a period on a newline.

Screenshots

image


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

Acceptance criteria

  • It should not be possible for the period after "Terms" in the footer to appear on a newline.

Implementation Brief

  • Add a noFlex = false prop to the Link component, with a related new class:
'googlesitekit-cta-link--no-flex': !! noFlex,
  • Add the noFlex to both Link components in assets/js/components/surveys/SurveyTerms.js
  • Add CSS to this class that sets display: inline;

See proof-of-concept at #9075.

Test Coverage

  • VRTs should continue to pass, except for the survey VRT which should need updating.

QA Brief

Changelog entry

  • Fix full-stop issue in the survey pop-up footer.
@tofumatt tofumatt self-assigned this Jul 25, 2024
@tofumatt tofumatt added P1 Medium priority javascript Pull requests that update Javascript code Type: Bug Something isn't working labels Jul 25, 2024
@tofumatt tofumatt removed their assignment Jul 25, 2024
@binnieshah binnieshah added the Team S Issues for Squad 1 label Jul 25, 2024
@tofumatt tofumatt self-assigned this Jul 25, 2024
@tofumatt tofumatt removed their assignment Jul 25, 2024
@eugene-manuilov eugene-manuilov self-assigned this Jul 26, 2024
@eugene-manuilov
Copy link
Collaborator

IB ✔️

@eugene-manuilov eugene-manuilov removed their assignment Jul 26, 2024
@tofumatt tofumatt assigned tofumatt and unassigned tofumatt Jul 30, 2024
@mohitwp mohitwp self-assigned this Aug 1, 2024
@mohitwp
Copy link
Collaborator

mohitwp commented Aug 4, 2024

QA Update ✅

  • Tested on dev environment.
  • Verified the Storybook story
  • Also, triggered the survey on site.
  • Verified that the survey footer not have a stray period/fullstop ('.') on a newline after the "Terms" link.
  • Also, verified other links on plugin. All looks good.

image

image

image

image

@mohitwp mohitwp removed their assignment Aug 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
javascript Pull requests that update Javascript code P1 Medium priority Team S Issues for Squad 1 Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants