hlxsites / merative2

Merative.com site on Franklin
https://merative.com
Apache License 2.0
2 stars 9 forks source link

feat(thought-leadership): Add sidebar filtering and results #274

Closed sahmad-merative closed 1 year ago

sahmad-merative commented 1 year ago

Issue MERATIVE-743

Fixes #

Description

Implement a "Thought Leadership" landing with filtering and results features Solutions Audience Content type Topics

Add a human-readable description/detailed summary of what the PR is changing and any details about how and why. If applicable, include a screenshot indicating an example or examples of what the PR is changing in the application.

New

Changed

Removed

Design Specs

If applicable, add the direct link to the design specs of the component/feature that's part of this PR.

Test URLs

Testing Instruction

If applicable, please describe the tests that you ran to verify your changes. Provide instructions and link to the hlx deploy preview so that QA and the design team can provide proper testing.

aem-code-sync[bot] commented 1 year ago

Hello, I'm Franklin Bot and I will run some test suites that validate the page speed. In case there are problems, just click the checkbox below to rerun the respective action.

aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/drafts/saad/blog PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/drafts/saad/blog PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
helms-charity commented 1 year ago

This seems to have some redundancy so the blog-home.js, are there plans to consolidate?

aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
sharathmrft commented 1 year ago

QA Observations/Issues

1.Load more()counts if it is single digit ex:1,4,6 it should show as Load more(01),Load more(04).instead of Load more(1),Load more(4) 2.White space is appearing in tab and mobile at the top-Reported Issue already 3.Underline is missing for the author name below each card
4.Filters navigation is not aligned properly

Screen shot for the 4th point is attached here side1

aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
proeung commented 1 year ago

@sahmad-merative These changes are much improved! Thanks for consolidating/reusing the existing code, however, I still noticed some bugs have not yet been addressed correctly.

Screen Shot 2023-07-12 at 11 51 22 AM

cc: @keith-kaplan @Shalini-SB

aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
sahmad-merative commented 1 year ago

Hi @proeung

You can find the comments to your feedback below

Updated the description

aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
sharathmrft commented 1 year ago

@sahmad-merative Selected filters are not showing with 'Clear all' enabled link side2

Tested the below scenario its looking correct. Ensure that the filter title/listing on the left sidebar is sorted by ascending.

@proeung Please suggest the way to test the below scenario Card results should be sorted by the latest published date (descending). Sorting order should be the same as what we have Blog landing page

cc @keith-kaplan @Shalini-SB

proeung commented 1 year ago

@sahmad-merative The items that are marked as done in this comment (https://github.com/hlxsites/merative2/pull/274#issuecomment-1633955162) look good, however, I'm still seeing missing styling for the "Thought Leadership" title on the sidebar (see attached). Please ensure that the style aligns with the design.

Screen Shot 2023-07-13 at 3 07 23 PM

Can you help me with this as I am not sure how to fetch the page names from a directory on sharepoint, Currently I am fetching page names from "query-index.xlsx" file as discussed earlier and also mentioned in the "proposed solution" section in ticket description(MERATIVE-743)

See my comment here (https://github.com/hlxsites/merative2/pull/274#discussion_r1262925185). Try taking a stab at what I mentioned, but if you're still blocked on this, I can push up a commit to create a function that returns only the solutions list, If I have availability tomorrow or Monday.

Filter results should be pulling from other content types and not just Blog -- I am pulling all content types but all the thought leadership pages are having content-type as Blog, hence its just showing Blog. Please refer the main "query-index.json" file https://main--merative2--hlxsites.hlx.page/query-index.json

Looks like we have some content tagging cleanup/adjustment to do for the other content types so that we have the right values to filter in the code. @keith-kaplan will get back to us on when the content authors will have this done.

aem-code-sync[bot] commented 1 year ago
Page Scores Audits Google
/thought-leadership PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
proeung commented 1 year ago

Closing out this PR in favor of this one (https://github.com/hlxsites/merative2/pull/286) that's opened by @amol-anand.