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

Issue 629763 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug



Sign in to add a comment

transform and focus doesn't work correctly together

Reported by edhopr...@gmail.com, Jul 20 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:47.0) Gecko/20100101 Firefox/47.0

Example URL:
http://codepen.io/nanaya/pen/JKpaGr

Steps to reproduce the problem:
1. Open the page
2. Click "start"

What is the expected behavior?
Red box, containing green box, containing text box

What went wrong?
Everything

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? N/A 

Does this work in other browsers? No Firefox, Edge, Internet Explorer, Safari

Chrome version: 54.0.2800.0 (Official Build) canary (64-bit)  Channel: canary
OS Version: 10.0
Flash Version: Shockwave Flash 22.0 r0

Broken in different way on other browsers. Disabling animation in chrome breaks it in other way. Only disabling focus callback helps.
 

Comment 1 by edhopr...@gmail.com, Jul 20 2016

disabling `overflow: hidden` also works but that defeats the point of the design.
Components: -Blink Blink>Animation
The focus call is likely doing a scroll into view or something like that.

But I'm deferring this to the animation team to investigate.

Comment 3 by flackr@chromium.org, Jul 20 2016

Cc: skobes@chromium.org
Re #2, yes the focus call is scrolling into view. If you observe the scrollLeft of the .boxes element it changes after the call to focus. I think this is the expected behavior at least for overflow: auto|scroll, though it seems dangerous to do this for an overflow: hidden element where the developer may not be expecting it to happen.

Comment 4 by suzyh@chromium.org, Jul 21 2016

Labels: Needs-Bisect
Status: Untriaged (was: Unconfirmed)

Comment 5 by ajha@chromium.org, Jul 21 2016

Cc: chrishtr@chromium.org ajha@chromium.org
Labels: -Pri-2 -Needs-Bisect M-54 hasbisect OS-Linux OS-Mac Pri-1
Owner: trchen@chromium.org
Status: Assigned (was: Untriaged)
Able to reproduce this on the latest stable(52.0.2743.82) and the latest canary(54.0.2803.0) on Windows-7, Mac OS 10.11.5 and Linux Ubuntu 14.04.

This is regressed in M-36.

Last good build: 36.0.1965.0
First bad build: 36.0.1967.0

Changelog: https://chromium.googlesource.com/chromium/src/+log/946cdb61e3aa5f6b7c1c8b4f143ca9099e50ccf9..755bd92cb16a37d39cf9b31e7dd5ed7df916e97a

There is blink changelog in the above range which is opening a blank page:https://build.chromium.org/f/chromium/perf/dashboard/ui/changelog_blink.html?url=/trunk&range=172819:172885&mode=html

From the blink changelog b/w 36.0.1965.0 and 36.0.1967.0: https://chromium.googlesource.com/chromium/blink/+log/c0ae1b653a201f4fba421dee3dad7eaf7e1874c2..c1639d903abb315a663dcb2734b67f61f1b0567e?pretty=fuller&n=10000

Suspecting: https://codereview.chromium.org/250773002 (trchen@) or https://codereview.chromium.org/260113004 (chrishtr@)

Could anyone of you please take a look and help in investigating this further.

Thank you! 

Comment 6 by suzyh@chromium.org, Jul 22 2016

Cc: suzyh@chromium.org
Labels: Update-Monthly

Comment 7 by suzyh@chromium.org, Aug 10 2016

Cc: trchen@chromium.org
Components: -Blink>Animation Blink>Focus
Labels: -Update-Monthly
Owner: ----
Status: Untriaged (was: Assigned)
transition and transform were unprefixed in M36; prior to that those fields would have been ignored. Not sure why the unprefixing didn't show up in the bisect range, however.

Updated example with prefixed versions included: http://codepen.io/anon/pen/yJGyJo

I attempted a bisect with:
$ python tools/bisect-builds.py -a linux64 -g 116827 -b 267191 --verify-range --use-local-cache -- --no-sandbox
and if I have done this correctly I think this is a non-regression issue.

While transitions are involved, I don't think this is an issue in the animations code. Bumping over to Blink>Focus, but could also be layout? I'll stay cc'd on the bug in case you need input from the animations team.

Comment 8 by kochi@chromium.org, Sep 2 2016

Components: Blink>Layout
(as a Blink>Focus owner) I'm not exactly sure what is going wrong in the example,
as I'm not familiar with animation - according to comment#3 by flackr@
scrollIntoView() is working as expected?

Comment 9 by e...@chromium.org, Sep 8 2016

Status: WontFix (was: Untriaged)
This is working as intended in that focus calls scrollIntoView. Overflow: hidden is used quite a bit to implement custom scroll behavior, we cannot break that.
Components: Blink>HTML>Focus
Components: -Blink>Focus

Sign in to add a comment