notices-loop

[danbp]

BP 2.4 rc1
Twenty Sixteen

• the notice delete message button use x instead of Delete Message.

This x is in the same column as activate/deactivate status and isn't pertinent (at least to me) when it comes as second line under the action name. Should at least be in a separate column if the aim is to gain some place. That said i ignore why this part is styled like this, as the similar one for message management is slightly different, and much better.

The template itself looks awful on Twenty Sixteen and contains some useless tags like strong and a mysterious empty td of 1% !

Here the code with real content of what i'm speaking about.

<tbody><tr id="notice-2" class="">
   <td width="1%"></td>
      <td width="38%">
	<strong>This is a notice to all users.</strong>
	<p>This is a notice test to all users.</p>
      </td>
      <td width="21%">
       <strong></strong>
       <span class="activity">Sent: 2 November 2015 at 17 h 44 min</span>
      </td>
      <td width="10%">
       <a class="button" href="http://wpbeta/members/daniel/messages/notices/deactivate/2/?_wpnonce=227dc95ecc">Deactivate</a>
       <a class="button" href="http://wpbeta/members/daniel/messages/notices/delete/2?_wpnonce=ded41a06e5" title="Delete Message">x</a>
      </td>
   </tr>
</tbody>

Concerned file is ​notices-loop

[danbp]

notice loop on twenty sixteen

[danbp]

Note that the above described issue doesn't apply to 2014 and 2015, (text on button is correct) but is again in 2013, 2012 et 2011.

[imath]

@hnla could you look at it, i think the layout for twentysixteen could be improved :)

[hnla]

@danbp Thanks for the heads up I'm looking into it.

@imath & @danbp Part of the problem is that I don't see the same issue at all and see some missing elements, Dan are you templates fully up to date they aren't overloaded?

[hnla]

Scrub that was looking at notifications screen, instead of messages *sigh*

Haven't a clue why that looks so bad, but sadly looks like some work to fixup.

Any patches gratefully accepted :)

[danbp]

My templates are daily updated on my local beta test site. ;-)

[hnla]

This is the widescreen view of a re-factored set of styles to counter the incorrect % widths and inherited display and float values.

[hnla]

Same layout at mobile/small screen view

[hnla]

With a few more minor tweaks this will probably be the best we can do with this for the moment the markup really needs to be updated.

My templates are daily updated on my local beta test site. ;-)

Yep sorry I was just being very slow :)

[hnla]

Revising the updated corrections to use the styles for message threads from the previous companion styles which maintains the same styles for sitewide-notices as message-threads, in porting styles across that were missed.

I'll be committing and closing this shortly.

[hnla]

Final revision - in keeping with the styles for message-threads on other companion sheets

[hnla]

In 10325:

Twentysixteen Companion Styles correction

Update corrects an issue where message-thread sitewide-notices table styles not ported over for twentysixteen.

Fixes #6710 Props danbp