Issue metadata
Sign in to add a comment
|
new tab button's "pressed" state is missing (or not visible) on samus |
||||||||||||||||||||
Issue descriptionGoogle Chrome 55.0.2878.0 (Official Build) dev (64-bit) Revision 0 Platform 8861.0.0 (Official Build) dev-channel samus see attached vid. not sure if this is md related. assigning to tdanderson@ for triage.
,
Oct 7 2016
no, it's default one.
,
Oct 10 2016
thanks for report, got over-zealous deleting stuff.
,
Oct 10 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/434b901825d725fb21e7a98ab2f27b128c504159 commit 434b901825d725fb21e7a98ab2f27b128c504159 Author: estade <estade@chromium.org> Date: Mon Oct 10 20:40:47 2016 Fix new tab button (broken when removing pre-MD code). BUG= 653905 Review-Url: https://codereview.chromium.org/2401423004 Cr-Commit-Position: refs/heads/master@{#424230} [modify] https://crrev.com/434b901825d725fb21e7a98ab2f27b128c504159/chrome/browser/ui/views/tabs/tab_strip.cc
,
Oct 10 2016
,
Oct 24 2016
oops, guess I need to merge this.
,
Oct 24 2016
,
Oct 24 2016
,
Oct 24 2016
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/15762bd6c5d58d85a75fb4ebcb21be18ed6040ee commit 15762bd6c5d58d85a75fb4ebcb21be18ed6040ee Author: Evan Stade <estade@chromium.org> Date: Mon Oct 24 17:48:42 2016 Fix new tab button (broken when removing pre-MD code). BUG= 653905 Review-Url: https://codereview.chromium.org/2401423004 Cr-Commit-Position: refs/heads/master@{#424230} (cherry picked from commit 434b901825d725fb21e7a98ab2f27b128c504159) Review URL: https://codereview.chromium.org/2447833002 . Cr-Commit-Position: refs/branch-heads/2883@{#241} Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768} [modify] https://crrev.com/15762bd6c5d58d85a75fb4ebcb21be18ed6040ee/chrome/browser/ui/views/tabs/tab_strip.cc
,
Oct 28 2016
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 28 2016
,
Nov 2 2016
Verified on ChromeOS 8872.35.0, 55.0.2883.35
,
Nov 4 2016
[Automated comment] removing mislabelled merge-merged-2840 |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by tdander...@chromium.org
, Oct 7 2016Labels: Proj-MaterialDesign-NativeUI
Owner: est...@chromium.org
Status: Assigned (was: Untriaged)