New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 867193 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 31
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 741083



Sign in to add a comment

Settings: search engine row taller than other rows with large font settings

Project Member Reported by scottchen@chromium.org, Jul 24

Issue description

I noticed with "very large" font, search engine row is taller than other rows. Its s"start" part has larger margins than other row's "start" parts, possibly due to the fact that it's using a <p> tag.

see attached screenshot and code:
https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/search_page/search_page.html?q=search_page.html&sq=package:chromium&dr&l=50
 
Screenshot from 2018-07-24 16-53-08.png
273 KB View Download
Blocking: 741083
Cc: -aee@chromium.org
Owner: aee@chromium.org
Status: Started (was: Available)
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 30

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

commit 95c73fc5717f7a1904981d3f08c4243b2a3030a9
Author: Esmael El-Moslimany <aee@chromium.org>
Date: Mon Jul 30 22:28:09 2018

Settings: change setting label from <p> to <div> to match other labels

Bug:  867193 
Change-Id: I140d9b979ba7f58a3ab8e62142797caddd148f44
Reviewed-on: https://chromium-review.googlesource.com/1154195
Reviewed-by: Scott Chen <scottchen@chromium.org>
Commit-Queue: Esmael El-Moslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579201}
[modify] https://crrev.com/95c73fc5717f7a1904981d3f08c4243b2a3030a9/chrome/browser/resources/settings/search_page/search_page.html

Status: Fixed (was: Started)

Sign in to add a comment