Skip to content
This repository was archived by the owner on Oct 29, 2020. It is now read-only.

Anchor card cta at bottom of the card#4233

Merged
sbsmith86 merged 1 commit intoDoSomethingArchive:devfrom
sbsmith86:permalink-cta-position
Mar 18, 2015
Merged

Anchor card cta at bottom of the card#4233
sbsmith86 merged 1 commit intoDoSomethingArchive:devfrom
sbsmith86:permalink-cta-position

Conversation

@sbsmith86
Copy link
Contributor

Fixes #4194

Made the height of the wrapper of the card column with the text & cta to be 100%. This way, when the cta moves to the bottom of the column on tablet, it will always be at the very bottom of the card no matter how much text is in the copy element above it.

@DoSomething/front-end

Copy link
Contributor

Choose a reason for hiding this comment

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

Is the border-collapse necessary? I thought you really only need this if actually using a <table> in the markup?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@weerd yea, we need it. Without it, the border around the cta doesn't show when it gets set to display: table-footer-group

Copy link
Contributor

Choose a reason for hiding this comment

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

Ahhh, ok!

@weerd
Copy link
Contributor

weerd commented Mar 18, 2015

👍

sbsmith86 added a commit that referenced this pull request Mar 18, 2015
Anchor card cta at bottom of the card
@sbsmith86 sbsmith86 merged commit eae9579 into DoSomethingArchive:dev Mar 18, 2015
@sbsmith86 sbsmith86 deleted the permalink-cta-position branch July 6, 2015 15:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

anchor buttons to the bottom of the permalink

2 participants