New issue
Advanced search Search tips

Issue 876364 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

[Autofill Views] Replace 24px with a more semantic constant

Project Member Reported by tmartino@chromium.org, Aug 21

Issue description

In fixing crbug.com/876291 we introduced a 24px constant to ensure the primary and secondary text does not run together.

We should replace this with something fetched from menu config, layout provider, etc.

This could be as simple as just adding a new constant (something like "horizontal distance between primary and secondary labels") since I don't see 24px (or anything really in the neighborhood) in use elsewhere. Another, jankier approach would be to declare that this spacer is three times the horizontal padding used elsewhere in the row (3 * 8 = 24).
 
Status: Assigned (was: Untriaged)
This issue has an owner, a component and a priority, but is still listed as untriaged or unconfirmed. By definition, this bug is triaged. Changing status to "assigned". Please reach out to me if you disagree with how I've done this.

Sign in to add a comment