Skip to content

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Dec 7, 2016

Switches the connected-position to pick the fallback position with the largest visible area, if all of the fallbacks didn't fit into the viewport.

Fixes #2049.

@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Dec 7, 2016
@crisbeto crisbeto requested a review from jelbourn December 7, 2016 22:08
…largest area of the element

Switches the `connected-position` to pick the fallback position with the largest visible area, if all of the fallbacks didn't fit into the viewport.

Fixes angular#2049.
@crisbeto crisbeto force-pushed the 2049/connected-position-area branch from 4f184d1 to 2f5a2ae Compare December 18, 2016 10:50
@jelbourn jelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker and removed pr: needs review labels Dec 20, 2016
@jelbourn jelbourn merged commit 4f5b9c5 into angular:master Dec 20, 2016
crisbeto added a commit to crisbeto/material2 that referenced this pull request Jan 16, 2017
…r element is on screen

Fixes some faulty logic, introduced in angular#2102, that meant that the overlay considers the viewport's scroll offset when determining how much it overflows on either side.

Fixes angular#2658.
mmalerba pushed a commit that referenced this pull request Jan 18, 2017
…r element is on screen (#2677)

Fixes some faulty logic, introduced in #2102, that meant that the overlay considers the viewport's scroll offset when determining how much it overflows on either side.

Fixes #2658.
kara pushed a commit to kara/material2 that referenced this pull request Jan 20, 2017
…r element is on screen (angular#2677)

Fixes some faulty logic, introduced in angular#2102, that meant that the overlay considers the viewport's scroll offset when determining how much it overflows on either side.

Fixes angular#2658.
@angular-automatic-lock-bot
Copy link

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-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ConnectedPositionStrategy should apply the "closest" fallback position
3 participants