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

fix(panel): allow numbers in offset methods #9609

Merged
merged 1 commit into from
Feb 9, 2018

Conversation

crisbeto
Copy link
Member

Allows for numbers to be pass to the withOffsetX and withOffsetY methods, assuming that the units are pixels.

Fixes #9604.

CC @ErinCoughlan @bradrich

@crisbeto crisbeto added the needs: review This PR is waiting on review from the team label Sep 14, 2016
Copy link
Contributor

@ErinCoughlan ErinCoughlan left a comment

Choose a reason for hiding this comment

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

LGTM


/**
* Adds units to a number value.
* @param {number} value
Copy link
Contributor

Choose a reason for hiding this comment

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

@param {string | number} value

@crisbeto crisbeto force-pushed the 9604/panel-units branch 2 times, most recently from 628ec9f to ae7a0d3 Compare September 15, 2016 05:26
@crisbeto crisbeto added the needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved label Sep 21, 2016
@crisbeto crisbeto removed the needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved label Sep 21, 2016
@crisbeto crisbeto added the needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved label Sep 26, 2016
@crisbeto crisbeto removed the needs: rebase This PR needs to be rebased on the latest commits from master and conflicts need to be resolved label Sep 26, 2016
@crisbeto
Copy link
Member Author

@ThomasBurleson this should be ready for presubmit.

Allows for numbers to be pass to the `withOffsetX` and `withOffsetY` methods, assuming that the units are pixels.

Fixes angular#9604.
@ThomasBurleson ThomasBurleson modified the milestones: 1.1.3, 1.2.0 Jan 1, 2017
@Splaktar Splaktar removed the needs: review This PR is waiting on review from the team label Feb 7, 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

@Splaktar Splaktar assigned Splaktar and unassigned ThomasBurleson Feb 7, 2018
@Splaktar Splaktar modified the milestones: Future, 1.1.7 Feb 7, 2018
@Splaktar Splaktar added P3: important Important issues that really should be fixed when possible. pr: merge ready This PR is ready for a caretaker to review labels Feb 7, 2018
@Splaktar Splaktar added the pr: lgtm This PR has been approved by the reviewer label Feb 7, 2018
@Splaktar Splaktar modified the milestones: 1.1.7, 1.1.8 Feb 8, 2018
@mmalerba mmalerba merged commit 0d276f3 into angular:master Feb 9, 2018
chmelevskij pushed a commit to chmelevskij/material that referenced this pull request Jun 19, 2018
Allows for numbers to be pass to the `withOffsetX` and `withOffsetY` methods, assuming that the units are pixels.

Fixes angular#9604.
Splaktar pushed a commit that referenced this pull request Jul 31, 2018
Allows for numbers to be pass to the `withOffsetX` and `withOffsetY` methods, assuming that the units are pixels.

Fixes #9604.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P3: important Important issues that really should be fixed when possible. pr: lgtm This PR has been approved by the reviewer pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants