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

#1207 Removing bottom margin from sky-row #1212

Merged

Conversation

Blackbaud-ScottFreeman
Copy link
Contributor

@Blackbaud-ScottFreeman Blackbaud-ScottFreeman commented Oct 16, 2017

This simply removes the bottom margin from the sky-row class and lets the content control margins.

Addresses: #1207

@blackbaud-sky-savage
Copy link
Collaborator

Tests passed. Automated cross-browser testing via BrowserStack and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED

Commit: c25965b
Build details: https://travis-ci.org/blackbaud-sky-savage/skyux2/builds/288625243

(Please note that this is a fully automated comment.)

@codecov-io
Copy link

codecov-io commented Oct 16, 2017

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1212   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files         355     355           
  Lines        6620    6620           
  Branches      853     853           
======================================
  Hits         6620    6620

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 e14b374...7214015. Read the comment docs.

@blackbaud-sky-savage
Copy link
Collaborator

Tests passed. Automated cross-browser testing via BrowserStack and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED

Commit: 81d6991
Build details: https://travis-ci.org/blackbaud-sky-savage/skyux2/builds/288722842

(Please note that this is a fully automated comment.)

@blackbaud-sky-savage
Copy link
Collaborator

Tests passed. Automated cross-browser testing via BrowserStack and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED

Commit: 7b73ce1
Build details: https://travis-ci.org/blackbaud-sky-savage/skyux2/builds/290207202

(Please note that this is a fully automated comment.)

@blackbaud-sky-savage
Copy link
Collaborator

Tests passed. Automated cross-browser testing via BrowserStack and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED

Commit: 7214015
Build details: https://travis-ci.org/blackbaud-sky-savage/skyux2/builds/290281127

(Please note that this is a fully automated comment.)

@Blackbaud-SteveBrush Blackbaud-SteveBrush merged commit 3a47952 into blackbaud:master Oct 20, 2017
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.

4 participants