Fix widget expansion behavior in Flow Part while editing ContentItem

[Karthikeyan-tv]

This pull request addresses an unintended behavior where widgets inside flow parts expand by default when the flow part is expanded. The issue was caused by a change made in commit 1107fc1, which aimed to remove extra spacing around editing contents, flow, and bag parts.

The specific change that led to this unintended behavior was related to the modification of the collapsed class logic for widgets:

<div class="widget widget-editor card my-1 @(Model.Inline != true ? "collapsed" : "")">

This logic was altered in the commit to:

<div class="widget widget-editor card my-1 @(Model.Inline ? " collapsed" : String.Empty)">

This change caused widgets to expand by default when the flow part was expanded.

Fix:

I have corrected the behavior by updating the widget class assignment logic. The widget expansion now correctly follows the intended behavior.

[github-actions]

Thank you for submitting your first pull request, awesome! 🚀 If you haven't already, please take a moment to review our contribution guide. This guide provides helpful information to ensure your contribution aligns with our standards. A core team member will review your pull request.

If you like Orchard Core, please star our repo and join our community channels.

[Karthikeyan-tv]

@dotnet-policy-service agree

[MikeAlhayek]

@Karthikeyan-tv Thank you for your contribution! I

believe it would be beneficial to implement this as a toggleable setting. Some users may prefer the items in the BagPart to be collapsed by default, while others may not want this behavior.

Could you add a setting in BagPartSettings and FlowPartSettings?

The setting could be called AutoCollapseContainedItems, with a default value of true (reflecting the current behavior). This would allow users to modify this behavior via the settings if desired.

Thank you!