-
Notifications
You must be signed in to change notification settings - Fork 1
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
Adds Mobile and Stacked templates to theme #120
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
cmaustnu
commented
May 29, 2020
•
edited by mestradanu
Loading
edited by mestradanu
- Updates page template to allow for mobile-order swapping
- Adds new template parts
- Adds terms page template
- Modifies existing template parts for cleanup and new template
- Adds new template header logo
- Updates header with template parts
- Updates / adds styles for new template
- Adds new icons to theme to replace fontello
- Adds floating labels js to theme
- Updates spacing classes
- Turns button styles into placeholders
- Adds email type to input styles
- Removes ::after from select and sets background-image as arrow
- Updates to terms template specific styles
- Adds images for mobile re-order layout
- Adds JS for fixed-cta for mobile re-order layout
- Updates image paths and alt text on fixed-cta
- Adds scss for mobile re-order layout
- Updates spacing utility classes and class names
- Adds fixed-cta to footer
- Slight scss adjustments
- Fixes eslint issues
- Adds phone icon for mobile layout
- Adds image to quote on mobile
- Updates fixed-cta display classes
- Breaks mobile theme stylesheet up into atomic structure
- Adds new hero template part variation
- Initial refactor of terms template scss into atomic structure
- Fixes visual conflicts with mobile
- Replaces static value props with sidebar
- Updates template name and associated paths
- Updates file names and classes
- Updates styles for consent description
- Updates linter issue
- Adds custom tooltip js / scss
- Updates spacing classes with logic
- Fixes display issue with quote on ie
- Updates tooltip styles
- Adds logic to spacing classes
- Fixes phpcs issues
- Updates linter issues
- Updates spacing classes
- Adds compiled assets
mestradanu
suggested changes
May 29, 2020
cmaustnu
force-pushed
the
new-page-templates-mobile-stacked
branch
2 times, most recently
from
June 1, 2020 17:04
f6b937c
to
750a68b
Compare
mestradanu
suggested changes
Jun 1, 2020
cmaustnu
force-pushed
the
new-page-templates-mobile-stacked
branch
from
June 2, 2020 16:11
750a68b
to
de85274
Compare
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.