Fix some regressions with recent form styling.

Review Request #12830 — Created Feb. 7, 2023 and submitted — Latest diff uploaded

Information

Review Board
release-5.0.x

Reviewers

The recent form styling fixes ended up causing neighboring some
neighboring form rows to be squashed together in some cases, and nested
fieldsets to have too much additional padding. Help text also got
squashed a bit with neighboring fields.

This was caused by a few things:

  1. Some assumptions that the first and last form rows in a fieldset
    should have their padding trimmed.

  2. Usage of padding instead of margin in places, which contributed
    to extra unwanted padding.

  3. Always applying padding to the top and bottom of
    .rb-c-fieldset__fields, even when nested.

This has been addressed by switching to margin in several places,
restoring top/bottom spacing for field rows and collections of fields in
all cases, and then ensuring that subforms don't have padding or margins
in their subform-level `__fields`` container (if present).

Went through every settings page and subform, various database pages
(repository, user, webhook, review groups, default reviewers), and the
My Account pages, checking for consistency in presentation.

There are still cases where the larger form row padding is less than
ideal (groups of checkboxes, for example), but that would have to be
solved separately.


Commits

Files