Minimize tab heights for non-touch |
||||
Issue descriptionThere are considerations of minimizing today's non-touch tab height from 36 to either 34 or 32. Not applicable to touchable refresh Mocks https://gallery.googleplex.com/projects/MCHbtQVoQ2HCZW9TjXpqhOa9/files/MCGRcS8QUWs-6Tz-0FXOylDf
,
Jul 12
,
Jul 12
If this is cheap, then let's go ahead and try 34 and try it out. If it is undesirable, we'll go ahead and revert the change.
,
Jul 13
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0670e881945cb0c5f3c49ab55f7f96007472c0e3 commit 0670e881945cb0c5f3c49ab55f7f96007472c0e3 Author: Allen Bauer <kylixrd@chromium.org> Date: Fri Jul 13 13:03:39 2018 [Refresh] Reduce the height of the tabs from 36 to 34. Inactive tab hit-testing overlap regions is now calculated to be 1 dip above the top of the tab separator. Bug: 859204 Change-Id: Idbd92f9209ff2bbcd5516f26d1acfbd0b83ae563 Reviewed-on: https://chromium-review.googlesource.com/1135726 Reviewed-by: Peter Kasting <pkasting@chromium.org> Reviewed-by: Scott Violet <sky@chromium.org> Commit-Queue: Allen Bauer <kylixrd@chromium.org> Cr-Commit-Position: refs/heads/master@{#574889} [modify] https://crrev.com/0670e881945cb0c5f3c49ab55f7f96007472c0e3/ash/public/cpp/ash_layout_constants.cc [modify] https://crrev.com/0670e881945cb0c5f3c49ab55f7f96007472c0e3/chrome/browser/ui/layout_constants.cc [modify] https://crrev.com/0670e881945cb0c5f3c49ab55f7f96007472c0e3/chrome/browser/ui/views/tabs/tab_strip.cc
,
Jul 13
|
||||
►
Sign in to add a comment |
||||
Comment 1 by pkasting@chromium.org
, Jun 29 2018