Suggested Sites widget should have 3 tiles on smaller devices |
|||||
Issue descriptionApp Version (from "Chrome Settings > About Chrome"): 62.0.3180.0 canary iOS Version: 10.2 Device: 5S Steps to reproduce: Add the widgets Observed behavior: Suggested sites widget has 4 icons which is too many. Expected behavior: Should have 3 icons (similar to NTP) Frequency: always Additional comments:
,
Aug 11 2017
,
Aug 24 2017
,
Aug 25 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/024192bb64497a2ac8e5676f4e841a21521a5170 commit 024192bb64497a2ac8e5676f4e841a21521a5170 Author: Elodie Banel <lod@google.com> Date: Fri Aug 25 09:38:53 2017 Adjust # of tiles in content widget for device size Bug: 754114 Change-Id: I005d5d13f3954de40f0f0f6dce594b27d64b170c Reviewed-on: https://chromium-review.googlesource.com/632576 Commit-Queue: Elodie Banel <lod@chromium.org> Reviewed-by: Olivier Robin <olivierrobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#497365} [modify] https://crrev.com/024192bb64497a2ac8e5676f4e841a21521a5170/ios/chrome/content_widget_extension/content_widget_view.h [modify] https://crrev.com/024192bb64497a2ac8e5676f4e841a21521a5170/ios/chrome/content_widget_extension/content_widget_view.mm [modify] https://crrev.com/024192bb64497a2ac8e5676f4e841a21521a5170/ios/chrome/content_widget_extension/content_widget_view_controller.mm [modify] https://crrev.com/024192bb64497a2ac8e5676f4e841a21521a5170/ios/chrome/content_widget_extension/most_visited_tile_view.h [modify] https://crrev.com/024192bb64497a2ac8e5676f4e841a21521a5170/ios/chrome/content_widget_extension/most_visited_tile_view.mm
,
Aug 28 2017
,
Oct 2 2017
3 tiles displayed and 2 rows. Verified on iPhoneSE with iOS 9.3.5, M62.0.3202.43 beta. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by jasonkliu@chromium.org
, Aug 10 2017