initCollapsed not always respected when collapsible is inside of array about payload

[franciscolourenco]

The same happens in existing posts, for any array item where the collapsed preference has not been saved yet.

[franciscolourenco]

Aside: In our use-case it would be preferable if the preferences would not be persisted in the document creation page.

[DanRibbens]

We need to make sure that the preferences for fields only send if the document ID is present. This should be fixed with one small condition being added to the collapsible field as well as any other field level preferences.

[franciscolourenco]

@DanRibbens that would be ideal. Although note that the issue in this bug report is not exactly that one. It is specifically related to preferences of collapsibles when used inside of array fields, and happens in existing documents as well, so they can probably be dealt with separately.

It is a less important issue though. What you describe seems more important and would have the most impact in usability.

[franciscolourenco]

Created another ticket to track that issue: #5747

[jacobsfletch]

Hey @franciscolourenco just looked into this and the root of the problem here is exactly what @DanRibbens suggested. In your screen recording I can see that your preferences doc has the key property set to undefined. This should be the ID of the document, but obviously new documents don't have a key yet. They simply shouldn't save any preferences (just as you've described in #5747). This fix will close both issues—initCollapsed was not being respected because it wrongly found preferences which held it open.

[franciscolourenco]

I can reproduce this issue in an existing document, so it seems they are independent issues:

• If there are no preferences for the array field at all, the initCollapsed option is respected.
• From the moment that preferences are saved for one of the array items, initCollapsed is ignored for all array items, and it defaults to expanded if an array item doesn't have preferences.

This issue seems minor, but still there:

Existing.Document.mp4

[jacobsfletch]

Thank you for the follow-up 👍. @PatrikKozak can you look into this?

[PatrikKozak]

@jacobsfletch @franciscolourenco Yup looking into this.