New issue
Advanced search Search tips

Issue 804603 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Task
Q2

Blocked on:
issue 804499



Sign in to add a comment

Show the correct tab grid toolbars when the size class changes

Project Member Reported by marq@chromium.org, Jan 23 2018

Issue description

Show the correct tab grid toolbars when the size class changes
 

Comment 1 by marq@chromium.org, Jan 23 2018

Labels: small

Comment 2 by marq@chromium.org, Jan 23 2018

Components: UI>Browser

Comment 3 by marq@chromium.org, Jan 23 2018

Blockedon: 804499

Comment 4 by marq@chromium.org, Jan 23 2018

Labels: Q1

Comment 5 by cma...@chromium.org, Jan 26 2018

Labels: Q2

Comment 6 by cmasso@google.com, Jan 27 2018

Labels: -Q1

Comment 7 by marq@chromium.org, Feb 26 2018

Owner: ----
Status: Available (was: Assigned)

Comment 8 by edchin@chromium.org, Mar 21 2018

Cc: marq@chromium.org
Owner: edchin@chromium.org
Status: Started (was: Available)
This is related to the page scrollView being set to a weird offset (between two pages) whenever the orientation changes. This was bothering me to no end. So I had to fix it. A CL is in-flight. 
Project Member

Comment 9 by bugdroid1@chromium.org, Mar 21 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a98c6ca6fa00f05052c8a494d46c5c9bf391c68a

commit a98c6ca6fa00f05052c8a494d46c5c9bf391c68a
Author: edchin <edchin@chromium.org>
Date: Wed Mar 21 14:29:40 2018

[ios] Fix tab grid rotation

This CL updates the correct page scroll offset and toolbar
configuration upon rotation.

Bug:  804603 
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I055ce3f63204393bdc8ced23f8ed3ec212b0eab8
Reviewed-on: https://chromium-review.googlesource.com/972666
Reviewed-by: edchin <edchin@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Commit-Queue: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544689}
[modify] https://crrev.com/a98c6ca6fa00f05052c8a494d46c5c9bf391c68a/ios/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm

Status: Fixed (was: Started)
Lookin' so good. =)

Sign in to add a comment