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

BarisW’s picture

Status:Active» Needs review
FileSize
47.54 KB
13.18 KB
PASSED: [[SimpleTest]]: [MySQL] 57,604 pass(es). View

Status:Needs review» Needs work
BarisW’s picture

Status:Needs work» Needs review
BarisW’s picture

Bump

mtift’s picture

Issue tags:+Usability, +Configuration system
FileSize
13.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.

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

mtift’s picture

FileSize
12.51 KB
FAILED: [[SimpleTest]]: [MySQL] Drupal installation failed. View
460 bytes

Oops. Missed a period.

mtift’s picture

FileSize
12.49 KB
FAILED: [[SimpleTest]]: [MySQL] 58,518 pass(es), 9 fail(s), and 1 exception(s). View
411 bytes

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
PASSED: [[SimpleTest]]: [MySQL] 58,848 pass(es). View

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.