Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(feature-card): fix icon position #4412

Merged
merged 10 commits into from
Nov 6, 2020

Conversation

asudoh
Copy link
Contributor

@asudoh asudoh commented Nov 5, 2020

Related Ticket(s)

Refs #4055.

Description

Fixes feature card's icon position in Safari browser, by using <dds-feature-card-footer> to make sure its style (that uses flex box to adjust the icon position) is in effect.

Also removes the styles for the old markup structure.

Changelog

Changed

  • Feature card stories ensuring that <dds-feature-card-footer> is used.

Removed

  • The styles for the old markup structure.

@asudoh asudoh added the package: web components Work necessary for the IBM.com Library web components package label Nov 5, 2020
Fixes feature card's icon position in Safari browser, by using
`<dds-feature-card-footer>` to make sure its style (that uses flex box
to adjust the icon position) is in effect.

Also removes the styles for the old markup structure.

Refs carbon-design-system#4055.
@asudoh asudoh force-pushed the feature-card-footer branch from 3f238cc to fd7b08d Compare November 5, 2020 11:33
@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 5, 2020

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 5, 2020

Copy link
Member

@ariellalgilmore ariellalgilmore left a comment

Choose a reason for hiding this comment

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

LGTM! One weird thing is in feature-card-block-medium where the footer is now positioned after the card. I think this could be because of the margin being applied the component.

@asudoh
Copy link
Contributor Author

asudoh commented Nov 6, 2020

One weird thing is in feature-card-block-medium where the footer is now positioned after the card. I think this could be because of the margin being applied the component.

Good catch @ariellalgilmore - Fixed.

Copy link
Member

@ariellalgilmore ariellalgilmore left a comment

Choose a reason for hiding this comment

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

LGTM 👍 ! Thanks @asudoh!

@kennylam kennylam added the Ready to merge Label for the pull requests that are ready to merge label Nov 6, 2020
@kodiakhq kodiakhq bot merged commit b6a2837 into carbon-design-system:master Nov 6, 2020
@asudoh asudoh deleted the feature-card-footer branch November 6, 2020 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: web components Work necessary for the IBM.com Library web components package Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants