Fixed
Status Update
Comments
ap...@google.com <ap...@google.com> #2
I don't think #1 makes a lot of sense. There are already potentially libraries out there without that metadata.
#2 makes the most sense to me, but in case we cannot be 100% reliable we should allow blacklisting libraries as described in #3. This should probably be handled globally for a project (in gradle.properpies rather than in the DSL)
#2 makes the most sense to me, but in case we cannot be 100% reliable we should allow blacklisting libraries as described in #3. This should probably be handled globally for a project (in gradle.properpies rather than in the DSL)
il...@google.com <il...@google.com> #3
As I have mentioned I don't think there is a silver bullet. #2 does not work on 100% and #1 will take time before devs will provide the metadata. #3 is in my opinion very important to introduce asap, as the developers have no way how to workaround it at this stage. Also in the future there can be any other issue and there is nothing wrong in doing #3 until we figure out something robust (or for instance finally migrate work manager and navigation to androidx that would make #2 a feasible solution).
Description
Component used: Fragment Version used: 1.3.0-alpha05
Fragments call b/149024125 , this runs before the View is attached, meaning
ViewCompat.requestApplyInsets()
immediately before callingonViewCreated()
. However, due torequestApplyInsets()
doesn't actually do anything.We should consistently wait for the View to be attached before calling
requestApplyInsets()
.