Remove isDetaching property from FocusTarget and FocusPropertiesNode [257141589]

Fixed

Internal Cleanup

Status Update

No update yet.

Description

ra...@google.com

created issue #1

Nov 3, 2022 12:18AM

Use Owner.registerOnEndApplyChangesListener to create a system where we mark nodes as dirty and defer the calculation of FocusProperties and propagation of FocusEvents to after we apply changes.

After this change we should not need a isDetaching property anymore.

Comments

ap...@google.com <ap...@google.com> #2Jan 4, 2023 03:27AM

We have some support in androidx.compose.ui.autofill

Leaving this bug open in case Ralston wants to add more info

ra...@google.com <ra...@google.com> Jan 4, 2023 06:59PM

Marked as fixed.

na...@google.com <na...@google.com> #3Jan 11, 2023 08:12PM

I found an example here. One problem is that it only works with the Google autofill service. When using any other service like Microsoft Authenticator or OneKey, the following error is shown:

D/Autofill Status: Autofill popup isn't shown because autofill is not available.
    
    Did you set up autofill?
    1. Go to Settings > System > Languages&input > Advanced > Autofill Service
    2. Pick a service
    
    Did you add an account?
    1. Go to Settings > System > Languages&input > Advanced
    2. Click on the settings icon next to the Autofill Service
    3. Add your account

Is this a bug on your side or do the app developers of these password managers need to change their implementation?

Issue 257141589

Description

Issue summary

Comments

ap...@google.com <ap...@google.com> #2Jan 4, 2023 03:27AM

ra...@google.com <ra...@google.com> Jan 4, 2023 06:59PM

na...@google.com <na...@google.com> #3Jan 11, 2023 08:12PM

Add comment

Issue metadata