Skip to content
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

Footer fix and visual enhancements #1009

Merged
merged 3 commits into from
Sep 23, 2020
Merged

Footer fix and visual enhancements #1009

merged 3 commits into from
Sep 23, 2020

Conversation

trevorsaint
Copy link
Contributor

What is the context of this PR?

Due to the length of the legal links a 'small' breakpoint prevented a stacked list from happening. Updating this breakpoint to medium solves the visual bug that happens.

How to review

Look at the visuals below.

Bug
image

Fixed
image

@codecov
Copy link

codecov bot commented Sep 22, 2020

Codecov Report

Merging #1009 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1009   +/-   ##
=======================================
  Coverage   92.89%   92.89%           
=======================================
  Files          34       34           
  Lines        1507     1507           
=======================================
  Hits         1400     1400           
  Misses        107      107           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 03c2711...cbfa504. Read the comment docs.

Copy link
Contributor

@rmccar rmccar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've noticed between page widths 763px and 740px the crest does still go over to the left under the ons logo. Not sure if its supposed to still be doing that

@trevorsaint trevorsaint changed the title Changed Accessibility to Accessibility statement. Changed breakpoint … Footer fix and visual enhancements Sep 23, 2020
@trevorsaint
Copy link
Contributor Author

I've noticed between page widths 763px and 740px the crest does still go over to the left under the ons logo. Not sure if its supposed to still be doing that

This has been resolved in the update @rmccar.

Time has been spent on visual enhancements and other bug fixes.

@Paul-Joel spotted a visual bug when using the footer in Craft and not declaring the crest. This has been resolved.

image

image

image

@trevorsaint trevorsaint requested a review from rmccar September 23, 2020 03:48
@trevorsaint trevorsaint merged commit 9207ea2 into master Sep 23, 2020
@trevorsaint trevorsaint deleted the footer-fix branch September 23, 2020 08:50
boxadesign pushed a commit that referenced this pull request Feb 17, 2023
* Changed Accessibility to Accessibility statement. Changed breakpoint to prevent bug

* Various fixes for footer due to bugs. Visual enhancements when the crest is used or not used
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants