Rename and sort the modules in the feature profiles table for easier skimming

[Johennes]

This renames the modules in the table to match the section titles and orders the table alphabetically within the two subsections of modules with and without requirements.

Adding the modules that were missing in v1.10 has been moved out into #1860.

Fixes: #1437

Pull Request Checklist

• Pull request includes a changelog file
• Pull request includes a sign off
• Pull request is classified as 'other changes'

Preview: https://pr1855--matrix-spec-previews.netlify.app

[turt2live]

Thanks! I'd suggest we split out the adding and resorting to their own PRs. It's not immediately clear to me that this is the ordering we'd want (previously it was loosely ordered on "required-ness"). Adding the missing stuff is certainly important, though.

[Johennes]

Ok, I can do that. Where would you suggest I insert the missing modules in the existing order?

[turt2live]

We typically just append them to the end, for better or worse. If they're "required" (they shouldn't be), put them under the existing required ones.

[Johennes]

I pushed #1860 which merely adds the three missing modules without touching the table ordering.

[Johennes]

The brief discussion in the matrix-spec room was, sadly, inconclusive. The suggested options so far are:

1. Sort alphabetically
2. Sort by section occurrence
3. Sort by requiredness (this is more or less how the table is sorted currently)
4. Sort by requiredness first and alphabetically second

As a middleground, I updated the PR to match option 4 but also renamed those rows where the module name didn't match the section title. This should hopefully still make it easy enough to cross-reference the table with the sections.

For convenience, the PR can be reviewed by commit:

• aa93f8b does the renaming
• 5afe6ed does the sorting

[Johennes]

CC @uhoreg / @anoadragon453 since you participated in the chat in matrix-spec.

[turt2live]

I'm going to abstain from reviewing this one - the order doesn't bother me.

abstain