IULD8-1236: Update Rivet footer for blog and libweb #17
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.
Changes from the previous codepens and the blogs-dev and the rivet-components examples include:
Proposed changes here: https://codepen.io/jameswilson/pen/MWZwOEd
1.- Refactored the tooltip functionality for improved tab navigation, accessibility, and look/feel theming. I’ve adopted the Rivet custom class approach with
rvt-c-tooltip
. This can be used generically on the site for Resource Access icons.Before:
After:
2.- I’ve overhauled the multi-column layout so that the Footer Resources lists stack properly on mobile. I’m using the Rivet nested grid approach to ensure the two columns stack down into what looks like a single column on mobile, and have pulled the button down to the bottom of each two-col list.
3.- Fixed a list alignment issue in the 4-column footer resources lists on desktop and tablet. The tooltip access icons were what was causing the height to be thrown off, fixed with css:
svg { height: 1em; width: 1em; }