Chore: tweaks PLP loading [FSSS-211]

[eduardoformiga]

What's the purpose of this pull request?

As now we have the Skeleton, we can change some logic regarding the loading on PLP:

• Removes the useTotalCount hook.
• Aims to load Skeleton from Sort and Filter Button once.
• Keep the latest expanded items from Filter while using the mobile version.

	BEFORE	AFTER
Desktop
Mobile

How to test it?

compare one of the previous deploy versions with the preview from this branch using the PLP or Search page.

Checklist

• CHANGELOG entry added

[netlify]

✔️ Deploy Preview for basestore ready!

🔨 Explore the source changes: c227613

🔍 Inspect the deploy log: https://app.netlify.com/sites/basestore/deploys/6227c1ec866f8b00070a8603

😎 Browse the preview: https://deploy-preview-362--basestore.netlify.app

[vtex-sites]

Preview is ready

This pull request generated a Preview

👀   Preview: https://preview-362--base.preview.vtex.app
🔬   Go deeper by inspecting the Build Logs
📝   based on commit 55937aa

[gatsby-cloud]

Gatsby Cloud Build Report

basestore

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 8m

Performance

Lighthouse report

Metric	Score
Performance	🔶 86
Accessibility	💚 100
Best Practices	💚 100
SEO	💚 93

🔗 View full report

[lucasfp13]

LGTM, nice job! 💯

Just noticed a strange scroll behavior that I'm not sure if is correct:

• When selecting some facet the page scrolls to the top (desktop version);

Screen.Recording.2022-02-23.at.01.03.45.mov

• When opening the Filter component the page scrolls to the top (mobile version).

Screen.Recording.2022-02-23.at.01.02.47.mov

Is it expected?

[eduardoformiga]

LGTM, nice job! 💯

Just noticed a strange scroll behavior that I'm not sure if is correct:

• When selecting some facet the page scrolls to the top (desktop version);

Screen.Recording.2022-02-23.at.01.03.45.mov

• When opening the Filter component the page scrolls to the top (mobile version).

Screen.Recording.2022-02-23.at.01.02.47.mov
Is it expected?

Nice catch! I think it's not expected.
But is also something not related to the changes on this PR since we can simulate on https://base.vtex.app/office as well.
I'm going to look into what is happening 👀

[filipewl]

LGTM!