XWIKI-23546: Improve usability of document tree macro configuration

[surli]

Jira URL

https://jira.xwiki.org/browse/XWIKI-23546

Changes

Description

• Provide grouping and ordering of various parameters

Clarifications

Screenshots & Video

Before

After

Executed Tests

mvn clean install -Pquality,integration-tests,docker on module xwiki-platform-index-tree. Should be enough: we don't have any integration tests manipulating the macro config UI.

Expected merging strategy

• Prefers squash: Yes
• Backport on branches:
  • None

[tkrieck]

Some things that could be reworked if possible:

• I would order the tabs by order of complexity.
  • Display Options (common quick customizations)
  • Elements to Show (same, a bit more expanded)
  • Filters
  • Optional Parameters
• The hints that are over explanatory. I guess this is outside the scope of this PR

Otherwise, seems fine the way it is.

[mflorea]

The hints that are over explanatory. I guess this is outside the scope of this PR

I agree that some hints are too long and can be simplified (no more than 2 lines of text ideally).

Furthermore, just by looking at the screenshots, I would:

• move the "Exclusions" parameter to the "Filters" group
• move the "Expand to level" to the "Display options" group (maybe also the "Sort pages by", but I'm not sure)

[mflorea]

Looks good, my only comment is the one posted above.

[surli]

I would order the tabs by order of complexity.

Thanks for the review. I would keep displaying "Optional parameters" first: IMO the first parameters (root / open to / limit / sort by) are probably the most used parameters and that's why I decided to fix their order. Unless I can find a common group name for them. I will check for ordering of the other tabs.

move the "Exclusions" parameter to the "Filters" group
move the "Expand to level" to the "Display options" group (maybe also the "Sort pages by", but I'm not sure)

OK I'll do that for Exclusions / Expand to. For sort I'd keep it in first tab to be quickly available.

The hints that are over explanatory. I guess this is outside the scope of this PR

Yeah they probably need some work, I'll see if I can improve them a bit, otherwise it could be done later.

[github-actions]

💚 All backports created successfully

Status	Branch	Result
✅	stable-17.9.x

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details