• 
      

    Fix some regressions with recent form styling.

    Review Request #12830 — Created Feb. 8, 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:
    Completed
    Change Summary:
    Pushed to release-5.0.x (eed96bb)