Consider renaming all *Constants.Default* to just *Constants.* [159982740]

Fixed

Feature Request

Status Update

No update yet.

Description

an...@google.com

created issue #1

Jun 26, 2020 01:29PM

See for details

https://android-review.googlesource.com/c/platform/frameworks/support/+/1351049/2/ui/ui-material/src/main/java/androidx/ui/material/Button.kt#257

Comments

js...@google.com <js...@google.com> #2Jun 29, 2020 10:07AM

Thanks for filing this.

This is a fair request and I think it will be done in the near future.

We already in progress of doing this for Buttons, so selection controls are coming some day :)

lp...@google.com <lp...@google.com> #3Dec 2, 2020 07:55PM

Reassigned to lp...@google.com.

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 62eb7deccbe3896c84e04000361bbbc7a69909af
Author: Louis Pullen-Freilich <lpf@google.com>
Date: Wed Sep 16 20:32:07 2020

Exposes InteractionState parameters in stateful Material components, such as Slider and Switch

This allows hoisting the InteractionState above the component, to control how a component appears in different Interactions, or manually setting states for a given component, such as to make a RadioButton appear pressed when a ListItem containing a RadioButton is pressed.

Bug:

b/168025711

Bug:

b/167164434

Test: ./gradlew updateApi
Relnote: "Exposes InteractionState parameters in stateful Material components, to allow hoisting the state and reading / controlling the state."
Change-Id: Iaca5f65f8132a6f2b45d461001fa740d08b6bacb

M compose/material/material/api/current.txt
M compose/material/material/api/public_plus_experimental_current.txt
M compose/material/material/api/restricted_current.txt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Checkbox.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/IconButton.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Menu.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/OutlinedTextField.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/RadioButton.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Slider.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Switch.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextField.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TextFieldImpl.kt

https://android-review.googlesource.com/1428672

ap...@google.com <ap...@google.com> #4Dec 3, 2020 09:51PM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-master-dev

commit bfdc10fdf728b1344934d2e193e16cf6700698ad
Author: Louis Pullen-Freilich <lpf@google.com>
Date: Thu Dec 03 16:27:34 2020

Renames *Constants objects to *Defaults

These objects do not just hold constants, but can also hold values used in default values for parameters, such as ButtonDefaults.colors(). Naming these as default and removing the unecessary `default` prefix makes it more consistent with their use.

Fixes:

b/159982740

Test: updateApi
Relnote: "Renamed *Constants objects such as ButtonConstants to end with Defaults instead, such as ButtonDefaults. Also removes unecessary `default` prefix from properties in these new objects."
Change-Id: Ibb915f0fd06a9c99ae6791e85634d5aea49374f6

M compose/androidview/androidview/integration-tests/androidview-demos/src/main/java/androidx/compose/androidview/demos/ComplexInteractions.kt
M compose/desktop/desktop/samples/src/jvmMain/kotlin/androidx/compose/desktop/examples/example1/Main.kt
M compose/desktop/desktop/samples/src/jvmMain/kotlin/androidx/compose/desktop/examples/popupexample/AppContent.kt
M compose/integration-tests/docs-snippets/src/main/java/androidx/compose/integration/docs/preview/LayoutPreview.kt
M compose/integration-tests/docs-snippets/src/main/java/androidx/compose/integration/docs/theming/Theming.kt
M compose/material/material/api/current.txt
M compose/material/material/api/public_plus_experimental_current.txt
M compose/material/material/api/restricted_current.txt
M compose/material/material/integration-tests/material-demos/src/main/java/androidx/compose/material/demos/ButtonDemo.kt
M compose/material/material/integration-tests/material-demos/src/main/java/androidx/compose/material/demos/TabDemo.kt
M compose/material/material/samples/src/main/java/androidx/compose/material/samples/ButtonSamples.kt
M compose/material/material/samples/src/main/java/androidx/compose/material/samples/ProgressIndicatorSamples.kt
M compose/material/material/samples/src/main/java/androidx/compose/material/samples/SelectionControlsSamples.kt
M compose/material/material/samples/src/main/java/androidx/compose/material/samples/TabSamples.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/ButtonTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/FloatingActionButtonTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/SwitchScreenshotTest.kt
M compose/material/material/src/androidAndroidTest/kotlin/androidx/compose/material/TabTest.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/BackdropScaffold.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/BottomSheetScaffold.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Button.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Checkbox.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Drawer.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Elevation.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/FloatingActionButton.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/ListItem.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Menu.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/ModalBottomSheet.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/ProgressIndicator.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/RadioButton.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Scaffold.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Slider.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Snackbar.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/SwipeToDismiss.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Swipeable.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Switch.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/Tab.kt
M compose/material/material/src/commonMain/kotlin/androidx/compose/material/TabRow.kt
M navigation/navigation-compose/integration-tests/navigation-demos/src/main/java/androidx/navigation/compose/demos/NavByDeepLinkDemo.kt
M navigation/navigation-compose/integration-tests/navigation-demos/src/main/java/androidx/navigation/compose/demos/NavPopUpToDemo.kt
M navigation/navigation-compose/samples/src/main/java/androidx/navigation/compose/samples/NavigationSamples.kt

https://android-review.googlesource.com/1515511

IssueTracker