Finalize All Member Notices admin screen

[mercime]

1. Current screen has an "Add New Notice" that's an empty link and the New Notice Form is already visible on screen above the table with the list of Notices. Question is, shall we remove the "Add New Notice" empty link and make it the heading for the New Notice Form or was there a plan to make another admin screen with the "Add New Notice" link?
2. Close Form tag needs.
3. Replace table markup of the New Notice form.

Patch coming up for issues 2 & 3 along with before and after screenshots.

[dcavins]

Did I break this when I moved it out of Nouveau, or are these further improvements that you're making to what exists? Either way, I'm in favor, and please let me know if you need assistance or testing or whatever. :)

[mercime]

Thanks @dcavins! I'll take you up on that offer of course. Great idea by the way to move the All Messages admin to core :D

The empty link and table layout were there before Nouveau merge. Left it alone at that time because I thought there was going to be some other elements to be added on before BP 3.0 rolls out.

Since we're close to the finish line for this dev cycle, it would great to get feedback for the final set up of this screen. If there's nothing else that's going to be added/removed, I propose we have the following structure:

<h1> - All Member Notices
( - ) delete Add New Notice empty link
( + ) <p>Manage all site-wide notices</p> etc
( + ) <h2> - Add New Notice
bp-panel - New notice form
( + ) <h2> - Notices List
<table>
( + ) table <caption> Set up column visibility at the Screen Options panel above.</caption>

For review/comments. Thanks.

[DJPaul]

Your comments are good @mercime and the improvements even better!

I don't think we need the "cancel notice" button -- seems unnecessary.
Would also suggest renaming "Save notice" to "Publish notice" to better match terminology on WP-Admin/Edit Post -- to hopefully hint that it goes live immediately, vs. saving a draft.

Let's rename the actual page / menu item away from "All Member Notices". I was going to do this anyway, we might as well group it with this change:

• We would need to use "Users" not "Members" to fit in with all the other WordPress menu items in this section.
• But, perhaps "Site Notices" is adequate.

?

[mercime]

Thanks @djpaul :D Attached are 2 patches and a screenshot showing two versions for comments/review.

[DJPaul]

Looks fine!

[mercime]

In 11910:

Finalize the new Site Notices admin screen.

Props mercime, djpaul.
Fixes #7721.