Add theme update counter in admin theme menu item

[mukesh27]

Add theme update counter in admin theme menu item as we show for plugin menu item.

[joyously]

Why does your code use the update-plugins capability and class name?

This used to work. Can you look back at older versions for that code?

[mukesh27]

updated patch

[mukesh27]

ohhh, i have updated new patch

[SergeyBiryukov]

#52425 was marked as a duplicate.

[zodiac1978]

@joyously I think using update-plugins as class name is not very intuitive, but at the moment correct.

The network menu is using it too:
​https://github.com/WordPress/WordPress/blob/b3b8942dfcb451eddb5559b63c1043fce5d9449e/wp-admin/network/menu.php#L65

We could either use it here too or add the class name to the CSS, because at the moment it does not exist:
​https://github.com/WordPress/WordPress/search?q=update-themes

@mukesh27 Looking at the patch and the screenshot it shows an inconsistent behavior: The badge is added to the first submenu item instead of the main menu item ("Appearance"), like it is seen for the "Plugins" menu item.

I would vote for using the same design as for Plugins (which is also used in a multisite):
​https://github.com/WordPress/WordPress/blob/b3b8942dfcb451eddb5559b63c1043fce5d9449e/wp-admin/network/menu.php#L59-L79

[zodiac1978]

Updated patch. Count is added to "appearance" now and is using the correct class names and caps.

[zodiac1978]

New screenshot showing the badge

[zodiac1978]

I am still experiencing a strange behavior. There is a paid theme in my test environment with available update, but the notice "Automatic update is unavailable for this theme."

After leaving the themes.php page the count is changing from 7 to 6. On all admin pages it stays on 6. Just on update-core.php it is showing 7 again.

[zodiac1978]

Corrected wrong comment text.

[Boniu91]

What to test:

1. If it displays the notification correctly
2. If it's removed after the update
3. Check it on the PHP8
4. Compare the number with the number inside the Dashboard > Updates
5. Check the multisite
6. Check the display on mobiles
7. Confirm that it works with Paid Themes

[francina]

Tested

• MacOS 11.3.1
• Safari 14.1
• WordPress 5.8-alpha-50427-src

1. Rolled back GeneratePress free and Twenty Twenty-one to previous versions
2. Applied the patch
3. The notification now appears on the Appearance menu
4. The number of updates matches in:

• Dashboard > Updates
• Appearance
• Horizontal bar

Updated the themes

5. The notification disappears once I update the themes

[francina]

Notification appears and matches where there is an update notification

[sannevndrmeulen]

Test feedback:

• MacOS 10.14.6
• Google Chrome 90.0.4430.93
• WordPress 5.8-alpha-50821

1. Once I got the patch working I see that the notification is displayed behind Appearance.
2. When I update a theme the notification updates too, lowering the number of remaining themes that require an update.
3. -
4. The number behind Appearance matches the number shown for Dashboard -> Updates.
5. -
6. Same behaviour as in step 1, 2, and 4. However, I did notice that the placement of the notification bubbles looks slightly different on mobile.
7. -

Steps that still require testing:

• Check it on the PHP8
• Check the multisite
• Confirm that it works with Paid Themes

[zodiac1978]

Replying to sannevndrmeulen:

Steps that still require testing:

• Check it on the PHP8
• Check the multisite
• Confirm that it works with Paid Themes

Thanks at you all for testing the patch! @francina @sannevndrmeulen

I will try to check this with PHP 8. But as this code is mainly adopted from the network version this should work fine.

Multisite is already showing this badge, because it uses another code. This code just runs on single sites.

The main reason for the needs-testing tag was my experience with paid themes (see comment above).

It would be great if someone can try to test this patch with a paid theme which needs an update and tries to reproduce the described problem.

[zodiac1978]

Check on PHP 8 is positive. Works without problems.

Updates on theme page, theme overview and update core page are working and the count is changed accordingly after update.

[SergeyBiryukov]

Replying to zodiac1978:

Looking at the patch and the screenshot it shows an inconsistent behavior: The badge is added to the first submenu item instead of the main menu item ("Appearance"), like it is seen for the "Plugins" menu item.

Thanks for the refreshed patch! I think it looks more consistent with the counter next to the "Themes" item though. Otherwise, when it's next to "Appearance", it's not quite clear what it refers to: themes, widgets, menus, etc.

So I would suggest going with the original approach here.

[SergeyBiryukov]

In 51022:

Themes: Display the number of available theme updates in the admin menu.

This brings some consistency with the similar update counter for plugins.

Props mukesh27, zodiac1978, pixolin, Boniu91, francina, sannevndrmeulen, joyously, SergeyBiryukov.
Fixes #43697.

[zodiac1978]

Replying to SergeyBiryukov:

Thanks for the refreshed patch! I think it looks more consistent with the counter next to the "Themes" item though. Otherwise, when it's next to "Appearance", it's not quite clear what it refers to: themes, widgets, menus, etc.

So I would suggest going with the original approach here.

I would highly recommend to rethink this decision.

1. The submenu "Themes" (and therefore the count badge) is just shown if the menu is opened. For a quick visual hint this is not useful IMHO. Showing the badge is meant to provide a visual reminder to update. If not seen, it does not work in this way.

2. It is a contradiction to the behavior in the network admin.

3. I don't know any case in which there is badge on the other items (widgets, menus, etc.) - if there are in the future, we could add them up to show an overall count on "Appearance" (and not just show waiting theme updates)

[SergeyBiryukov]

Replying to zodiac1978:

It is a contradiction to the behavior in the network admin.

I would say the inconsistency is not necessarily in the counter placement, but rather in that the network admin menu does not have an Appearance item, and has the Themes item at the root level instead. Currently, both menus have the counter next to Themes item, which seems consistent enough to me.

I don't know any case in which there is badge on the other items (widgets, menus, etc.) - if there are in the future, we could add them up to show an overall count on "Appearance" (and not just show waiting theme updates)

As experienced users, we know that under the Appearance menu only themes support updates, but for those who don't have a lot of experience yet, I think it might be confusing as to what the counter next to the Appearance menu item is supposed to mean, and what they are supposed to do to remove it.

I would suggest bringing up the ticket at one of the ​Design team meetings to see how others feel about the counter placement, and we can iterate on it with more feedback :)

[zodiac1978]

Replying to SergeyBiryukov:

Replying to zodiac1978:
I would say the inconsistency is not necessarily in the counter placement, but rather in that the network admin menu does not have an Appearance item, and has the Themes item at the root level instead. Currently, both menus have the counter next to Themes item, which seems consistent enough to me.

You are absolutely right. This eliminates point 2. Still thinking about point 1 though ...

As experienced users, we know that under the Appearance menu only themes support updates, but for those who don't have a lot of experience yet, I think it might be confusing as to what the counter next to the Appearance menu item is supposed to mean, and what they are supposed to do to remove it.

Yes, this makes sense, too. And showing the badge on both items seems weird.

I would suggest bringing up the ticket at one of the ​Design team meetings to see how others feel about the counter placement, and we can iterate on it with more feedback :)

It is not easy for me to attend those meetings. I've added the "needs-design-feedback" and the "2nd-opinion" tag to open the discussion a bit more.

Would be great to get some more voices. Maybe I am overthinking this here 😅

I am totally fine with keeping it on the "Themes" item. Just want to be sure that this decision is based on a good foundation.