Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

Conversation

dokmic
Copy link
Contributor

@dokmic dokmic commented Sep 17, 2018

fix(select): perform full cleanup of the select drop-down after the close

it affects on the drop-down size if resize is happening after using the select

Fixes #11447

PR Checklist

Please check that your PR fulfills the following requirements:

  • The commit message follows our guidelines
  • Tests for the changes have been added or this is not a bug fix / enhancement
  • Docs have been added, updated, or were not required

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Enhancement
  • Documentation content changes
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
    -[ ] CI related changes
  • Infrastructure changes
  • Other... Please describe:

What is the current behavior?

Issue Number: #11447

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

…lose

it affects on the drop-down size if resize is happening after using the select

Fixes #11447
@googlebot googlebot added the cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ label Sep 17, 2018
@Splaktar Splaktar self-assigned this Sep 19, 2018
@Splaktar Splaktar self-requested a review September 19, 2018 02:35
@Splaktar Splaktar added needs: review This PR is waiting on review from the team type: bug P4: minor Minor issues. May not be fixed without community contributions. labels Sep 19, 2018
Copy link
Contributor

@Splaktar Splaktar left a comment

Choose a reason for hiding this comment

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

LGTM. Tested on Chrome, Safari, and Firefox.

'bottom': '',
'left': '',
'font-size': '',
'min-width': ''
Copy link
Contributor

Choose a reason for hiding this comment

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

Thankfully it looks like it results in these styles being cleared rather than having them set to empty strings.

@Splaktar Splaktar added pr: merge ready This PR is ready for a caretaker to review and removed needs: review This PR is waiting on review from the team labels Sep 19, 2018
@Splaktar Splaktar added this to the 1.1.11 milestone Sep 19, 2018
@Splaktar Splaktar added the pr: lgtm This PR has been approved by the reviewer label Sep 19, 2018
@Splaktar
Copy link
Contributor

Thank you very much for your first contribution to AngularJS Material! It looks great to me.

We'll send this to run some presubmit tests in Google and then merge it if successful.

@Splaktar Splaktar assigned mmalerba and unassigned mmalerba Sep 19, 2018
@mmalerba mmalerba merged commit dfba062 into angular:master Sep 21, 2018
nobitagit pushed a commit to nobitagit/material that referenced this pull request Sep 24, 2018
…lose (angular#11448)

it affects on the drop-down size if resize is happening after using the select

Fixes angular#11447
@dokmic dokmic deleted the wip/fix-select-dropdown-position branch September 27, 2018 09:45
marosoft pushed a commit to marosoft/material that referenced this pull request Nov 11, 2018
…lose (angular#11448)

it affects on the drop-down size if resize is happening after using the select

Fixes angular#11447
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ P4: minor Minor issues. May not be fixed without community contributions. pr: lgtm This PR has been approved by the reviewer pr: merge ready This PR is ready for a caretaker to review type: bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

select: wrong drop-down position after screen resize or orientation change
4 participants