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(module:input): fix input disabled style bug #108

Merged
merged 1 commit into from
Aug 26, 2017
Merged

fix(module:input): fix input disabled style bug #108

merged 1 commit into from
Aug 26, 2017

Conversation

giscafer
Copy link
Contributor

@giscafer giscafer commented Aug 21, 2017

input component and directive

close #103

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
    [ ] Feature
    [ ] Code style update (formatting, local variables)
    [ ] Refactoring (no functional changes, no api changes)
    [ ] Build related changes
    [ ] CI related changes
    [ ] Documentation content changes
    [ ] Application (the showcase website) / infrastructure changes
    [ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

@coveralls
Copy link

Coverage Status

Coverage remained the same at 52.233% when pulling 004c2f7 on giscafer:nz-input-disabled into 15f6aad on NG-ZORRO:master.

@vthinkxie
Copy link
Member

please keep your commit history clean, you can use squash to remove unnecessary commit.

@giscafer
Copy link
Contributor Author

@vthinkxie How can i remove the unnecessary commit in this pr?or i need to recreate a new pr?

i had try git rebase -i xxx to squash this two commit,it will be 3 commits in pr commit list。

@vthinkxie
Copy link
Member

vthinkxie commented Aug 21, 2017

FYI
http://www.jianshu.com/p/964de879904a

@coveralls
Copy link

Coverage Status

Coverage remained the same at 52.233% when pulling 50129c5 on giscafer:nz-input-disabled into 15f6aad on NG-ZORRO:master.

@giscafer
Copy link
Contributor Author

@vthinkxie it must use git push -f to force commit,i just use git push,thanks for the patient answer

@coveralls
Copy link

Coverage Status

Coverage remained the same at 52.233% when pulling 5082955 on giscafer:nz-input-disabled into 15f6aad on NG-ZORRO:master.

@@ -9,14 +9,14 @@ import {
EventEmitter,
ContentChild,
forwardRef,
AfterContentInit, HostListener
AfterContentInit, HostListener, HostBinding
Copy link
Member

@vthinkxie vthinkxie Aug 25, 2017

Choose a reason for hiding this comment

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

please remove HostBinding since it was not used.

input component and directive

close #103
@giscafer
Copy link
Contributor Author

@vthinkxie already remove the unused HostBinding

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 40.05% when pulling f223e31 on giscafer:nz-input-disabled into b442863 on NG-ZORRO:master.

@vthinkxie vthinkxie merged commit 5d666fc into NG-ZORRO:master Aug 26, 2017
suraciii pushed a commit to suraciii/ng-zorro-antd that referenced this pull request Aug 30, 2017
* master:
  refactor(module:dropdown): improve performance (NG-ZORRO#148)
  feat(module:tooltip,popconfirm,popover): support OnPush (NG-ZORRO#143)
  release(0.5.0-rc.3): pre-release 0.5.0-rc.3 (NG-ZORRO#166)
  fix(module:carousel): fix carousel auto play bug (NG-ZORRO#164)
  fix(module:input): fix input disabled style bug (NG-ZORRO#160)
  fix(module:input): fix input disabled style bug (NG-ZORRO#108)
  fix(module:affix,anchor,back-top): fix and improve rxjs usage (NG-ZORRO#159)
  feat(module:affix&anchor&back-top&avatar): add components to library (NG-ZORRO#88)
  fix(module:select): fix select reset bug in form (NG-ZORRO#153)
  fix(module:pagination) fix pagination double binding (NG-ZORRO#146)
  fix(module:select): fix option incorrect active status (NG-ZORRO#141)
  feat(module:root): make nz-root optional (NG-ZORRO#36)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] nz-input width nzDisabled
3 participants