Commit d034974e authored by Alex Keng's avatar Alex Keng Committed by Commit Bot

Range: update the rendering of the slider ticks

This change updates the position of the slider ticks to be below the
slider (overlapping the thumb by 1 pixel) and shortens the ticks by 2 pixels
(from 6 to 4)

Bug: 1014373
Change-Id: I498e8f76259f677378f6ec5af4780e74653c8a12
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1860513
Commit-Queue: Alex Keng <shihken@microsoft.com>
Reviewed-by: default avatarKent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#706124}
parent 0b556b9c
...@@ -182,13 +182,19 @@ Color LayoutThemeDefault::PlatformInactiveSelectionForegroundColor( ...@@ -182,13 +182,19 @@ Color LayoutThemeDefault::PlatformInactiveSelectionForegroundColor(
IntSize LayoutThemeDefault::SliderTickSize() const { IntSize LayoutThemeDefault::SliderTickSize() const {
if (UseMockTheme()) if (UseMockTheme())
return IntSize(1, 3); return IntSize(1, 3);
return IntSize(1, 6); if (RuntimeEnabledFeatures::FormControlsRefreshEnabled())
return IntSize(1, 4);
else
return IntSize(1, 6);
} }
int LayoutThemeDefault::SliderTickOffsetFromTrackCenter() const { int LayoutThemeDefault::SliderTickOffsetFromTrackCenter() const {
if (UseMockTheme()) if (UseMockTheme())
return 11; return 11;
return -16; if (RuntimeEnabledFeatures::FormControlsRefreshEnabled())
return 7;
else
return -16;
} }
void LayoutThemeDefault::AdjustSliderThumbSize(ComputedStyle& style) const { void LayoutThemeDefault::AdjustSliderThumbSize(ComputedStyle& style) const {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment