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

Patch follows.

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
#11 interdiff-9-11.txt1.49 KBBarisW
#9 interdiff.txt411 bytesmtift
#9 2058499-combine-config-menu-items-9.patch12.49 KBmtift
#8 interdiff.txt460 bytesmtift
#8 2058499-combine-config-menu-items-8.patch12.51 KBmtift
#7 2058499-combine-config-menu-items-7.patch12.52 KBmtift
#5 2058499-combine-config-menu-items-5.patch13.2 KBmtift
#1 drupal-2058499-config-combine-configuration-management-menu-items.patch13.18 KBBarisW
#1 after.png47.54 KBBarisW

Comments

BarisW’s picture

BarisW commented
Status: Active » Needs review
StatusFileSize
new after.png47.54 KB
new drupal-2058499-config-combine-configuration-management-menu-items.patch13.18 KB

Status: Needs review » Needs work

The last submitted patch, drupal-2058499-config-combine-configuration-management-menu-items.patch, failed testing.

BarisW’s picture

BarisW commented
Status: Needs work » Needs review

#1: drupal-2058499-config-combine-configuration-management-menu-items.patch queued for re-testing.

BarisW’s picture

BarisW commented

Bump

mtift’s picture

mtift
he/him
Primary language English
Location Minnesota, USA
commented
Issue tags: +Usability, +Configuration system
StatusFileSize
new 2058499-combine-config-menu-items-5.patch13.2 KB

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

yoroy commented
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

mtift
he/him
Primary language English
Location Minnesota, USA
commented
Status: Needs work » Needs review
Issue tags: +Needs usability review
StatusFileSize
new 2058499-combine-config-menu-items-7.patch12.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

mtift
he/him
Primary language English
Location Minnesota, USA
commented
StatusFileSize
new 2058499-combine-config-menu-items-8.patch12.51 KB
new interdiff.txt460 bytes

Oops. Missed a period.

mtift’s picture

mtift
he/him
Primary language English
Location Minnesota, USA
commented
StatusFileSize
new 2058499-combine-config-menu-items-9.patch12.49 KB
new interdiff.txt411 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

BarisW commented
Status: Needs work » Needs review
StatusFileSize
new interdiff-9-11.txt1.49 KB
new 2058499-combine-config-menu-items-11.patch13.22 KB

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

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs review » Reviewed & tested by the community

Looks like all great stuff to me.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed
StatusFileSize
new Screen Shot 2013-09-24 at 11.33.27 PM.png34.79 KB
new Screen Shot 2013-09-24 at 11.33.20 PM.png15.18 KB
new Screen Shot 2013-09-24 at 11.26.56 PM.png26.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.