Skip to content
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

[Internal] Stop using deprecated androidx.core.view.* methods #4117

Conversation

pubiqq
Copy link
Contributor

@pubiqq pubiqq commented Mar 26, 2024

The PR replaces deprecated classes and methods from androidx.core.view.* packages with the recommended ones.

Among them:

  • ViewCompat.LAYOUT_DIRECTION_RTL -> View.LAYOUT_DIRECTION_RTL
  • ViewCompat.getLayoutDirection(view) -> view.getLayoutDirection()
  • ViewCompat.setBackground(view, background) -> view.setBackground(background)
  • ViewCompat.setFitsSystemWindows(view, true) -> view.setFitsSystemWindows(true)
  • AccessibilityEventCompat.TYPE_WINDOW_CONTENT_CHANGED -> AccessibilityEvent.TYPE_WINDOW_CONTENT_CHANGED
  • MarginLayoutParamsCompat class -> ViewGroup.MarginLayoutParams class

and so on.

@paulfthomas
Copy link
Member

@pubiqq can you add more context in your descriptions? It helps with the reviews, thanks!

@pubiqq pubiqq force-pushed the internal/remove-deprecated-core-view branch from 7525d2b to dac8815 Compare April 11, 2024 18:04
@pubiqq
Copy link
Contributor Author

pubiqq commented Apr 11, 2024

Done.

@paulfthomas paulfthomas self-requested a review April 11, 2024 18:20
@pubiqq pubiqq force-pushed the internal/remove-deprecated-core-view branch from dac8815 to 611792c Compare April 23, 2024 04:05
@paulfthomas
Copy link
Member

I see 2 files added on my end when trying to sync to our internal repo:

  • third_party/java_src/android_libs/material_components/lib/java/com/google/android/material/transition/platform/MaterialContainerTransform.java
  • third_party/java_src/android_libs/material_components/lib/java/com/google/android/material/transition/platform/SlideDistanceProvider.java

These files do exist already under third_party/java_src/android_libs/material_components/lib/java/com/google/android/material/transition

Can you take a look?

@pubiqq pubiqq force-pushed the internal/remove-deprecated-core-view branch from 611792c to 5758dba Compare April 24, 2024 15:47
@pubiqq
Copy link
Contributor Author

pubiqq commented Apr 24, 2024

Ok, I've removed the changes from the auto-generated classes, but unfortunately I have no way to check if they're generated correctly because I don't have access to your build system, so you'll have to check it yourself.

@pubiqq pubiqq force-pushed the internal/remove-deprecated-core-view branch 2 times, most recently from 605b361 to 5b20c4f Compare April 30, 2024 20:16
@pubiqq pubiqq mentioned this pull request May 6, 2024
@pubiqq pubiqq force-pushed the internal/remove-deprecated-core-view branch 3 times, most recently from dd157e8 to 4b07938 Compare May 9, 2024 14:37
@leticiarossi leticiarossi self-assigned this May 9, 2024
@pubiqq pubiqq force-pushed the internal/remove-deprecated-core-view branch from 4b07938 to 47dd802 Compare May 9, 2024 20:13
@dsn5ft dsn5ft closed this in 510881a May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants