Switch entry, review, and comment header styling to Flexbox.

Review Request #14285 — Created Jan. 2, 2025 and submitted

Information

Review Board
release-7.1.x

Reviewers

The styling for review request entries (reviews, etc.) and comment
headers now use Flexbox, which helps us manage styles more consistently
and avoids some hacks and workarounds to get things to line up in mobile
and desktop modes correctly.

This is necessary for some in-progress work on user badges, which impact
the layout of this area.

Tested the styling for reviews, changes, and status updates, in both
expanded and collapsed mode.

Tested these on both mobile and desktop at various widths.

Tested with in-progress changes for user badges.

Checked alignment on the text to make sure everything lined up
correctly.

Summary ID
Switch entry, review, and comment header styling to Flexbox.
The styling for review request entries (reviews, etc.) and comment headers now use Flexbox, which helps us manage styles more consistently and avoids some hacks and workarounds to get things to line up in mobile and desktop modes correctly. This is necessary for some in-progress work on user badges, which impact the layout of this area.
57e582068f5697e52ad715b4d14cbb3a4f6b818f
Description From Last Updated

Can you switch this to #rb-ns-pages.base.on-shell-mobile-mode since .on-mobile-medium-screen-720 is deprecated? Or you can do #rb-ns-ui.screen.on-width-lte(720px, ...) if this really depends …

maubinmaubin
There are no open issues
chipx86
maubin
  1. 
      
  2. Show all issues

    Can you switch this to #rb-ns-pages.base.on-shell-mobile-mode since .on-mobile-medium-screen-720 is deprecated? Or you can do #rb-ns-ui.screen.on-width-lte(720px, ...) if this really depends on the 720 pixel size.

    1. Good points. I was copy/pasting. Updating.

  3. 
      
chipx86
maubin
  1. Ship It!
  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-7.x (1f28568)
Loading...