Skip to content

fix(range): realign input range thumb margin on Webkit#1632

Merged
julien-deramond merged 2 commits into
mainfrom
main-jd-fix-input-range-thumb-margin-on-chrome
Nov 17, 2022
Merged

fix(range): realign input range thumb margin on Webkit#1632
julien-deramond merged 2 commits into
mainfrom
main-jd-fix-input-range-thumb-margin-on-chrome

Conversation

@julien-deramond

@julien-deramond julien-deramond commented Nov 16, 2022

Copy link
Copy Markdown
Contributor

Related issues

Fixes #1606

Description

Change the calculation to position the input range thumb on Webkit.

TBH I haven't checked on all browsers. Dear reviewer, could you help me doing that 🙏

Types of change

  • Bug fix (non-breaking which fixes an issues)

Live previews

Checklist

  • I have read the contributing guidelines
  • My code follows the code style of the project (using npm run lint)
  • All new and existing tests passed

@netlify

netlify Bot commented Nov 16, 2022

Copy link
Copy Markdown

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit affc25a
🔍 Latest deploy log https://app.netlify.com/sites/boosted/deploys/6375f79fe25cc9000824e28d
😎 Deploy Preview https://deploy-preview-1632--boosted.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@louismaximepiton

This comment was marked as resolved.

@julien-deramond

This comment was marked as resolved.

Comment thread scss/forms/_form-range.scss Outdated
louismaximepiton added a commit that referenced this pull request Nov 17, 2022
Co-authored-by: Louis-Maxime Piton <louismaxime.piton@orange.com>
@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@louismaximepiton louismaximepiton left a comment

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.

If you're fine with this implementation, let's go with this!

@julien-deramond julien-deramond merged commit 46b6734 into main Nov 17, 2022
@julien-deramond julien-deramond deleted the main-jd-fix-input-range-thumb-margin-on-chrome branch November 17, 2022 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs > Forms > range: The round to be grab is not centered vertically (Chrome only)

2 participants