Skip to content
This repository has been archived by the owner on Jul 10, 2020. It is now read-only.

Fix a spacing bug in the expandable content #471

Merged
merged 4 commits into from
Apr 24, 2017

Conversation

jimmynotjim
Copy link
Contributor

@jimmynotjim jimmynotjim commented Apr 21, 2017

The expandable content has been missing it's bottom spacing for some time. Half the problem is that we started removing bottom margin from the p:last-child a while back. This fixes that particular issue by adding bottom margin to the :after pseudo element. Margin bottom is used over padding because it naturally collapses when the height is set to 0.

Fixes #462

Changes

  • Added spacing to the bottom of expandable content

Testing

  • While this fixes the css side of things, there's also a bug in the expandable js code that is being addressed at Add onReady utility AtomicComponent#4. If you want to test this you'll need to symlink the atomic component into your cf-expandables directory. Follow these steps

  • cd to this repo, checkout this branch, and make the link connections

    cd ~./[Your Repos Directory]/capital-framework/
    git checkout bugfix/expandable-spacing
    npm run cf-link // this makes each component available to other projects and installs the latest from Atomic Component
    
  • In another tab cd to the CF sandbox

    cd ~./[Your Repos Directory]/capital-framework-sandbox
    npm run cf-link // this connects each of the local CF components to this repo
    npm run build // build the stuff
    npm start // start the server
    
  • navigate to http://localhost:3000/components/cf-expandables/

Review

Screenshots

screen shot 2017-04-21 at 2 45 49 pm

Notes

  • Sorry testing is such a bear, if you'd rather, feel free to wait till the AtomicComponent has been fixed to review this PR. The corresponding AtomicComponent release has been published. Testing instructions have been updated.

Checklist

  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows the standards laid out in the front end playbook
  • Passes all existing automated tests
  • New functions include new tests
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged
  • Visually tested in supported browsers and devices
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

@jimmynotjim jimmynotjim merged commit 454a64f into canary Apr 24, 2017
@jimmynotjim jimmynotjim deleted the bugfix/expandable-spacing branch April 24, 2017 21:05
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.

2 participants