Code review comment for lp:~bilalakhtar/update-manager/unity-crash-fix

Revision history for this message
Bilal Akhtar (bilalakhtar) wrote :

I guess this would need more discussion before implementation.

I've filed merge request https://code.launchpad.net/~bilalakhtar/unity/fix-unity-crash-on-hidden-quicklistitem/+merge/61067 which fixes the Unity crash on setting VISIBLE to false.

Now we have 2 choices:

1) Go with the current arrangement, since that Unity branch should land in very soon.

2) Use the ENABLED property to disable/enable the Install All Updates button, even when the Unity bug gets fixed.

Its an ergonomic difference. Michael?

« Back to merge proposal