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

feat(dependencies): upgrade covalent dependencies @[email protected] #1096

Merged
merged 15 commits into from
Jan 19, 2018

Conversation

emoralesb05
Copy link
Contributor

@emoralesb05 emoralesb05 commented Jan 18, 2018

Description

Now that material introduced their MatDividerModule, we can directly use it without using MatListModule.. To be able to use this we need to upgrade material to 5.1

So to keep consistency we are gonna have a minimal requirement of [email protected]

Test Steps

  • npm run serve
  • See everything working fine

General Tests for Every PR

  • npm run serve:prod still works.
  • npm run tslint passes.
  • npm run stylelint passes.
  • npm test passes and code coverage is not lower.
  • npm run build:release still works.

@jennmedellin jennmedellin self-requested a review January 18, 2018 23:04
Copy link
Collaborator

@jennmedellin jennmedellin left a comment

Choose a reason for hiding this comment

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

LGTM

@ghost
Copy link

ghost commented Jan 19, 2018

http://localhost:4200/#/components/json-formatter
looks like divider is going right though the middle of the "Original JSON Object:" text.
screen shot 2018-01-18 at 5 56 42 pm

@ghost
Copy link

ghost commented Jan 19, 2018

http://localhost:4200/#/components/dynamic-forms
looks like divider below "Select a type, add element then update" is not staying in its box
screen shot 2018-01-18 at 6 02 58 pm

@ghost
Copy link

ghost commented Jan 19, 2018

@ghost ghost self-requested a review January 19, 2018 02:10
@emoralesb05
Copy link
Contributor Author

The last find is in the code-editor repo, not in this one.. so we have to fix it there @stevenov7

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

LGTM

@emoralesb05 emoralesb05 changed the title chore(dependencies): upgrade covalent dependencies @[email protected] feat(dependencies): upgrade covalent dependencies @[email protected] Jan 19, 2018
@emoralesb05 emoralesb05 merged commit bffb143 into develop Jan 19, 2018
@emoralesb05 emoralesb05 deleted the feature/upgrade-dependencies branch January 19, 2018 18:03
kriswinbush pushed a commit to kriswinbush/covalent that referenced this pull request Feb 20, 2020
…ata#1096)

* chore(dependencies): upgrade covalent dependencies @angular@5.1

* perf(divider): import divider module instead of list module in layout and menu modules

* feat(enums): leverage typescript string enums to remove the <any> cast

* fix(chips): underline not working properly on 5.1 material

* fix(chips): remove disabled chip since now we cant focus a chip if its disabled

* chore(): rollback test.ts

* fix(chips): deselect first option only if it has been selected before and was done internally

* fix(docs): divider issue in json-formatter docs

* fix(docs): divider issue in dynamic-forms docs

* fix(docs): divider issue in layouts overview

* fix(docs): divider issue in logo docs

* fix(docs): divider issue in stepper pattern docs

* fix(docs): divider issue in empty state pattern docs

* feat(divider): replace matInset with [inset]="true"

* chore(): upgrade to angular/cli 1.6.5 and TS 2.6.2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants