Skip to:
Content

BuddyPress.org

Opened 5 months ago

Closed 4 months ago

#8357 closed defect (bug) (fixed)

Make the BP Tools administration screen more consistent with WP Admin appearance

Reported by: imath Owned by: slaFFik
Milestone: 7.0.0 Priority: low
Severity: minor Version: 2.0
Component: Administration Keywords: has-patch
Cc:

Description

Once a tool is performed and the notice message id displayed, we see some lack of consistency with the WordPress administration general appearance:

  1. The notice message does not used the width you could find in other administration screens.
  2. This notice message also has a bigger height, giving the impression the dismissible button (x) is not vertically aligned.
  3. Moreover, it's minor but, the color of the information above the "Repair tools" form (which seems to be an important advice) is lighter than the rest of the text.

I suggest the attached patch to fix the issue. I'm also adding some screenshots (without the patch & with the patch applied).

Attachments (3)

8357.patch (4.3 KB) - added by imath 5 months ago.
tools-preview-02.png (350.3 KB) - added by imath 5 months ago.
Without
tools-preview-04.png (374.5 KB) - added by imath 5 months ago.
With the patch applied

Download all attachments as: .zip

Change History (7)

@imath
5 months ago

@imath
5 months ago

Without

@imath
5 months ago

With the patch applied

#1 @imath
5 months ago

  • Milestone changed from Awaiting Review to 7.0.0

This ticket was mentioned in Slack in #buddypress by imath. View the logs.


4 months ago

#3 @espellcaste
4 months ago

Nice work! All good to me! :)

#4 @imath
4 months ago

  • Resolution set to fixed
  • Status changed from new to closed

In 12739:

Make the BP Tools Admin screen more consistent with WP Admin ones

  • the notice message now uses the same width & height than in other administration screens.
  • improve the visibility of the advice about not running multiple tools at the same time.
  • edit the HTML markup to use existing WordPress styling and save some code lines into our common.css file.

Fixes #8357

Note: See TracTickets for help on using tickets.