mozilla-mobile / fenix

⚠️ Fenix (Firefox for Android) moved to a new repository. It is now developed and maintained as part of: https://github.com/mozilla-mobile/firefox-android
https://github.com/mozilla-mobile/firefox-android
Mozilla Public License 2.0
6.47k stars 1.27k forks source link

Add Button Ripples Animations to Dialog and Snackbar Buttons #9516

Closed theFireProject closed 1 year ago

theFireProject commented 4 years ago

Why/User Benefit/User Problem

It's a visual improvement that enhances look and feel of the graphical user interface. This is a common feature for Material Design that gives feedback to the user when they interact with a button. That behavior is known and expected by users. Missing button animations for dialogues and snackbars in Fenix is a regression from Fennec.

What/Requirements

Add the standard button animations from Android as known from Fennec to Fenix when interacting with buttons of dialogues and the button on snackbars:

Acceptance Criteria (how do I know when I’m done?)

If interaction with buttons of all dialogues and all snackbars is animated.

Screenshots (6)

20200330_152119 20200330_152157

20200330_152247 20200330_152303 20200330_152409 20200330_152335

┆Issue is synchronized with this Jira Task

Poopooracoocoo commented 4 years ago

The ripple animations you described sound like Android 7-8. Android 5-6 and Android 9+ have different animations.

sv-sdeiac commented 4 years ago

Hi, I tested this issue and I found 3 different problems:

Build: Firefox Preview Nightly 4/21 #21121811.

Devices:

@ekager, can you take a look at those details and come with updates about them? I'll remove the qa:needed flag till we have some updates, thanks.

Poopooracoocoo commented 4 years ago

@sv-sdeiac those buttons don't have a shape too. On Android buttons are typically rounded rectangles but some people customise the shape of them.

image https://material.io/components/dialogs https://material.io/components/buttons

ekager commented 4 years ago

Yep, those all look like valid issues! I'm going to unassign myself for now, but anyone is free to pick this up if they have ideas.

czlucius commented 2 years ago

Yep, those all look like valid issues! I'm going to unassign myself for now, but anyone is free to pick this up if they have ideas.

I would like to work on this issue

czlucius commented 2 years ago

android:colorControlHighlight seems to control the ripple color when using ?android:attr/selectableItemBackgroundBorderless

https://cs.android.com/android/platform/superproject/+/master:packages/apps/Contacts/res/drawable/item_background_material_borderless_dark.xml?q=selectableItemBackgroundBorderless which has a color of ripple_material_dark: https://android.googlesource.com/platform/frameworks/base/+/master/core/res/res/color/ripple_material_dark.xml

csadilek commented 1 year ago

Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1807131

Change performed by the Move to Bugzilla add-on.