New issue
Advanced search Search tips

Issue 864990 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 23
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Bug
Q2



Sign in to add a comment

RxR leading edge completion icon and completion type icon should match

Project Member Reported by pschaffner@chromium.org, Jul 18

Issue description

In the omnibox, the leading edge completion icons are currently the old ones, and are not in sync with the new completion type icons shown in the completion view.

You can use the same assets and tint colors provided here https://bugs.chromium.org/p/chromium/issues/detail?id=821817#c22.
 
IMG_0076.PNG
418 KB View Download
Summary: RxR leading edge completion icon and completion type icon should match (was: Leading edge completion icon and completion type icon should match)
Labels: Proj-UIRefresh
Cc: -stkhapugin@chromium.org pschaffner@chromium.org martijnb@chromium.org
Labels: Q2 M-69 MS-Omnibox
Owner: stkhapugin@chromium.org
Status: Ass (was: Available)
Status: Assigned (was: ass)
Status: Star (was: Assigned)
Status: Started (was: Star)
Project Member

Comment 7 by bugdroid1@chromium.org, Jul 19

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

commit ba3188821a3d6050924de5bec497c73bbfb0327c
Author: stkhapugin@chromium.org <stkhapugin@chromium.org>
Date: Thu Jul 19 16:19:12 2018

Use UI Refresh icons for suggestions in the omnibox.

Uses new completion icons in the omnibox.

Bug:  864990 
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: Ic062601ec2b0eb665c8e80779ffc04d90bbb5723
Reviewed-on: https://chromium-review.googlesource.com/1143274
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576516}
[modify] https://crrev.com/ba3188821a3d6050924de5bec497c73bbfb0327c/ios/chrome/browser/ui/omnibox/omnibox_mediator.mm

Status: Fixed (was: Started)
Labels: UIRefresh-NoMerge
Status: Verified (was: Fixed)
Verified in 69.0.3497.58 Beta in iPad Air(iOS 12 beta9), iPhone 2018(iOS 11.4.1)

Omnibox now displays the new completion icons

Link to screenshot:
https://drive.google.com/file/d/1nhTInnGkfs-qrDFWO2M-nbrD_dE5Gjk5/view?usp=sharing

Correction in comment#10 incorrect device name mentioned below are the correct devices
iPad Air(iOS 12 beta9), iPad 2018(iOS 11.4.1)

Sign in to add a comment