Regression: Scrolling node is not correctly set |
|||
Issue description1. Navigate to https://output.jsbin.com/lijuno 2. Enable touch emulation. 3. Long press on yellow div until context menu opens 4. Press esc to cancel context menu. 5. Press down and up arrow keys on keyboard. 6. Long press on red div until context menu opens 7. Press esc to cancel context menu. 8. Press down and up arrow keys on keyboard. What is expected: 5. Yellow rectangle scrolls. 8. Red rectangle scrolls. What happened: 5. Yellow rectangle does not scroll. 8. Red rectangle scrolls. Maybe this particular effect doesn't matter as much. But this is a side effect of https://codereview.chromium.org/2249663002/ which bypasses setting some internal variables in some cases and for example m_clickNode one causes this change in behavior.
,
Aug 30 2016
Yup. I meant red. I realized the tap also has the same bug right now and you just changed the long press to do the same as tap. I don't know how long the tap bug was there but if nobody complains I wonder if it is worth fixing the issue at all just for the sake of keeping the old behavior.
,
Aug 30 2016
,
Dec 7 2017
This doesn't seem to be particularly important as no one else complained about it either. As the old behavior wasn't specced or standard or anything we should be fine with the current behavior. |
|||
►
Sign in to add a comment |
|||
Comment 1 by mustaq@chromium.org
, Aug 30 2016