Skip to content

Update _card.scss - border on one card accordions#28060

Closed
dangowans wants to merge 2 commits into
twbs:masterfrom
dangowans:patch-1
Closed

Update _card.scss - border on one card accordions#28060
dangowans wants to merge 2 commits into
twbs:masterfrom
dangowans:patch-1

Conversation

@dangowans

Copy link
Copy Markdown

Add back the border for accordions that only have a single card inside.

Add back the border for accordions that only have a single card inside.
@dangowans dangowans requested a review from a team as a code owner January 15, 2019 16:40
@mdo

mdo commented Jan 15, 2019

Copy link
Copy Markdown
Member

Do we have a demo in the docs to show this one? Want to make sure we're able to see these things better when merging PRs :).

@dangowans

Copy link
Copy Markdown
Author

First timer here, so not entirely sure what you mean. (Sorry.)

But here's the behaviour I'm seeing.
https://jsfiddle.net/8t5oev9r/1/

And here's the card accordion with my pull request.
https://jsfiddle.net/8t5oev9r/2/

@MartijnCuppens MartijnCuppens left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @dangowans,

Thanks for the PR, but I think we can fix this in one line. Could you just change line 275 to:
&:first-of-type:not(:last-of-type) {

@XhmikosR

XhmikosR commented Feb 7, 2019

Copy link
Copy Markdown
Member

@dangowans: ping

@XhmikosR XhmikosR changed the base branch from v4-dev to master February 19, 2019 14:07
@XhmikosR

Copy link
Copy Markdown
Member

@MartijnCuppens can you tackle this yourself?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants