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

Issue 859543 link

Starred by 3 users

Issue metadata

Status: Fixed
Merged: issue 859540
Owner:
Closed: Aug 6
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Refresh: omnibox text moves when you trigger zero suggest

Project Member Reported by bettes@chromium.org, Jul 2

Issue description

Chrome Version:  69.0.3478.0
OS: MacOS

What steps will reproduce the problem?
(1)airbnb.com, chase.com, any EV cert site
(2)focus omnibox
(3)see airbnb.com jog to the right without typing

What is the expected result?
URL should not move upon focus


 
Jul-01-2018 22-38-24.gif
2.5 MB View Download
Status: Available (was: Untriaged)
Mergedinto: 859540
Status: Duplicate (was: Available)
The issue here is not actually the EV cert chip, it's contextual zero suggest.
Blocking: -852828
Labels: -Pri-1 OS-Chrome OS-Linux OS-Windows Pri-2
Status: Started (was: Duplicate)
Even if the text should move on input ( issue 859540 ), it shouldn't move in this case (zero suggest). The presence of text in the location icon button means that the text in the omnibox and the text in the suggestions won't line up regardless, so adding additional indent just moves the text for no apparent reason.
Summary: Refresh: omnibox text moves when you trigger zero suggest (was: Refresh: omnibox text moves when you focus (EV Cert only))
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 3

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

commit 6310a574660cea1442fd5a932d9963ecec12c321
Author: Justin Donnelly <jdonnelly@chromium.org>
Date: Fri Aug 03 16:46:55 2018

[omnibox] Don't indent the text if the location icon text is showing.

The point of the text indent is to align with the suggestions in the
dropdown. But if the location icon has text, it won't align anyway so
the indent is not useful and looks odd.

Bug:  859543 

Change-Id: I355c81016a2691bcc237b108eec80318dccf1b50
Reviewed-on: https://chromium-review.googlesource.com/1159229
Reviewed-by: Tommy Li <tommycli@chromium.org>
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Justin Donnelly <jdonnelly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580571}
[modify] https://crrev.com/6310a574660cea1442fd5a932d9963ecec12c321/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
[modify] https://crrev.com/6310a574660cea1442fd5a932d9963ecec12c321/chrome/browser/ui/views/location_bar/icon_label_bubble_view.h
[modify] https://crrev.com/6310a574660cea1442fd5a932d9963ecec12c321/chrome/browser/ui/views/location_bar/location_bar_view.cc

Cc: -tommycli@chromium.org jdonnelly@chromium.org
Owner: tommycli@chromium.org
tommycli: would you mind verifying this and requesting merge on Monday?
Labels: Needs-Feedback
Unable to reproduce the issue on reported chrome version 69.0.3478.0 using Windows, Mac 10.13.5 and Ubuntu 17.10.Attaching screen-cast for reference.
Steps:
---------
1. Launched reported Chrome 
2. Navigated to given URL's in the comment #0 as per attached screen-cast
3. Focused on omnibox by clicking tab
We have Observed that "URL not moved upon focus"

@Reporter: Please find the attached screen-cast and let us know if we have missed anything in the process.

Thanks!
859543.mp4
4.2 MB View Download
Re c#7, try it again with chrome://flags/#upcoming-ui-features enabled and nothing else. Make sure you are signed in and syncing as well.
Labels: Merge-Request-69
I just tested it. I can verify that:
 - The original bug reproduces on: 3510.2
 - It's fixed on:                  3514.0 (Today's Canary)
Project Member

Comment 10 by sheriffbot@chromium.org, Aug 6

Labels: -Merge-Request-69 Merge-Review-69 Hotlist-Merge-Review
This bug requires manual review: M69 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), kariahda@(iOS), cindyb@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Re: Merge Review

This is a user visible change with low technical risk.

It's not as critical as a crash, but it has a good benefit to risk ratio, so it's merge-worthy in my opinion.
Labels: -Merge-Review-69 Merge-Approved-69
Approving merge to M69 branch 3497 based on comment #9 and #11. Please merge. Thank you.
Project Member

Comment 13 by bugdroid1@chromium.org, Aug 6

Labels: -merge-approved-69 merge-merged-3497
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8478d21fe29ce1e19b198b38c0f976c158ff1401

commit 8478d21fe29ce1e19b198b38c0f976c158ff1401
Author: Justin Donnelly <jdonnelly@chromium.org>
Date: Mon Aug 06 19:22:35 2018

[omnibox] Don't indent the text if the location icon text is showing.

The point of the text indent is to align with the suggestions in the
dropdown. But if the location icon has text, it won't align anyway so
the indent is not useful and looks odd.

Bug:  859543 

Change-Id: I355c81016a2691bcc237b108eec80318dccf1b50
Reviewed-on: https://chromium-review.googlesource.com/1159229
Reviewed-by: Tommy Li <tommycli@chromium.org>
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Justin Donnelly <jdonnelly@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#580571}(cherry picked from commit 6310a574660cea1442fd5a932d9963ecec12c321)
Reviewed-on: https://chromium-review.googlesource.com/1163943
Cr-Commit-Position: refs/branch-heads/3497@{#429}
Cr-Branched-From: 271eaf50594eb818c9295dc78d364aea18c82ea8-refs/heads/master@{#576753}
[modify] https://crrev.com/8478d21fe29ce1e19b198b38c0f976c158ff1401/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
[modify] https://crrev.com/8478d21fe29ce1e19b198b38c0f976c158ff1401/chrome/browser/ui/views/location_bar/icon_label_bubble_view.h
[modify] https://crrev.com/8478d21fe29ce1e19b198b38c0f976c158ff1401/chrome/browser/ui/views/location_bar/location_bar_view.cc

Status: Fixed (was: Started)
Merged to 69. Thanks all!
Cc: ligim...@chromium.org
Test team was not able to repro, hence cannot verify the fix. bettes@ would you mind verifying?
Re c#7, try it again with chrome://flags/#upcoming-ui-features enabled and nothing else. Make sure you are signed in and syncing as well.

Labels: TE-Verified-M70 TE-Verified-70.0.3514.0
Able to reproduce the issue on Mac 10.13.5, Windows 10 and Ubuntu 17.10 on reported version 69.0.3478.0.

Verified the fix on Mac 10.13.5, Windows 10 and Ubuntu 17.10, as per comment#16 on latest chrome version #70.0.3515.0 and 70.0.3514.0.
Attaching screen shot for reference.
Observed that the URL is not moving upon focus
Hence, the fix is working as expected.
Adding the verified labels.

Thanks...!
859543(1).mp4
10.0 MB View Download
Labels: -TE-Verified-M70 -TE-Verified-70.0.3514.0 TE-Verified-69.0.3497.32 TE-Verified-M69
Able to reproduce the issue on Mac 10.13.5, Windows 10 and Ubuntu 17.10 on reported version 69.0.3478.0.

Verified the fix on Mac 10.13.5, Windows 10 and Ubuntu 17.10,on chrome version #69.0.3497.32.
Attaching screen shot for reference.
Observed that the URL is not moving upon focus
Hence, the fix is working as expected.
Adding the verified labels.

Thanks...!
859543(2).mp4
7.1 MB View Download
Cc: abdulsyed@chromium.org
+abdulsyed@ fyi, M69 merges taken for Proj-MdRefresh .

Sign in to add a comment