Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BarisW’s picture

Status: Active » Needs review
FileSize
47.54 KB
13.18 KB

Status: Needs review » Needs work
BarisW’s picture

Status: Needs work » Needs review
BarisW’s picture

Bump

mtift’s picture

A re-rolled patch is attached.

I'm no UX expert, but I like this approach a lot. It removes two items for the main configuration screen and puts all of the core configuration utilities in one area.

We expect to see contributed modules using the the new configuration API and this gives them a place to put their new features.

+1 from me.

yoroy’s picture

Status: Needs review » Needs work

Makes total sense, lets do this. One tweak for the description:

"Import, export or synchronize your site configuration" (comma instead of slash)

mtift’s picture

Status: Needs work » Needs review
Issue tags: +Needs usability review
FileSize
12.52 KB

Rerolled because "pattern" is now "path", and other changes to HEAD. Also changed the punctuation due to #6 and added a serial comma (which is important: http://imgur.com/rb4LaTV).

mtift’s picture

Oops. Missed a period.

mtift’s picture

Gosh darn it

Status: Needs review » Needs work

The last submitted patch, 2058499-combine-config-menu-items-9.patch, failed testing.

BarisW’s picture

Status: Needs work » Needs review
FileSize
1.49 KB
13.22 KB

It seems that you've forgotten two menu paths in the test scripts.

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Looks like all great stuff to me.

webchick’s picture

Status: Reviewed & tested by the community » Fixed
FileSize
34.79 KB
15.18 KB
26.59 KB

Before:

After:

Screen Shot 2013-09-24 at 11.33.20 PM.png

Screen Shot 2013-09-24 at 11.33.27 PM.png

Looks great to me! :)

Committed and pushed to 8.x. Thanks!

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