Autocomplete ng-messages wrong font-size #7477

Activity

1. devversion commented on Mar 9, 2016

   

   devversion

   on Mar 9, 2016

   Member

   More actions

   Great demo, I'll take a look at this soon.
3. topherfangio

   assigned

   devversion

   on Mar 17, 2016
4. topherfangio

   added

   P1: urgentUrgent issues that should be addressed in the next minor or patch release.Urgent issues that should be addressed in the next minor or patch release.

   on Mar 17, 2016
5. devversion commented on Mar 18, 2016

   

   devversion

   on Mar 18, 2016

   Member

   More actions

   Just investigated a bit and it seems to relate to the scope transclusion.

   I tried to create a scope transclusion myself to take a look at its most elegant solution.

   http://codepen.io/DevVersion/pen/WwRqaO?editors=1010

   So just want to notify you, that this issue is in progress.
6. devversion

   added

   needs: investigationThe cause of this issue is not well understood and needs to be investigated by the team or communityThe cause of this issue is not well understood and needs to be investigated by the team or community

   on Mar 18, 2016
7. EladBezalel commented on Mar 19, 2016

   

   EladBezalel

   on Mar 19, 2016

   Member

   More actions

   This is related to the styles applied,
   Take a look at input messages style
8. devversion commented on Mar 20, 2016

   

   devversion

   on Mar 20, 2016

   Member

   More actions

   @EladBezalel Yes actually it is related to the styes. But updating the style selectors won't be the right solution, because the ngMessage isn't retrieving the animation class correctly.

   This is caused by the parent transclusion from the autocomplete.

   (https://github.com/angular/material/blob/master/src/components/input/input.js#L711)
9. EladBezalel commented on Mar 20, 2016

   

   EladBezalel

   on Mar 20, 2016

   Member

   More actions

   👍
10. ThomasBurleson

    modified the milestone: BacklogThis milestone has been deleted on Apr 20, 2016
11. cairomassimo commented on Apr 26, 2016

    

    cairomassimo

    on Apr 26, 2016

    More actions

    I am not sure if this may helps:

    Pull request #5637 removed the code the replaces a wrapper div generated by mdAutocomplete. See this line:
    05d7f24#diff-f700fe0b273e851967e22440f2c464eeL25
    Also, see the occurences of the string md-autocomplete-replace in the whole repository. It was used as a marker class on the div to remove, but it is never taken into account.

    Since mdInputContainer requires ngMessages to be a direct child, this might be the cause of the problem.
12. devversion

    added 2 commits that reference this issue on May 9, 2016

    fix(input): add support for dynamically loaded ngMessage directives.

    ...

    This issue will close once commit cc4b209 is merged into the 'master' branch.

    cc4b209

    fix(input): add support for dynamically loaded ngMessage directives.

    ...

    This issue will close once commit befb1c4 is merged into the 'master' branch.

    befb1c4
13. devversion

    added

    has: Pull RequestA PR has been created to address this issueA PR has been created to address this issue

    and removed

    needs: investigationThe cause of this issue is not well understood and needs to be investigated by the team or communityThe cause of this issue is not well understood and needs to be investigated by the team or community

    on May 9, 2016
14. devversion

    added 2 commits that reference this issue on May 18, 2016

    fix(input): add support for dynamically loaded ngMessage directives.

    ...

    This issue will close once commit f701272 is merged into the 'master' branch.

    f701272

    fix(input): add support for dynamically loaded ngMessage directives.

    ...

    This issue will close once commit 758ea6c is merged into the 'master' branch.

    758ea6c
15. ThomasBurleson

    closed this as completedin 06e7e99on May 19, 2016
16. Splaktar

    removed this from the - Backlog milestone on Feb 23, 2019
17. JonFerrera

    added a commit that references this issue on Sep 28, 2024

    fix(input): add support for dynamically loaded ngMessage directives.

    ...

    This issue will close once commit 06e7e99 is merged into the 'master' branch.

    06e7e99