Fix some regressions with recent form styling.

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

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.

Summary ID
Fix some regressions with recent form styling.
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).
9184223ddd4ae3ea36048c37b73e865eb62ffe50

david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-5.0.x (eed96bb)
Loading...