Skip to content

Support sliders with CSS scale()#293

Merged
paranoidjk merged 2 commits intoreact-component:masterfrom
captbaritone:bouding-rect
Jul 3, 2017
Merged

Support sliders with CSS scale()#293
paranoidjk merged 2 commits intoreact-component:masterfrom
captbaritone:bouding-rect

Conversation

@captbaritone
Copy link
Copy Markdown
Contributor

Fixes #279. Includes #289 with a small refactor and also resolves the failing tests.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-4.05%) to 45.27% when pulling 6b684dc on captbaritone:bouding-rect into 89353ad on react-component:master.


// TODO: should update the following test cases for it should test API instead implementation
it('should set `dragOffset` to correct value when the left handle is clicked off-center', () => {
fit('should set `dragOffset` to correct value when the left handle is clicked off-center', () => {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

fit => it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Doh. Fixed.

Fix mocking of `getBoundingClientRect`.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-4.05%) to 45.27% when pulling 2be1869 on captbaritone:bouding-rect into 89353ad on react-component:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 49.324% when pulling 2be1869 on captbaritone:bouding-rect into 89353ad on react-component:master.

@paranoidjk paranoidjk merged commit a499d74 into react-component:master Jul 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wrong handle position when scaled

5 participants