-
Notifications
You must be signed in to change notification settings - Fork 992
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Settings transitions #477
Settings transitions #477
Conversation
β¦verride exitTransition everywhere. Changed Discovery -> Settings transition to slide up.
This feels a bit jarring to me. Agreed with the sentiment, though. How about we remove the transition completely? The Drawer -> Section transition (instantaneous, in this suggestion) should occur before the drawer closes. Gmail on Android is a good example of this. |
Gotcha. The Activities no-transition still feels less interruptive to meβat
least in that GIF format.
β¦On Tue, Feb 12, 2019 at 5:06 PM Izzy Oji ***@***.***> wrote:
Gmail is using Fragments, since we're using Activities, it's not possible
(we're going from 1 screen with a drawer to a new one). Here's what it
looks like with no transition:
[image: device-2019-02-12-170149 2019-02-12 17_06_10]
<https://user-images.githubusercontent.com/1289295/52671235-8333a100-2ee8-11e9-9a49-903013669ebc.gif>
β
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#477 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AC9f-f4ro4MJPskZz8A77eBZ8mYzIPChks5vMzrogaJpZM4a4GTX>
.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great code cleanup!
β Tested
- Tested on the
Samsung s8
<style name="SettingsActivityTransitions" parent="android:style/Animation.Activity"> | ||
<item name="android:activityOpenEnterAnimation">@anim/settings_slide_in_from_bottom</item> | ||
<item name="android:activityOpenExitAnimation">@null</item> | ||
<item name="android:activityCloseEnterAnimation">@null</item> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
β¨
What β
Created standard style
SettingsActivity
for settings transitions so we don't need to overrideexitTransition
everywhere.Removed Discovery -> Settings transition.
Story π
Trello
See π
From Discovery
![device-2019-02-12-162352 2019-02-12 16_26_19](https://user-images.githubusercontent.com/1289295/52671235-8333a100-2ee8-11e9-9a49-903013669ebc.gif)
Top Level Settings
![device-2019-02-12-161633 2019-02-12 16_26_45](https://user-images.githubusercontent.com/1289295/52669002-1073f700-2ee3-11e9-97c8-fcc96d9e2ae2.gif)
Account & Payment methods
![device-2019-02-12-162915 2019-02-12 16_30_22](https://user-images.githubusercontent.com/1289295/52669217-85473100-2ee3-11e9-92e6-0d60ef9f6142.gif)
(change email is omitted from gif on purpose)
Notifications
![device-2019-02-12-162118 2019-02-12 16_26_03](https://user-images.githubusercontent.com/1289295/52669271-a3149600-2ee3-11e9-80eb-3e2bc191b802.gif)