Clean up classes and styles for the admin sidebar.

Review Request #10407 — Created Feb. 15, 2019 and submitted — Latest diff uploaded

Information

Review Board
release-4.0.x

Reviewers

The admin sidebar was kind of a mess. It used the class names and styles
from the admin widgets, but then did a bunch of overrides and other
element-specific styles. This change breaks it apart and makes it so that
future redesign work will be much easier.

This involves several pieces:

  • Renaming the template so it doesn't claim to be a widget.
  • Switching away from the .admin-widget class and associated
    sub-element classes, and moving towards our new CSS naming standards.
  • Defining new rules for the new classes, bringing across only those
    things which are necessary for the look of the sidebar.
  • Eliminating style rules specific to each section of the sidebar and
    making those general.

There is one visual change in here, removing the text from the "Add"
links within the "Manage" section. This text really doesn't add anything
to the user experience, and it cluttered up both the visual appearance
and the style rules. I've moved this into a tooltip instead (and changed
it to say "Add new").

Verified the visual appearance and behavior of the admin sidebar.

Commits

Files

    Loading...