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: (Core) adapt the Breaking Changes from fundamental-styles v.0.13.0 #3895

Merged
merged 2 commits into from
Nov 20, 2020

Conversation

InnaAtanasova
Copy link
Contributor

@InnaAtanasova InnaAtanasova commented Nov 18, 2020

Please provide a brief summary of this pull request.

Address the breaking changes introduced in v.0.13.0 release of fundamental-styles.

  1. Removed form-select from form control
  2. Added a deprecation message to Inline help and brought temporary the styling in ngx
  3. Adapted the template of Breadcrumb

Please check whether the PR fulfills the following requirements

Documentation checklist:

  • [NA] update README.md
  • Breaking Changes Wiki
  • [NA] Documentation Examples
  • [NA] Stackblitz works for all examples

@InnaAtanasova InnaAtanasova added this to the Sprint 50 - Tulum milestone Nov 18, 2020
@InnaAtanasova InnaAtanasova requested a review from a team November 18, 2020 18:22
@InnaAtanasova InnaAtanasova self-assigned this Nov 18, 2020
@netlify
Copy link

netlify bot commented Nov 18, 2020

Deploy preview for fundamental-ngx ready!

Built with commit 06592b4

https://deploy-preview-3895--fundamental-ngx.netlify.app

@InnaAtanasova InnaAtanasova force-pushed the fix/latest-migration-styles branch from 55c8949 to bc58077 Compare November 18, 2020 18:39
@InnaAtanasova InnaAtanasova added the core Core library specific issues label Nov 18, 2020
@@ -1,7 +1,11 @@
<header>Inline Help</header>
<description
>Inline help is used to display help text in a popover, often inline with headers, body text and form labels. It
also uses the fd-popover component and supports much of its functionality.</description
also uses the fd-popover component and supports much of its functionality.
<fd-message-strip [type]="'warning'" [dismissible]="false">
Copy link
Contributor

Choose a reason for hiding this comment

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

unnecessary binding [type]="'warning'" => type="warning"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed. Thanks for the catch

@InnaAtanasova InnaAtanasova merged commit 4fdf211 into master Nov 20, 2020
@InnaAtanasova InnaAtanasova deleted the fix/latest-migration-styles branch November 20, 2020 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core library specific issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants