Animate split screen phantom windows. |
||||
Issue description
,
Feb 22 2018
,
Feb 23 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8250507fce543357e051cbd9bcf0573bd3c25991 commit 8250507fce543357e051cbd9bcf0573bd3c25991 Author: Sammie Quon <sammiequon@google.com> Date: Fri Feb 23 00:08:28 2018 splitview: Animate phantom windows p2. spec: https://mccanny.users.x20web.corp.google.com/www/splitscreen-motion/index.html Performs the animations after the setup in p1. I held off shifting the text label back and forth as the spec states because there is already a transform on it (to rotate the label). Perhaps animating the bounds would work, but some math would be required to do that. Test: manual Bug: 808283 Change-Id: If315601c63f4d8850e026eecffb390de3096fee7 Reviewed-on: https://chromium-review.googlesource.com/900484 Reviewed-by: Mitsuru Oshima <oshima@chromium.org> Reviewed-by: Xiaoqian Dai <xdai@chromium.org> Commit-Queue: Sammie Quon <sammiequon@chromium.org> Cr-Commit-Position: refs/heads/master@{#538628} [modify] https://crrev.com/8250507fce543357e051cbd9bcf0573bd3c25991/ash/wm/overview/window_selector_item.cc [modify] https://crrev.com/8250507fce543357e051cbd9bcf0573bd3c25991/ash/wm/splitview/split_view_overview_overlay.cc [modify] https://crrev.com/8250507fce543357e051cbd9bcf0573bd3c25991/ash/wm/splitview/split_view_utils.cc [modify] https://crrev.com/8250507fce543357e051cbd9bcf0573bd3c25991/ash/wm/splitview/split_view_utils.h
,
Feb 28 2018
+oshima Oshima mentioned renaming phantom window and overview overlay in cl 1. How does phantom window -> splitview preview area overview overlay -> splitview drag indicators sound? xdai any suggestions?
,
Feb 28 2018
Sounds good to me.
,
Mar 1 2018
oops forgot to add +oshima oshima what do you think of the suggestions in #4
,
Mar 1 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/59159caab58eb9232c6b4a35ee8f07e8de036d96 commit 59159caab58eb9232c6b4a35ee8f07e8de036d96 Author: Sammie Quon <sammiequon@google.com> Date: Thu Mar 01 06:05:14 2018 splitview: Animate phantom windows p3. Preserve round edges after animation. Calculates the bounds after animation and applies identity transform. Test: manual Bug: 808283 Change-Id: I8370f63983bf3fd568d7cb5712e7cd09e380edeb Reviewed-on: https://chromium-review.googlesource.com/940568 Commit-Queue: Sammie Quon <sammiequon@chromium.org> Reviewed-by: Xiaoqian Dai <xdai@chromium.org> Reviewed-by: Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#540068} [modify] https://crrev.com/59159caab58eb9232c6b4a35ee8f07e8de036d96/ash/wm/splitview/split_view_overview_overlay.cc
,
Mar 1 2018
#6, sgtm
,
Mar 14 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d commit be8f7b095ed9bba9830783a4297fbf30f4ad4f1d Author: Sammie Quon <sammiequon@google.com> Date: Wed Mar 14 02:08:17 2018 splitview: Rename phantom windows and overlay to more specific. Phantom window and overlay have other meanings for other uses. Phantom windows -> preview area as it is a highlight which gives a preview of the bounds of a snapped window. Overlay -> drag indicators. The class handles displaying various indications for entering split view. Test: ash_unittests SplitViewWindowSelectorTest.* Bug: 808283 Change-Id: If8bb36792daa7d89c8dac96cc5baf841bee3bdee Reviewed-on: https://chromium-review.googlesource.com/946887 Commit-Queue: Sammie Quon <sammiequon@chromium.org> Reviewed-by: Xiaoqian Dai <xdai@chromium.org> Reviewed-by: Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#542984} [modify] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/BUILD.gn [modify] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/wm/overview/overview_window_drag_controller.cc [modify] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/wm/overview/overview_window_drag_controller.h [modify] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/wm/overview/window_selector.cc [modify] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/wm/overview/window_selector.h [modify] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/wm/overview/window_selector_unittest.cc [rename] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/wm/splitview/split_view_drag_indicators.cc [rename] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/wm/splitview/split_view_drag_indicators.h [modify] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/wm/splitview/split_view_utils.cc [modify] https://crrev.com/be8f7b095ed9bba9830783a4297fbf30f4ad4f1d/ash/wm/splitview/split_view_utils.h
,
Mar 17 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e61bdc1e8a71cd3e4796cbfc494438372d41e12d commit e61bdc1e8a71cd3e4796cbfc494438372d41e12d Author: Sammie Quon <sammiequon@google.com> Date: Sat Mar 17 02:05:54 2018 splitview: Animate split view indicators p4. Last one of spec. Animates the "drag to use splitview" labels (slide in and out and also fade in and out). Create a new child view, which is the one which will get a rotation transform. The parent view will be the layer that gets translated. Note: p1 p2 p3 are called "splitview: Animate phantom window". Test: manual Bug: 808283 Change-Id: I95e1cef7056c1ab4272c05fc8a1a32187576e6bd Reviewed-on: https://chromium-review.googlesource.com/961497 Reviewed-by: Xiaoqian Dai <xdai@chromium.org> Commit-Queue: Sammie Quon <sammiequon@chromium.org> Cr-Commit-Position: refs/heads/master@{#543914} [modify] https://crrev.com/e61bdc1e8a71cd3e4796cbfc494438372d41e12d/ash/wm/splitview/split_view_drag_indicators.cc [modify] https://crrev.com/e61bdc1e8a71cd3e4796cbfc494438372d41e12d/ash/wm/splitview/split_view_utils.cc [modify] https://crrev.com/e61bdc1e8a71cd3e4796cbfc494438372d41e12d/ash/wm/splitview/split_view_utils.h
,
Mar 17 2018
|
||||
►
Sign in to add a comment |
||||
Comment 1 by bugdroid1@chromium.org
, Feb 16 2018