Skip to content

Conversation

@ViktorSlavov
Copy link
Contributor

Change the progress bar to use *ngIf instead of ngClass as the latter requires a specific polyfill and package in order to work in IE (classlist.js).

Additional information (check all that apply):

  • Bug fix
  • New functionality
  • Documentation
  • Demos
  • CI/CD

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code
  • This PR includes API docs for newly added methods/properties
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes
  • This PR includes behavioral changes and the feature specification has been updated with them

@ViktorSlavov ViktorSlavov force-pushed the vslavov/linear-progress-bar-ngIf-8.1.x branch from ff871d8 to c344f15 Compare October 30, 2019 16:13
@ViktorSlavov ViktorSlavov changed the title refactor(linear-bar): use *ngIf instead of ngClass in progress bar template - 8.1.x refactor(circular-bar): use *ngIf instead of ngClass in progress bar template - 8.1.x Oct 31, 2019
@ViktorSlavov ViktorSlavov added the squash-merge Merge PR with "Squash and Merge" option label Oct 31, 2019
@PlamenaMiteva PlamenaMiteva added 💥 status: in-test PRs currently being tested ✅ status: verified Applies to PRs that have passed manual verification and removed ❌ status: awaiting-test PRs awaiting manual verification 💥 status: in-test PRs currently being tested labels Nov 4, 2019
@rkaraivanov rkaraivanov merged commit c3c2342 into 8.1.x Nov 5, 2019
@rkaraivanov rkaraivanov deleted the vslavov/linear-progress-bar-ngIf-8.1.x branch November 5, 2019 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

linear-progress refactoring squash-merge Merge PR with "Squash and Merge" option version: 8.1.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants