New issue
Advanced search Search tips

Issue 750171 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Aug 2017
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Highlighter stroke should have rounded corners

Project Member Reported by kaznacheev@chromium.org, Jul 28 2017

Issue description

The current implementation of the highlighter stroke
(in ash::HighlighterView) is a  4x14dp rectangle being
dragged around while keeping the vertical orientation.

This looks nice most of the time, but sharp rectangular
corners are often visible at both ends of the stroke.

The corners should be rounded.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b442dfacc61a91a2fc7ee48ab641c367f2b7b29f

commit b442dfacc61a91a2fc7ee48ab641c367f2b7b29f
Author: Vladislav Kaznacheev <kaznacheev@google.com>
Date: Tue Aug 01 02:37:02 2017

Add rounded corners to highlighter stroke

Draw the outline of each segment with 4dp
stroke and round joints.

Bug:  750171 
Change-Id: I5db66e65ed4d526d890486c7b96c7379453b70b7
Reviewed-on: https://chromium-review.googlesource.com/591610
Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#490838}
[modify] https://crrev.com/b442dfacc61a91a2fc7ee48ab641c367f2b7b29f/ash/highlighter/highlighter_view.cc

Project Member

Comment 2 by bugdroid1@chromium.org, Aug 3 2017

Labels: merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fc46d5f4397a8ba5e52393fb4227507e3032017c

commit fc46d5f4397a8ba5e52393fb4227507e3032017c
Author: Vladislav Kaznacheev <kaznacheev@google.com>
Date: Thu Aug 03 00:58:24 2017

Add rounded corners to highlighter stroke

Draw the outline of each segment with 4dp
stroke and round joints.

TBR=kaznacheev@google.com

(cherry picked from commit b442dfacc61a91a2fc7ee48ab641c367f2b7b29f)

Bug:  750171 
Change-Id: I5db66e65ed4d526d890486c7b96c7379453b70b7
Reviewed-on: https://chromium-review.googlesource.com/591610
Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#490838}
Reviewed-on: https://chromium-review.googlesource.com/599153
Reviewed-by: Vladislav Kaznacheev <kaznacheev@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#259}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/fc46d5f4397a8ba5e52393fb4227507e3032017c/ash/highlighter/highlighter_view.cc

Status: Fixed (was: Assigned)

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment