CAVaccineInventory / site

Home of the static site
https://vaccinateCA.com
MIT License
1 stars 0 forks source link

[Embed] Scroll back to top after results change #906

Closed alexcole closed 3 years ago

alexcole commented 3 years ago

Now anytime we update the list of sites in the embed page, scroll back up to the top so the user isn't randomly in the middle of the list.

Resolves https://github.com/CAVaccineInventory/site/issues/893

Link to Deploy Preview: https://deploy-preview-906--vaccinateca.netlify.app/embed


Manual Testing (QA)

(Note: Use your best judgement for time management. If this PR is a minor content adjustment, you might not invest in the full list of QA checks below. But for medium-large PRs, we recommend a thorough review.)

Open the deploy preview and manually perform the following tests with the browser's developer console open. Fix or log any unexpected errors you see in the console. Check off the following manual tests as you go through them. Make sure to resize the screen and check for poorly positioned or spaced items at mobile and tablet sizes (60%+ of our audience is on mobile devices).

Homepage

/near-me

County Vaccination Site List page

County Policies page

Other pages

Did you remember to:

alexcole commented 3 years ago

@khaister - We should make sure to think about how this and https://github.com/CAVaccineInventory/site/pull/897/files should work together. I'm not quite sure what is supposed to happen if someone selects a site in embed and then does a search or moves the map.