Under Configuration we see 3 menu items that involve configuration management: Import, Export and Sync.
Can we combine them into one?
Patch follows.
Comment | File | Size | Author |
---|---|---|---|
#13 | Screen Shot 2013-09-24 at 11.26.56 PM.png | 26.59 KB | webchick |
#13 | Screen Shot 2013-09-24 at 11.33.20 PM.png | 15.18 KB | webchick |
#13 | Screen Shot 2013-09-24 at 11.33.27 PM.png | 34.79 KB | webchick |
#11 | 2058499-combine-config-menu-items-11.patch | 13.22 KB | BarisW |
#11 | interdiff-9-11.txt | 1.49 KB | BarisW |
Comments
Comment #1
BarisW CreditAttribution: BarisW commentedComment #3
BarisW CreditAttribution: BarisW commented#1: drupal-2058499-config-combine-configuration-management-menu-items.patch queued for re-testing.
Comment #4
BarisW CreditAttribution: BarisW commentedBump
Comment #5
mtiftA 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.
Comment #6
yoroy CreditAttribution: yoroy commentedMakes total sense, lets do this. One tweak for the description:
"Import, export or synchronize your site configuration" (comma instead of slash)
Comment #7
mtiftRerolled 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).
Comment #8
mtiftOops. Missed a period.
Comment #9
mtiftGosh darn it
Comment #11
BarisW CreditAttribution: BarisW commentedIt seems that you've forgotten two menu paths in the test scripts.
Comment #12
Gábor HojtsyLooks like all great stuff to me.
Comment #13
webchickBefore:
After:
Looks great to me! :)
Committed and pushed to 8.x. Thanks!