box-shadow behaviour on <td> elements while using overflow: auto
Reported by
lukas.po...@googlemail.com,
Sep 8 2017
|
|||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.79 Safari/537.36 Steps to reproduce the problem: Run: https://jsfiddle.net/1q98o385/5/ in "Version 61.0.3163.79 (Official Build) (64-Bit)" What is the expected behavior? No scrollbars while hovering. What went wrong? Scrollbars are shown when right or bottom table cells are getting hovered. The width of the specified box-shadow seems to be taken into account. Did this work before? Yes Not exactly sure. It worked until this update. Does this work in other browsers? Yes Chrome version: 61.0.3163.79 Channel: stable OS Version: 10.0 Flash Version: Shockwave Flash 26.0 r0 https://jsfiddle.net/1q98o385/5/
,
Sep 8 2017
Able to reproduce the issue on Windows 7,10 and the same wasn't reproducible on Mac. Please find bisect result below : You are probably looking for a change made after 476218 (known good), but no lat er than 476219 (first known bad). CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/3ac5e57d366931ea57244c45b01d55c01774fdfb..ed6724976ba56938ae7be863b63758456e0b2387
,
Sep 8 2017
Will fix in M-62.
,
Sep 10 2017
,
Sep 12 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b5eb082d971e76308d9f95167db1bb67222a28e7 commit b5eb082d971e76308d9f95167db1bb67222a28e7 Author: Xianzhu Wang <wangxianzhu@chromium.org> Date: Tue Sep 12 06:18:43 2017 Fix table layout overflow that included all contents visual overflows crrev.com/c/515606 overly simplified table layout overflow which is expected to cover all collapsed border overflows. The CL added all visual overflow into layout overflow causing it to also unexpectedly cover all contents overflows (e.g. shadow and outlines of descendants). Fix the issue by just include collapsed border overflow. Bug: 763337 Change-Id: I6c53afba3f00729a483588dfea46c86a8b78562a Reviewed-on: https://chromium-review.googlesource.com/657815 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by: Morten Stenshorne <mstensho@opera.com> Cr-Commit-Position: refs/heads/master@{#501201} [modify] https://crrev.com/b5eb082d971e76308d9f95167db1bb67222a28e7/third_party/WebKit/LayoutTests/platform/mac-mac10.10/fast/forms/file/file-input-direction-expected.txt [modify] https://crrev.com/b5eb082d971e76308d9f95167db1bb67222a28e7/third_party/WebKit/LayoutTests/platform/mac-mac10.9/fast/forms/file/file-input-direction-expected.txt [modify] https://crrev.com/b5eb082d971e76308d9f95167db1bb67222a28e7/third_party/WebKit/LayoutTests/platform/mac/fast/forms/file/file-input-direction-expected.txt [modify] https://crrev.com/b5eb082d971e76308d9f95167db1bb67222a28e7/third_party/WebKit/Source/core/layout/LayoutTable.cpp [modify] https://crrev.com/b5eb082d971e76308d9f95167db1bb67222a28e7/third_party/WebKit/Source/core/layout/LayoutTable.h [modify] https://crrev.com/b5eb082d971e76308d9f95167db1bb67222a28e7/third_party/WebKit/Source/core/layout/LayoutTableTest.cpp
,
Sep 13 2017
,
Sep 13 2017
,
Sep 14 2017
This bug requires manual review: M62 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), cmasso@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Sep 14 2017
Approving merge to M62. Branch:3202
,
Sep 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4d28d639f2b1e403f10c8352b5ed95e60d37d13e commit 4d28d639f2b1e403f10c8352b5ed95e60d37d13e Author: Xianzhu Wang <wangxianzhu@chromium.org> Date: Thu Sep 14 23:21:41 2017 Fix table layout overflow that included all contents visual overflows crrev.com/c/515606 overly simplified table layout overflow which is expected to cover all collapsed border overflows. The CL added all visual overflow into layout overflow causing it to also unexpectedly cover all contents overflows (e.g. shadow and outlines of descendants). Fix the issue by just include collapsed border overflow. TBR=wangxianzhu@chromium.org (cherry picked from commit b5eb082d971e76308d9f95167db1bb67222a28e7) Bug: 763337 Change-Id: I6c53afba3f00729a483588dfea46c86a8b78562a Reviewed-on: https://chromium-review.googlesource.com/657815 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by: Morten Stenshorne <mstensho@opera.com> Cr-Original-Commit-Position: refs/heads/master@{#501201} Reviewed-on: https://chromium-review.googlesource.com/667744 Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/branch-heads/3202@{#240} Cr-Branched-From: fa6a5d87adff761bc16afc5498c3f5944c1daa68-refs/heads/master@{#499098} [modify] https://crrev.com/4d28d639f2b1e403f10c8352b5ed95e60d37d13e/third_party/WebKit/LayoutTests/platform/mac-mac10.10/fast/forms/file/file-input-direction-expected.txt [modify] https://crrev.com/4d28d639f2b1e403f10c8352b5ed95e60d37d13e/third_party/WebKit/LayoutTests/platform/mac-mac10.9/fast/forms/file/file-input-direction-expected.txt [modify] https://crrev.com/4d28d639f2b1e403f10c8352b5ed95e60d37d13e/third_party/WebKit/LayoutTests/platform/mac/fast/forms/file/file-input-direction-expected.txt [modify] https://crrev.com/4d28d639f2b1e403f10c8352b5ed95e60d37d13e/third_party/WebKit/Source/core/layout/LayoutTable.cpp [modify] https://crrev.com/4d28d639f2b1e403f10c8352b5ed95e60d37d13e/third_party/WebKit/Source/core/layout/LayoutTable.h [modify] https://crrev.com/4d28d639f2b1e403f10c8352b5ed95e60d37d13e/third_party/WebKit/Source/core/layout/LayoutTableTest.cpp
,
Sep 14 2017
,
Sep 22 2017
Issue 767286 has been merged into this issue.
,
Sep 25 2017
,
Sep 27 2017
Issue 768803 has been merged into this issue.
,
Sep 28 2017
Issue 768767 has been merged into this issue.
,
Sep 28 2017
Issue 769791 has been merged into this issue.
,
Sep 28 2017
PMs, given that multiple bugs have been reported on M61, it seems that the bug has some real impact. Do you think we should merge the fix into M61? The fix has been in beta channel for 2 weeks and should have been baked well.
,
Sep 28 2017
This bug requires two merges(change listed at #5 and #6). Will both changes will be fully safe to merge as M61 is already out at 100% stable? Also at the moment there is no plan for M61 Desktop respin. Is this bug itself require stable respin? Or we can pick it for future M61 refresh (if any)?
,
Sep 28 2017
I think both of the merges are safe. It doesn't require a respin. It'll be good if there is a refresh.
,
Sep 28 2017
Issue 769697 has been merged into this issue.
,
Oct 2 2017
Issue 770835 has been merged into this issue.
,
Oct 5 2017
Let's just wait for M-62 stable.
,
Oct 17 2017
|
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 Deleted