Current behavior:

  • install latest 7.x-dev tarball
  • navigate to admin/appearance on a standard D7.dev installation
  • theme list does not include any link other than settings for default theme ( see attached normal.png )

Proposed behavior:

  • theme list shows all links for all themes, but have links disabled for default theme, and have appropriate title text to denote that the links are meaningless for the default theme. ( see attached proposed.png )

This basically adds two disabled items next to theme settings:
"Disable", with title text "Cannot disable default theme", and
"Set Default", with title text "Already default"

CommentFileSizeAuthor
proposed.png130.17 KBhexblot
normal.png127.47 KBhexblot
system.admin_.inc-theme_list_default_links.patch756 byteshexblot

Comments

eustace’s picture

eustace commented
Status: Needs review » Reviewed & tested by the community

Patch applies as proposed. Does not change program logic or behavior but added disabled links adds information for consistency and completeness. Thx

RTBC

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Reviewed & tested by the community » Needs review
Issue tags: -D7UX usability +Needs usability review

Requires a review of the UX team.

Bojhan’s picture

Bojhan commented
Issue tags: -Needs usability review

I dont think this adds value, we don't show disabled links elsewhere?

mgifford’s picture

mgifford
he/him
Primary language English
commented
Assigned: hexblot » Unassigned
Issue summary: View changes
Status: Needs review » Closed (won't fix)

Sounds like a won't fix rather than needs review.