The new theme admin page at admin/appearance looks great, but has regressed functionality. Installing a new theme now requires two steps on this page: enable THEN set default.

CommentFileSizeAuthor
#1 650622.drupal.theme-system-enable-and-set-default.patch891 bytesjoachim
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim’s picture

joachim CreditAttribution: joachim commented
FileSize
650622.drupal.theme-system-enable-and-set-default.patch891 bytes

Patch adds an 'Enable and set default' on disabled themes.

joachim’s picture

joachim CreditAttribution: joachim commented
Status: Active » Needs review

Oops. Hi bot!

Bojhan’s picture

Bojhan CreditAttribution: Bojhan commented

This was in the original mockup, and existed in the Apperance page patch until comment #116 - this was obviously a mistake.

http://drupal.org/node/491214#comment-2312944

Bojhan’s picture

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

Putting this to RTOBC, I can see why yoroy decided not to - as aesthetically it looks more pleasing to have just one link there. But given the usecase of this whole area of the page, it makes far less sense to do so.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Yeah, agreed that we do need this even though it's a bit more clutter-y. At least it's far less confusing than checking a box and choosing a radio as you had to in D6 and below...

Committed to HEAD.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.