Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Migrating to AndroidX #62

Open
pavel163 opened this issue Dec 11, 2018 · 7 comments
Open

Migrating to AndroidX #62

pavel163 opened this issue Dec 11, 2018 · 7 comments
Labels

Comments

@pavel163
Copy link

No description provided.

@ssaxena01
Copy link

ssaxena01 commented May 23, 2019

@johan12345 Does this library have support for AndroidX?
After migrating to AndroidX, I'm receiving this error:
Failed to transform artifact 'multiline-collapsingtoolbar.aar (net.opacapp:multiline-collapsingtoolbar:1.6.0)' to match attributes {artifactType=jar}
Please see related Google Issue: https://issuetracker.google.com/issues/118658603#comment15
This suggests the author provide support for AndroidX.

@johan12345
Copy link
Contributor

As discussed in #63 , the update to AndroidX is not a trivial task, as the library is calling a lot of internal support library interfacs which are not part of the public API. Previously, this was not a problem, but with AndroidX, these cannot be called anymore due to the RestrictTo annotations.

There are ways to work around this, but that would involve copying a lot of additional classes from AndroidX into our library, which again increases the effort of maintaining it with future updates. Thus, from my point of view, the only viable solution is to finally get Google to implement this in the main Material Components library (see material-components/material-components-android#6).

@raphaelm
Copy link
Member

I don't have a lot of hope that this will ever happen, but maybe we should create a third feature request here: https://issuetracker.google.com/issues/new?component=439535

@johan12345
Copy link
Contributor

We have now opened a PR at material-components/material-components-android#413 and hope this feature can then be merged into the main Material Components library.

@PattaFeuFeu
Copy link

It seems like we are getting a lot closer‽ The PR has been merged!

I wasn’t able to test the changes just yet but will soon.

@johan12345
Copy link
Contributor

johan12345 commented Feb 19, 2020

Yep, the PR is merged finally :) I'm not completely sure about the next steps, but I suppose it will be included in the next alpha release (1.2.0-alpha05?)

@johan12345
Copy link
Contributor

Multiline support is included in the latest alpha release.

However, I found a small bug in this new version that still needs to be fixed:
material-components/material-components-android#1052

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants