Issue metadata
Sign in to add a comment
|
Focus is not shifting to next filter on clicking forward, backward arrows in Camera.app |
||||||||||||||||||||||
Issue descriptionVersion: 54.0.2792.0/8573.0.0 (Official Build) dev-channel Gnawty, Peppy, Daisy OS: Chrome What steps will reproduce the problem? (1)Sign in to user>> go to Camera.app>> go to toggle filters>> try to click forward, backward arrows>> and observe Expected: On-Clicking forward, backward arrows blue focus should be shifted to next filter Actual: Instead focus is not shifting to next filter This is a Regression as crash is not seen in M51: 51.0.2704.103/8172.60.0 -Stable-Channel Daisy Issue is not applicable to Linux Windows
,
Jul 11 2016
Able to reproduce the issue on chrome(Peppy) using Version: 54.0.2792.0/8573.0.0. Observed the focus is not moving to the next filter instead the scroll bar is moving left and right.
,
Jul 11 2016
,
Jul 11 2016
Seems like a recent regression. I don't have it on 54.0.2787.0. I expect some temporary bug in Blink, as we didn't change anything recently in the Camera app.
,
Jul 11 2016
It seems to be the scroll is very erratic and creating bad user experience, so increasing the priority by marking it as RB-Beta. Please feel free to edit or remove if this is not the case.
,
Jul 11 2016
,
Jul 11 2016
Camera app is still using the deprecated keyIdentifier (https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/keyIdentifier); I'll submit a fix.
,
Jul 19 2016
The following revision refers to this bug: https://chromium.googlesource.com/apps/camera/+/92b247a37496a2ad77b3085a360e6523653d6a42 commit 92b247a37496a2ad77b3085a360e6523653d6a42 Author: Yuli Huang <yuli@chromium.org> Date: Wed Jul 13 11:37:06 2016 Support both KeyboardEvent keyIdentifier and key. TEST=Tested manually on peppy. BUG= chromium:627029 Change-Id: I97a8c1a20a36ecfba2a45eaa57f8dfdade737c80 Reviewed-on: https://chromium-review.googlesource.com/360292 Reviewed-by: Tomasz Mikolajewski <mtomasz@chromium.org> Tested-by: Yuli Huang <yuli@chromium.org> [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/util.js [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/views/camera.js [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/views/album.js [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/views/gallery_base.js [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/views/dialog.js
,
Aug 1 2016
Please help to verify the fix on go/cros-camera.
,
Aug 1 2016
Verified on Chome OS 54.0.2809.0/8650.0.0 samus
,
Aug 2 2016
Thanks for verifying. I'm going to push the new version today.
,
Jan 15
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/92b247a37496a2ad77b3085a360e6523653d6a42 commit 92b247a37496a2ad77b3085a360e6523653d6a42 Author: Yuli Huang <yuli@chromium.org> Date: Tue Jul 19 05:25:21 2016 Support both KeyboardEvent keyIdentifier and key. TEST=Tested manually on peppy. BUG= chromium:627029 Change-Id: I97a8c1a20a36ecfba2a45eaa57f8dfdade737c80 Reviewed-on: https://chromium-review.googlesource.com/360292 Reviewed-by: Tomasz Mikolajewski <mtomasz@chromium.org> Tested-by: Yuli Huang <yuli@chromium.org> [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/util.js [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/views/album.js [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/views/camera.js [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/views/dialog.js [modify] https://crrev.com/92b247a37496a2ad77b3085a360e6523653d6a42/src/js/views/gallery_base.js
,
Today
(11 hours ago)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d05463dc69133c53b49f5a530b2b83a8e0afc7e3 commit d05463dc69133c53b49f5a530b2b83a8e0afc7e3 Author: Yuli Huang <yuli@chromium.org> Date: Tue Jul 19 05:25:21 2016 Support both KeyboardEvent keyIdentifier and key. TEST=Tested manually on peppy. BUG= chromium:627029 Change-Id: I97a8c1a20a36ecfba2a45eaa57f8dfdade737c80 Reviewed-on: https://chromium-review.googlesource.com/360292 Reviewed-by: Tomasz Mikolajewski <mtomasz@chromium.org> Tested-by: Yuli Huang <yuli@chromium.org> |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by sandeepk...@techmahindra.com
, Jul 11 2016