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

Issue 841594 link

Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 10
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 1
Type: Bug

Blocking:
issue 838733



Sign in to add a comment

[New Answer Layout] add icons for answer types

Project Member Reported by dschuyler@chromium.org, May 9 2018

Issue description

In the New Answer Layout we want new icons for answer types such as:
- definition
- time
- sunrise
- sunset
- stock
- calendar
- currency
- calculator
- translate
- food/dining


These icons are already provided. They are large sets of icons (rather than the one-offs listed above):
- weather (there are a few hundred of them)
- sports teams (I expect there a few hundred of these as well) 

 
(relates to email with subject "Rich Answer icons/images").
Blocking: 838733
Labels: M-69
Description: Show this description
Cc: bklmn@chromium.org
Dave, for the rich answers that we don't have icons yet for, but that we plan to be the blue circle with white logo, maybe you can just throw a blue circle in so that we can see what it will look like?
I keep forgetting to bring this up, but we need something for knowledge answers (e.g. [how old is dwayne johnson]), also. Or maybe just the Google icon?

Comment 6 by bklmn@chromium.org, Jun 6 2018

I think we should have a generic mono chrome "G" as a default/fallback where we don't have defined icons for. Ill make sure I add in my new asset drop. Redrawing 16px @1x specific icons for clarity. 

Comment 7 by bklmn@chromium.org, Jun 8 2018

New svg assets here...I did a custom draw of the @1x/16px icons and set of @2x/32px:
https://drive.google.com/drive/folders/18FUVv56qBE8SvoaBPvVDVB0yoOP272i-?usp=sharing
Project Member

Comment 8 by bugdroid1@chromium.org, Jun 9 2018

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

commit 4da2d553b5a4006498986991c928c60ce867fe21
Author: Dave Schuyler <dschuyler@chromium.org>
Date: Sat Jun 09 02:42:13 2018

[Omnibox] New answer layout images/icons

This CL adds vector icons for some answers in suggest. These images are
not final. They are used behind a feature flag. The intention is to get
Canary review by UX/PM before finalizing the images.

Bug:  841594 
Change-Id: I52fe73ec904bfed9a26def2351626ab3e35c8407
Reviewed-on: https://chromium-review.googlesource.com/1087757
Commit-Queue: Dave Schuyler <dschuyler@chromium.org>
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565827}
[modify] https://crrev.com/4da2d553b5a4006498986991c928c60ce867fe21/chrome/browser/ui/views/omnibox/omnibox_match_cell_view.cc
[modify] https://crrev.com/4da2d553b5a4006498986991c928c60ce867fe21/components/omnibox/browser/BUILD.gn
[modify] https://crrev.com/4da2d553b5a4006498986991c928c60ce867fe21/components/omnibox/browser/suggestion_answer.h
[add] https://crrev.com/4da2d553b5a4006498986991c928c60ce867fe21/components/omnibox/browser/vector_icons/answer_currency.icon
[add] https://crrev.com/4da2d553b5a4006498986991c928c60ce867fe21/components/omnibox/browser/vector_icons/answer_dictionary.icon
[add] https://crrev.com/4da2d553b5a4006498986991c928c60ce867fe21/components/omnibox/browser/vector_icons/answer_finance.icon
[add] https://crrev.com/4da2d553b5a4006498986991c928c60ce867fe21/components/omnibox/browser/vector_icons/answer_sunrise.icon
[add] https://crrev.com/4da2d553b5a4006498986991c928c60ce867fe21/components/omnibox/browser/vector_icons/answer_when_is.icon

icons were added for 
- definition (dictionary)
- sunrise/sunset (sunrise)
- stock (finance)
- calendar (when_is)
- currency

These icons should be handled (because we currently get results/suggestions for them):
- calculator
- translate


But these don't seem to generate results/suggestions (so is there a current need for icons):
- time
- food/dining

Status: Started (was: Assigned)
#7, the new icons are in https://chromium-review.googlesource.com/c/chromium/src/+/1102105
Updating this comment to include #5 and #6.
Each row has the name we've been using (the icon name in parenthesis) "a quoted search string example".


icons were added for:
[These are done]
- definition      (dictionary) "define set point"
- sunrise/sunset  (sunrise)    "sunrise in los angeles"
- stock           (finance)    "abc stock price"
- calendar        (when_is)    "when is <holiday>"
- currency        (currency)   "33 dollars in British pounds"


These icons should be handled (because we currently get results/suggestions for them):
[These might get a generic "G" icon, they are not done, we need icons]
- calculator            (?) "3+5"
- knowledge answers     (?) "how old is <person>"
- translate             (?) "hello in Japanese"
- generic monochrome G  (?)


But these don't seem to generate results/suggestions (so is there a current need for icons):
[These won't be in m69 (because there are no related suggest results)]
- time         (?) "<unknown>"
- food/dining  (?) "<unknown>"

Labels: Needs-Feedback
Tested this issue on Windows 10, Mac OS 10.13.5 and Ubuntu 14.04 on the latest M-69 69.0.3464.0 build by following the below steps.

1. Launched Chrome and enabled the flag #omnibox-new-answer-layout.
2. Entered the search strings mentioned in comment #12 and answers are shown in the suggestion list, but icons are missing for few search strings.
Attached is the screen cast for reference.

Note: Couldn't get the icon with answer type for "33 dollars in British pounds".

dschuyler@ Request you to please check and confirm if anything is missed from our end in verifying the issue?

Thanks..
841594-M69.mp4
4.0 MB View Download

Comment 14 by bklmn@chromium.org, Jun 18 2018

This is great. Testing this out today. 


Realizing now these missing assets below never made it into my export (obviously). Im exporting now and will get to you today. 

- calculator           
- knowledge answers     
- translate             
- generic monochrome G
FYI, regarding #15. It looks like those icons won't be used for now.

- "money" conversions have the type "finance" (e.g. "33 dollars in British pounds")
- "weight or volume" conversions have the type calculator (e.g. "4.5 oz in cups")
- "time" conversions are not (afaik) active on desktop

To use them (in the future) the back-end (gws and/or complete server) will need to send the client discernible responses.
Updating this comment to include #5 and #6.
Each row has the name we've been using (the icon name in parenthesis) "a quoted search string example".


icons were added for:
[These are done]
- definition      (dictionary) "define set point"
- sunrise/sunset  (sunrise)    "sunrise in los angeles"
- stock           (finance)    "abc stock price"
- calendar        (when_is)    "when is <holiday>"
- currency        (currency)   "33 dollars in British pounds"
- knowledge answers     (mono-G) "how old is <person>"
- translate             (translation) "hello in Japanese"
- generic monochrome G  (mono-G)


These icons should be handled (because we currently get results/suggestions for them):
- calculator            (?) "3+5" There's a CL at https://chromium-review.googlesource.com/c/chromium/src/+/1111460


But these don't seem to generate results/suggestions (so is there a current need for icons):
[These won't be in m69 (because there are no related suggest results)]
- conversions  (?) "<unknown>" these appear to be blended with other types (currency/calculator)
- time         (?) "<unknown>"
- food/dining  (?) "<unknown>"


When the CL at 1111460 lands, I plan to mark this fixed.
Cc: dschuyler@chromium.org
Owner: jdonnelly@chromium.org
I'm going to take this over because, while all the answer types have an icon now, there's still an issue with the sizing of the icons that bklmn and I are going to iterate on.
Owner: orinj@chromium.org
I just noticed that orinj did some similar work on icon sizing in https://crrev.com/c/1112643.
Labels: Proj-MdRefresh
Project Member

Comment 21 by bugdroid1@chromium.org, Jul 10

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

commit 544d7d11d782b053a2978b8a1cc89bbdc6a3144d
Author: Orin Jaworski <orinj@chromium.org>
Date: Tue Jul 10 19:38:05 2018

[omnibox] Change render size and technique for answer icons

Answer icons were being rendered using an odd/even technique
with a circle of diameter 24, but the internal icons were not
properly sized.  Now the icons are kept intact with the original
design size of 16, and the circle is rendered manually.

Bug:  841594 
Change-Id: I5d31e43c4c0bdb70f6fe7dcbb85228a268d73571
Reviewed-on: https://chromium-review.googlesource.com/1130350
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Orin Jaworski <orinj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573860}
[modify] https://crrev.com/544d7d11d782b053a2978b8a1cc89bbdc6a3144d/chrome/browser/ui/views/omnibox/omnibox_match_cell_view.cc
[modify] https://crrev.com/544d7d11d782b053a2978b8a1cc89bbdc6a3144d/components/omnibox/browser/vector_icons/answer_currency.icon
[modify] https://crrev.com/544d7d11d782b053a2978b8a1cc89bbdc6a3144d/components/omnibox/browser/vector_icons/answer_default.icon
[modify] https://crrev.com/544d7d11d782b053a2978b8a1cc89bbdc6a3144d/components/omnibox/browser/vector_icons/answer_dictionary.icon
[modify] https://crrev.com/544d7d11d782b053a2978b8a1cc89bbdc6a3144d/components/omnibox/browser/vector_icons/answer_finance.icon
[modify] https://crrev.com/544d7d11d782b053a2978b8a1cc89bbdc6a3144d/components/omnibox/browser/vector_icons/answer_sunrise.icon
[modify] https://crrev.com/544d7d11d782b053a2978b8a1cc89bbdc6a3144d/components/omnibox/browser/vector_icons/answer_when_is.icon

Is there more to do here or can we mark this Fixed?
Status: Fixed (was: Started)
This should be it.  If there's anything else, bklmn@ will let me know and we can re-open.
Tested this issue on Windows 10, Mac OS 10.13.3 and Ubuntu 14.04 on the latest M-69 69.0.3488.0 build by following the below steps.

1. Launched Chrome and enabled the flag #omnibox-new-answer-layout.
2. Entered the search strings mentioned in comment #17 and answers are shown in the suggestion list along with the icons on Windows and Ubuntu 14.04. But the icons are not seen on Mac OS 10.13.3.
Attached is the screen casts of Windows/Linux and Mac OS behaviors.

orinj@ Request you to please check and confirm if anything is missed from our end in verifying the issue?

Thanks..
841594-Mac.mp4
2.4 MB View Download
841594-Windows.mp4
4.8 MB View Download
Yes, what seems to be missing is the same as in https://bugs.chromium.org/p/chromium/issues/detail?id=855723#c11  --  I don't know why your Mac version appears and behaves so differently from Windows.  Shouldn't the tabs look rounded?  Is Refresh turned on?  Try enabling the upcoming UI features flag: it affects several other flags.
I just got a hint from today's news: https://www.engadget.com/2018/07/11/googles-refreshed-chrome-design-is-nearly-here/

It says the new Material Design is enabled by default on Windows canary but is still behind the flag on Mac.  So try enabling chrome://flags/#upcoming-ui-features.

Sign in to add a comment