Under Configuration we see 3 menu items that involve configuration management: Import, Export and Sync.
Can we combine them into one?

Patch follows.

Files: 
CommentFileSizeAuthor
#13 Screen Shot 2013-09-24 at 11.26.56 PM.png26.59 KBwebchick
#13 Screen Shot 2013-09-24 at 11.33.20 PM.png15.18 KBwebchick
#13 Screen Shot 2013-09-24 at 11.33.27 PM.png34.79 KBwebchick
#11 2058499-combine-config-menu-items-11.patch13.22 KBBarisW
PASSED: [[SimpleTest]]: [MySQL] 58,848 pass(es).
[ View ]
#11 interdiff-9-11.txt1.49 KBBarisW
#9 interdiff.txt411 bytesmtift
#9 2058499-combine-config-menu-items-9.patch12.49 KBmtift
FAILED: [[SimpleTest]]: [MySQL] 58,518 pass(es), 9 fail(s), and 1 exception(s).
[ View ]
#8 interdiff.txt460 bytesmtift
#8 2058499-combine-config-menu-items-8.patch12.51 KBmtift
FAILED: [[SimpleTest]]: [MySQL] Drupal installation failed.
[ View ]
#7 2058499-combine-config-menu-items-7.patch12.52 KBmtift
FAILED: [[SimpleTest]]: [MySQL] Drupal installation failed.
[ View ]
#5 2058499-combine-config-menu-items-5.patch13.2 KBmtift
PASSED: [[SimpleTest]]: [MySQL] 58,461 pass(es).
[ View ]
#1 drupal-2058499-config-combine-configuration-management-menu-items.patch13.18 KBBarisW
PASSED: [[SimpleTest]]: [MySQL] 57,604 pass(es).
[ View ]
#1 after.png47.54 KBBarisW

Comments

Status:Active» Needs review
StatusFileSize
new47.54 KB
new13.18 KB
PASSED: [[SimpleTest]]: [MySQL] 57,604 pass(es).
[ View ]

Status:Needs review» Needs work

Status:Needs work» Needs review

Bump

Issue tags:+Usability, +Configuration system
StatusFileSize
new13.2 KB
PASSED: [[SimpleTest]]: [MySQL] 58,461 pass(es).
[ View ]

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.

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)

Status:Needs work» Needs review
Issue tags:+Needs usability review
StatusFileSize
new12.52 KB
FAILED: [[SimpleTest]]: [MySQL] Drupal installation failed.
[ View ]

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).

StatusFileSize
new12.51 KB
FAILED: [[SimpleTest]]: [MySQL] Drupal installation failed.
[ View ]
new460 bytes

Oops. Missed a period.

StatusFileSize
new12.49 KB
FAILED: [[SimpleTest]]: [MySQL] 58,518 pass(es), 9 fail(s), and 1 exception(s).
[ View ]
new411 bytes

Gosh darn it

Status:Needs review» Needs work

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

Status:Needs work» Needs review
StatusFileSize
new1.49 KB
new13.22 KB
PASSED: [[SimpleTest]]: [MySQL] 58,848 pass(es).
[ View ]

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

Status:Needs review» Reviewed & tested by the community

Looks like all great stuff to me.

Status:Reviewed & tested by the community» Fixed
StatusFileSize
new34.79 KB
new15.18 KB
new26.59 KB

Before:

After:

Screen Shot 2013-09-24 at 11.33.20 PM.pngScreen 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.