New issue
Advanced search Search tips

Issue 775227 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 725683


Participants' hotlists:
Fixing-touch


Sign in to add a comment

Cros splitview - Do not allow a window snap to a position where its size is smaller than the window's minimal size

Project Member Reported by x...@chromium.org, Oct 16 2017

Issue description

Currently we have 3 fixed snapped position ratio: 0.33, 0.5, 0.67. But it's possible that a window's minimal width is larger than 0.33 * screen_width but smaller than 0.5 * screen_width (in landscape orientation). In this case we still allow the window enter into split view mode, but not allow it snap to a position that make its size smaller than its minimal size. 

- When the divider passes over the middle of the screen (or the window's minimal bounds), show the black scrim. 
- When the gesture is released, snap to a position where its size is larger than its minimal size.


 

Comment 1 by x...@chromium.org, Oct 16 2017

Summary: Cros splitview - Do not allow a window snap to a position where its size is smaller than the window's minimal size (was: Cros splitview - Move the window to the next )

Comment 2 by x...@chromium.org, Oct 16 2017

Blocking: 725683

Comment 3 by minch@chromium.org, Oct 28 2017

Labels: M-64
Status: Started (was: Untriaged)
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 1 2017

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

commit d1d132951382f1189ea06d6e76be2f04f5db0a04
Author: MinChen <minch@chromium.org>
Date: Wed Nov 01 19:42:24 2017

Don't allow the window be snapped to a position that is smaller than its minimum size.

Changes:
1. Update the optional divider ratios (0.33f, 0.67f) according to the minimum size of
   the snapped windows.
2. Show the black scrim when the divider passes over the window's minimum size.

Test: SplitViewControllerTest.DividerPositionWithWindowMinimumSizeTest
Bug:  775227 
Change-Id: Ia7ca6b42639a2e8963d5bfb95eb449e520f048fa
Reviewed-on: https://chromium-review.googlesource.com/735980
Commit-Queue: min c <minch@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#513222}
[modify] https://crrev.com/d1d132951382f1189ea06d6e76be2f04f5db0a04/ash/wm/splitview/split_view_controller.cc
[modify] https://crrev.com/d1d132951382f1189ea06d6e76be2f04f5db0a04/ash/wm/splitview/split_view_controller.h
[modify] https://crrev.com/d1d132951382f1189ea06d6e76be2f04f5db0a04/ash/wm/splitview/split_view_controller_unittest.cc

Comment 5 by minch@chromium.org, Nov 1 2017

Status: Fixed (was: Started)

Sign in to add a comment