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.

Sign up for GitHub

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

Jump to bottom

docs: add ie polyfills #1342

Merged
merged 3 commits into from
Jan 29, 2018
Merged

docs: add ie polyfills #1342

merged 3 commits into from
Jan 29, 2018

Conversation

Samuell1
Copy link
Member

@Samuell1 Samuell1 commented Dec 25, 2017 •
edited
Loading

Close #1220

@Samuell1 Samuell1 requested a review from marcosmoura December 25, 2017 10:11
VdustR
VdustR reviewed Dec 25, 2017
View reviewed changes
Copy link
Member

@VdustR VdustR left a comment

Choose a reason for hiding this comment

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

Would you use yarn instead of npm?

marcosmoura
marcosmoura requested changes Dec 26, 2017
View reviewed changes
Copy link
Member

@marcosmoura marcosmoura left a comment

Choose a reason for hiding this comment

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

Can you use yarn instead of NPM?

@VdustR
Copy link
Member

VdustR commented Dec 27, 2017

@marcosmoura he already removed package-lock.json and there is nothing need to change with yarn.lock because babel-polyfill has been depended by babel-cli.

yarn.lock won't be modified after yarn add -D babel-polyfill

Check yarn.lock.

@VdustR VdustR mentioned this pull request Jan 3, 2018
Merged
@marcosmoura marcosmoura merged commit daf58da into dev Jan 29, 2018
@marcosmoura marcosmoura deleted the docs/add-ie-polyfills branch January 29, 2018 16:30
marcosmoura pushed a commit that referenced this pull request Jan 29, 2018
@marcos-hotmart
Merge remote-tracking branch 'origin/dev'
a162827 
* origin/dev: (72 commits)
  docs: add ie polyfills (#1342)
  fix(MdTooltip): `mdActive` never updated to true using `.sync` modifier
  chore: new component script (#1455)
  fix(MdTabs): fix indicator (#1448)
  fix(MdSelect): Reactive options (#1447)
  fix(MdTooltip): overlaying in dialog (#1446)
  fix(MdTabs): correct indicator position while alignment changed (#1442)
  fix(MdProgressSpinner): undefined navigator on SSR (#1441)
  docs(MdTabs): add missing md-icon prop (#1439)
  fix(MdRipple): clear ripples if called programatically (#1431)
  feat(MdListItemExpand): reactive expansion (#1435)
  feat(MdDatepicker): emit date without confirming (#1426)
  feat(MdDatepicker): add dateformat (#1436)
  feat(MdRipple): multiple waves (#1419)
  feat(MdDatepicker): custom first day of a week (#1409)
  feat(MdDatepicker): remove fixed label position (#1417)
  docs(MdDialog): add note about scrollable content (#1416)
  Update sponsorship links (#1427)
  fix(MdAutocomplete): prevent show options twice (#1410)
  fix(MdField): Consider number `0` as hasValue (#1407)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VdustR VdustR VdustR left review comments

@marcosmoura marcosmoura marcosmoura requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Samuell1 @VdustR @marcosmoura