Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Fix chevron positioning #26

Merged
merged 4 commits into from
Jan 15, 2019
Merged

Conversation

Blackbaud-SteveBrush
Copy link
Member

@Blackbaud-SteveBrush Blackbaud-SteveBrush commented Jan 15, 2019

The elements that make up the animated chevron are positioned related to their host element (e.g., sky-chevron). If the element's width changes, the chevron's positioning is affected and appears broken. To see this for yourself, simply add a red border to .sky-chevron and serve the SPA.

screen shot 2019-01-15 at 9 21 03 am

This branch changes the positioning to absolute, so that the various pieces are always positioned correctly, even when the container's width changes.

@codecov-io
Copy link

codecov-io commented Jan 15, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #26   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          28     28           
  Lines         591    591           
  Branches      107    107           
=====================================
  Hits          591    591

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 a3ed172...f6431ef. Read the comment docs.

Copy link
Contributor

@blackbaud-conorwright blackbaud-conorwright left a comment

Choose a reason for hiding this comment

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

LGTM

@Blackbaud-SteveBrush Blackbaud-SteveBrush merged commit 378cd9c into master Jan 15, 2019
@Blackbaud-SteveBrush Blackbaud-SteveBrush deleted the fix-chevron-position branch January 15, 2019 16:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants