Commit f5f169d1 authored by Roman Arora's avatar Roman Arora Committed by Josip Sokcevic

Tab Search: UX review audit related changes

UX audit related changes:
- Search field, list item, and feedback footer height changes
- Search field background color and styling changes
- Component alignment related changes

Dependends on CL:
https://chromium-review.googlesource.com/c/chromium/src/+/2417548

Screenshot:
https://drive.google.com/file/d/1ahWvFnnpFgJOJXgWqDihMZFcoE0KLzf3/view?usp=sharing

Bug: 1128599
Change-Id: I231688d955cbda00c4109dd48ddb5eed77f94165
Reviewed-on: https://chrome-internal-review.googlesource.com/c/chrome/browser/resources/tab_search/+/3276513Reviewed-by: default avatarJohn Lee <johntlee@chromium.org>
Reviewed-by: default avatarTom Lukaszewicz <tluk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#819609}
parent c1400ed1
...@@ -23,7 +23,7 @@ ...@@ -23,7 +23,7 @@
#feedback-footer { #feedback-footer {
border: none; border: none;
border-top: 1px solid var(--google-grey-refresh-500); border-top: 1px solid var(--google-grey-refresh-500);
height: 32px; height: 40px;
width: 100%; width: 100%;
} }
......
<style include="cr-icons"> <style include="cr-icons">
:host { :host {
align-items: center; align-items: center;
background-color: var(--google-grey-refresh-100); background-color: var(--mwb-background-color);
border-radius: 32px;
display: flex; display: flex;
height: 28px; height: var(--mwb-item-height);
margin: 8px; padding: 0 var(--mwb-list-item-horizontal-margin);
padding: 0 8px;
user-select: none; user-select: none;
} }
@media (prefers-color-scheme: dark) {
:host {
background-color: var(--google-grey-800);
}
}
#searchIcon { #searchIcon {
height: var(--mwb-icon-size); height: var(--mwb-icon-size);
padding-inline-end: 4px; padding-inline-end: var(--mwb-list-item-horizontal-margin);
width: var(--mwb-icon-size); width: var(--mwb-icon-size);
} }
#searchInput {
padding: 0;
}
@media (prefers-color-scheme: dark) { @media (prefers-color-scheme: dark) {
#searchIcon { #searchIcon {
color: var(--google-blue-refresh-300); color: var(--google-blue-refresh-300);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment