Issue metadata
Sign in to add a comment
|
Regression : Unable to view Selected Profile picture in 'Create Supervised User' overlay |
||||||||||||||||||||||
Issue descriptionChrome Version: 63.0.3212.0/9930.0.0 dev-channel Daisy,Candy and Reks OS: Chrome What steps will reproduce the problem? (1)In sign-out screen click on three dot Menu to 'Add supervised User' (2)Click on 'Create Supervised User' -> Now enter password in'Choose the manager for this supervised user' Overlay -> Navigates to 'Create Supervised User' overlay (3)Now at Profile pictures select last picture(Coffee) ->Press Left arrow key and observe Profile picture is selected(Preview is seen) but observe at profile pictures selected picture is not seen (Please refer Video) Expected: At Profile pictures selected picture should be seen Actual: Instead unable to view Selected Profile picture This is Regression Issue as same is working fine in M-61
,
Sep 11 2017
Thanks for the report. I'll investigate this asap.
,
Sep 12 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/802456f677be6e34686a96c01e79f08e91c2fa5e commit 802456f677be6e34686a96c01e79f08e91c2fa5e Author: David Reveman <reveman@chromium.org> Date: Tue Sep 12 01:51:34 2017 oobe: Fix profile picture support when adding supervised user. This makes a minimal set up changes to get the supervised user creation screen to support the new set of default user images. The changes are: - Limit default images to 23 first images of current set. - Increase size of image grid to 272px to be able to display 23 images. - Always scale default image to 220x220px. - Use 576x576 capture size to match settings UI. - Disable animation for grid images. - Always use 2x resource for preview. Limiting the number of default images to 23 avoids more complicated changes needed to support proper overflow handling for the grid. This can be improved as part of more general UI updates to the supervised user creation flow. Bug: 763780 , 763777 , 763776 Test: manual Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I64346ee6baa8dedb1b3acc8d738716c96f6aa0d7 Reviewed-on: https://chromium-review.googlesource.com/661862 Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: David Reveman <reveman@chromium.org> Cr-Commit-Position: refs/heads/master@{#501150} [modify] https://crrev.com/802456f677be6e34686a96c01e79f08e91c2fa5e/chrome/browser/resources/chromeos/login/screen_supervised_user_creation.css [modify] https://crrev.com/802456f677be6e34686a96c01e79f08e91c2fa5e/chrome/browser/resources/chromeos/login/screen_supervised_user_creation.js [modify] https://crrev.com/802456f677be6e34686a96c01e79f08e91c2fa5e/chrome/browser/resources/chromeos/user_images_grid.js
,
Sep 12 2017
,
Sep 12 2017
This bug requires manual review: Request affecting a post-stable build Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Sep 13 2017
Your change meets the bar and is auto-approved for M62. Please go ahead and merge the CL to branch 3202 manually. Please contact 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 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c5dad61c441bcec2dc060bc854d3b0d5aab6285b commit c5dad61c441bcec2dc060bc854d3b0d5aab6285b Author: David Reveman <reveman@chromium.org> Date: Wed Sep 13 03:21:13 2017 oobe: Fix profile picture support when adding supervised user. This makes a minimal set up changes to get the supervised user creation screen to support the new set of default user images. The changes are: - Limit default images to 23 first images of current set. - Increase size of image grid to 272px to be able to display 23 images. - Always scale default image to 220x220px. - Use 576x576 capture size to match settings UI. - Disable animation for grid images. - Always use 2x resource for preview. Limiting the number of default images to 23 avoids more complicated changes needed to support proper overflow handling for the grid. This can be improved as part of more general UI updates to the supervised user creation flow. TBR=reveman@chromium.org (cherry picked from commit 802456f677be6e34686a96c01e79f08e91c2fa5e) Bug: 763780 , 763777 , 763776 Test: manual Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I64346ee6baa8dedb1b3acc8d738716c96f6aa0d7 Reviewed-on: https://chromium-review.googlesource.com/661862 Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: David Reveman <reveman@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#501150} Reviewed-on: https://chromium-review.googlesource.com/663306 Reviewed-by: David Reveman <reveman@chromium.org> Cr-Commit-Position: refs/branch-heads/3202@{#194} Cr-Branched-From: fa6a5d87adff761bc16afc5498c3f5944c1daa68-refs/heads/master@{#499098} [modify] https://crrev.com/c5dad61c441bcec2dc060bc854d3b0d5aab6285b/chrome/browser/resources/chromeos/login/screen_supervised_user_creation.css [modify] https://crrev.com/c5dad61c441bcec2dc060bc854d3b0d5aab6285b/chrome/browser/resources/chromeos/login/screen_supervised_user_creation.js [modify] https://crrev.com/c5dad61c441bcec2dc060bc854d3b0d5aab6285b/chrome/browser/resources/chromeos/user_images_grid.js
,
Sep 13 2017
Approving merge to M61.
,
Sep 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fe777a64902291225fe918761a2e4c17daf13419 commit fe777a64902291225fe918761a2e4c17daf13419 Author: David Reveman <reveman@chromium.org> Date: Wed Sep 13 21:28:30 2017 oobe: Fix profile picture support when adding supervised user. This makes a minimal set up changes to get the supervised user creation screen to support the new set of default user images. The changes are: - Limit default images to 23 first images of current set. - Increase size of image grid to 272px to be able to display 23 images. - Always scale default image to 220x220px. - Use 576x576 capture size to match settings UI. - Disable animation for grid images. - Always use 2x resource for preview. Limiting the number of default images to 23 avoids more complicated changes needed to support proper overflow handling for the grid. This can be improved as part of more general UI updates to the supervised user creation flow. TBR=reveman@chromium.org (cherry picked from commit 802456f677be6e34686a96c01e79f08e91c2fa5e) Bug: 763780 , 763777 , 763776 Test: manual Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I64346ee6baa8dedb1b3acc8d738716c96f6aa0d7 Reviewed-on: https://chromium-review.googlesource.com/661862 Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: David Reveman <reveman@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#501150} Reviewed-on: https://chromium-review.googlesource.com/665821 Reviewed-by: David Reveman <reveman@chromium.org> Cr-Commit-Position: refs/branch-heads/3163@{#1190} Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528} [modify] https://crrev.com/fe777a64902291225fe918761a2e4c17daf13419/chrome/browser/resources/chromeos/login/screen_supervised_user_creation.css [modify] https://crrev.com/fe777a64902291225fe918761a2e4c17daf13419/chrome/browser/resources/chromeos/login/screen_supervised_user_creation.js [modify] https://crrev.com/fe777a64902291225fe918761a2e4c17daf13419/chrome/browser/resources/chromeos/user_images_grid.js
,
Nov 3 2017
Pretty sure this made it into the 63 branch but approving merge just in case.
,
Nov 6 2017
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
,
Nov 10 2017
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 |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by mmanchala@chromium.org
, Sep 11 2017