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

update to dev13 #28

Merged
merged 2 commits into from Jun 14, 2020
Merged

update to dev13 #28

merged 2 commits into from Jun 14, 2020

Conversation

luca992
Copy link
Contributor

@luca992 luca992 commented Jun 11, 2020

Updates to dev13, update compile and target sdk to 30.

I know you did not want the samples to reference the router module directly. And it should instead reference an existing release. But after being forced to update gradle, the only way I could get the lifelike module to compile was to update it to dev13. Which meant it needed access to router built with dev13.

Also I couldn't get the nested sample to build using dev09 with the gradle update so I was able to update that one to dev12

Copy link
Contributor

@ShikaSD ShikaSD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for updating this! :)
A couple of style / code related things here

}
}
)
/*Table layout was removed temporarily until we will make it available again with a refreshed API. (Id88a7)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest to remove old API completely, as it will be refreshed anyway
@zsoltk what do you think?

app-nested-containers/build.gradle Outdated Show resolved Hide resolved
@luca992
Copy link
Contributor Author

luca992 commented Jun 11, 2020

No problem 👍. The interactive preview is working in the latest android studio now. So that's pretty cool

@ShikaSD
Copy link
Contributor

ShikaSD commented Jun 12, 2020

@zsoltk Looks good to me, can you take a look?
Layout is a bit broken, but it has been for awhile tbh.

@zsoltk
Copy link
Owner

zsoltk commented Jun 14, 2020

@luca992 When trying to scroll photos of album in app-lifelike (with new AdapterList) it crashes with

java.lang.IllegalStateException: LayoutCoordinate operations are only valid when isAttached is true

I won't block the release, as it's the Router that's important, but would be nice to fix it in another ticket.

@zsoltk zsoltk merged commit d675d6b into zsoltk:master Jun 14, 2020
@luca992
Copy link
Contributor Author

luca992 commented Jun 14, 2020

@zsoltk
Yeah I know. I referenced that bug in a comment:

scrolling fast may cause exception: https://issuetracker.google.com/issues/154653504

The bug fix should be included in one of the next compose releases

@zsoltk
Copy link
Owner

zsoltk commented Jun 14, 2020

Ah, I missed that. Fair enough.

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