-
Notifications
You must be signed in to change notification settings - Fork 3
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
SHS-5963: FE performance refactor theme javascript #1691
base: 11.6.1-release
Are you sure you want to change the base?
SHS-5963: FE performance refactor theme javascript #1691
Conversation
…indiviual JS files
…ithub.com:SU-HSDO/suhumsci into shs-5963-fe-performance-refactor-theme-javascript
…ll, and remove text area template and attach library of page scroll to field
…t, create templates for tables in views and attach library to them
…ideo with caption and expand collapse timeline scripts
…paragraph, attach photo album libraries to the template instead
@cienvaras Ready for review again! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mariannuar Good work, thanks for all the fixes, it was a complicated ticket. I added some extra fixes, we can review them when we come back from the break.
@ahughes3 This one is ready for you to review next year! I was a big refactor, but there shouldn't be any content, visual or behavior change, it was a low level update on how JS code is organized and loaded. I did a thoroughly review before approving the PR and confirmed that all changes work as expected. Mari added instructions on how to change each feature, but if you want to do a quick review you can visit some representative pages and test the interactive elements.
READY FOR REVIEW
Summary
Improve FE performance by refactoring the theme javascript
Need Review By (Date)
12/03
Urgency
medium
Steps to Test
Testing instructions for Carousel Slides Height Script
/components/hero-images-full-width-region-and-main-region
and go to each hero imagemin-height
in the text wrapper:Testing instructions for Table wrap Script
/components/text-area-typography
/default-views/courses
(uses table pattern)/default-views/publications
scroll down to table (uses a normal table)table-wrap
classTesting instructions for Table scope Script
/components/text-area-typography
and/default-views/publications
scroll down to tablescope
attributeTesting instructions for Table pattern Script
/default-views/courses
Testing instructions for Navigation Index Scripts (Main Menu Toggle, Main Menu Nested Toggler, Collapse Main Menu, Secondary Toggler)
Testing instructions for Equal Height Grid Script
/components/collections/collections-postcards/vertical-linked-cards
/private-pages/private-page-collection-cards
Field Formatter Class
field, add a.hb-stretch-vertical-linked-cards
classTesting instructions for Video with caption Script
Text Area / WYSIWYG
component<figure>
elementTesting instructions for Page Scroll Animation Script
/admin/appearance/settings/humsci_colorful
(or traditional)Animation Enhancements
/components/hero-images-full-width-region-and-main-region
and go to /components/text-area-typographyanimate
class when they are being intersected and animations work as expected:Testing instructions for Expanded Collapse Timeline Script
/components/vertical-timeline-without-and-collection
Collapse by default
field checked, are closed by default, and when you open them, they work as expectedTesting instructions for Accordion Toggle All Script
/components/accordion
, edit one or add a new oneAdd Expand/Collapse All
fieldExpand / Collapse All
button works as expectedTesting instructions for Linked Cards All Script
/components/collections/collections-postcards/vertical-cards
/components/collections/collections-postcards/vertical-linked-cards
/default-views/events
is-focused
class inVertical Cards
,Vertical Linked Cards
andDate Stacked Vertical Cards
when the card title is a linkVertical Cards
andVertical Linked Cards
Testing instructions for Editoria11y Script
/components/collections/collections-postcards/vertical-cards
thetoggle-count
text should be black instead of redTesting instructions for Prefered Reduced Motion Script
/components/hero-images-full-width-region-and-main-region
and /components/photo-albumTesting instructions for Main Content Fallback Script
Layout
tab and confirm that theMain Content Anchor Target
is presentMain Content Anchor Target
from the layout.Skip to Main Content
linkTesting instructions for Mega Menu Script
Testing instructions for Add to cal Script
Add to calendar
button and thearia-expanded
status of it changed from to true to false or otherwise depending if the options are being shownTesting instructions for Vertical Tabs Script
Testing instructions for Colorbox Script
/components/photo-album
PR Checklist