WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#15751 closed defect (bug) (fixed)

Improve string for no themes in site admin for multisite

Reported by: ocean90 Owned by:
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.1
Component: Themes Keywords: needs-patch
Focuses: Cc:

Description

http://core.trac.wordpress.org/browser/trunk/wp-admin/includes/class-wp-themes-list-table.php#L66

If you have more themes installed but not enabled for this site, the string doesn't tell the truth, so maybe add a link to themes.php or site-themes.php?id=x. Also it's not translated.



Some examples from nacin:

printf( __( 'You only have one theme activated for this site right now. Visit the Network Admin to <a href="%1$s">activate</a> or <a href="%2$s">install</a> more themes.' ), network_admin_url( 'site-themes.php?id=' . $GLOBALS['blog_id'] ), network_admin_url( 'theme-install.php' ) );

---

You only have one theme enabled for this site right now. Visit the Network Admin to <a href="%1$s">enable</a> or <a href="%2$s">install</a> more themes.

Change History (1)

comment:1 nacin3 years ago

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

(In [16838]) Adjust branching and errors in WP_Themes_List_Table::no_items(). fixes #15751.

Note: See TracTickets for help on using tickets.