Skip to content

fix(autocomplete): auto-highlighted first option not display correctly if the floating label is disabled - #14414

Merged
vivian-hu-zz merged 1 commit into
angular:masterfrom
crisbeto:13734/autocomplete-auto-active-final
Dec 12, 2018
Merged

fix(autocomplete): auto-highlighted first option not display correctly if the floating label is disabled#14414
vivian-hu-zz merged 1 commit into
angular:masterfrom
crisbeto:13734/autocomplete-auto-active-final

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Fixes the first option not appearing as highlighted if the consumer has set floatLabel="never" and autoActiveFirstOption. It looks like we were depending on the transitioned event from the label to kick off the change detection that shows the highlight.

Fixes#13734.

…y if the floating label is disabled
Fixes the first option not appearing as highlighted if the consumer has set `floatLabel="never"` and `autoActiveFirstOption`. It looks like we were depending on the `transitioned` event from the label to kick off the change detection that shows the highlight.
Fixesangular#13734.
@crisbetocrisbeto added merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: patch This PR is targeted for the next patch release labels Dec 6, 2018
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Dec 6, 2018
@crisbeto

Copy link
Copy Markdown
MemberAuthor

Caretaker note: this is a resubmit of #13774 which had to be reverted due to sync issues.

@mmalerbammalerba added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Dec 6, 2018
@vivian-hu-zz
vivian-hu-zz merged commit 53d2b58 into angular:masterDec 12, 2018
vivian-hu-zz added a commit that referenced this pull request Dec 13, 2018
…correctly if the floating label is disabled (#14414)"
This reverts commit 53d2b58.
vivian-hu-zz added a commit that referenced this pull request Dec 13, 2018
…correctly if the floating label is disabled" (#14505)
* Revert "Revert "fix(snack-bar): announcing same message twice to screen readers" (#14494)"
This reverts commit e9466a4.
* Revert "Revert "fix(dialog,bottom-sheet): invert backdrop color on dark themes" (#14492)"
This reverts commit 3eb7e9c.
* Revert "Revert "fix(menu): allow text wrapping in menu items" (#14493)"
This reverts commit a3da8d2.
* Revert "build: update nodejs bazel rules (#14466)"
This reverts commit a3fef46.
* Revert "docs(form-field): native select reset values not working in examples (#14236)"
This reverts commit 97f52c0.
* Revert "fix(menu): allow text wrapping in menu items (#11430)"
This reverts commit 3661abe.
* Revert "fix(dialog,bottom-sheet): invert backdrop color on dark themes (#13065)"
This reverts commit fe96f38.
* Revert "build: replace deprecated tslint rule with compiler option (#13186)"
This reverts commit 5dfa45f.
* Revert "fix(snack-bar): announcing same message twice to screen readers (#13298)"
This reverts commit 3fb4b23.
* Revert "feat(drag-drop): allow entire group of drop lists to be disabled (#14179)"
This reverts commit 94e76de.
* Revert "fix(drag-drop): prevent text selection while dragging on Safari (#14405)"
This reverts commit 220e388.
* Revert "fix(autocomplete): auto-highlighted first option not display correctly if the floating label is disabled (#14414)"
This reverts commit 53d2b58.
josephperrott pushed a commit to josephperrott/components that referenced this pull request Jan 14, 2019
…y if the floating label is disabled (angular#14414)
Fixes the first option not appearing as highlighted if the consumer has set `floatLabel="never"` and `autoActiveFirstOption`. It looks like we were depending on the `transitioned` event from the label to kick off the change detection that shows the highlight.
Fixesangular#13734.
josephperrott pushed a commit to josephperrott/components that referenced this pull request Jan 14, 2019
…correctly if the floating label is disabled" (angular#14505)
* Revert "Revert "fix(snack-bar): announcing same message twice to screen readers" (angular#14494)"
This reverts commit e9466a4.
* Revert "Revert "fix(dialog,bottom-sheet): invert backdrop color on dark themes" (angular#14492)"
This reverts commit 3eb7e9c.
* Revert "Revert "fix(menu): allow text wrapping in menu items" (angular#14493)"
This reverts commit a3da8d2.
* Revert "build: update nodejs bazel rules (angular#14466)"
This reverts commit a3fef46.
* Revert "docs(form-field): native select reset values not working in examples (angular#14236)"
This reverts commit 97f52c0.
* Revert "fix(menu): allow text wrapping in menu items (angular#11430)"
This reverts commit 3661abe.
* Revert "fix(dialog,bottom-sheet): invert backdrop color on dark themes (angular#13065)"
This reverts commit fe96f38.
* Revert "build: replace deprecated tslint rule with compiler option (angular#13186)"
This reverts commit 5dfa45f.
* Revert "fix(snack-bar): announcing same message twice to screen readers (angular#13298)"
This reverts commit 3fb4b23.
* Revert "feat(drag-drop): allow entire group of drop lists to be disabled (angular#14179)"
This reverts commit 94e76de.
* Revert "fix(drag-drop): prevent text selection while dragging on Safari (angular#14405)"
This reverts commit 220e388.
* Revert "fix(autocomplete): auto-highlighted first option not display correctly if the floating label is disabled (angular#14414)"
This reverts commit 53d2b58.
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 10, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License Agreementmerge: caretaker noteAlert the caretaker performing the merge to check the PR for an out of normal action needed or notetarget: patchThis PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Autocomplete]: autoActiveFirstOption not working when tabbing in with disabled floatLabel

4 participants

@crisbeto@googlebot@mmalerba@vivian-hu-zz