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

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

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

@ErinCoughlanErinCoughlan 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

@crisbetocrisbetoforce-pushed the 9604/panel-units branch 2 times, most recently from 628ec9f to ae7a0d3CompareSeptember 15, 2016 05:26
@crisbetocrisbeto 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
@crisbetocrisbeto 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
@crisbetocrisbeto mentioned this pull request Sep 22, 2016
3 tasks
@crisbetocrisbeto 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
@crisbetocrisbeto 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
MemberAuthor

@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. Fixesangular#9604.
@ThomasBurlesonThomasBurleson modified the milestones: 1.1.3, 1.2.0Jan 1, 2017
@SplaktarSplaktar removed the needs: review This PR is waiting on review from the team label Feb 7, 2018
Copy link
Contributor

@SplaktarSplaktar left a comment

Choose a reason for hiding this comment

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

LGTM

@SplaktarSplaktar modified the milestones: Future, 1.1.7Feb 7, 2018
@SplaktarSplaktar 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
@SplaktarSplaktar added the pr: lgtm This PR has been approved by the reviewer label Feb 7, 2018
@SplaktarSplaktar modified the milestones: 1.1.7, 1.1.8Feb 8, 2018
@mmalerbammalerba merged commit 0d276f3 into angular:masterFeb 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. Fixesangular#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 freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

P3: importantImportant issues that really should be fixed when possible.pr: lgtmThis PR has been approved by the reviewerpr: merge readyThis PR is ready for a caretaker to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@crisbeto@ErinCoughlan@Splaktar@ThomasBurleson@mmalerba