Closed (fixed)
Project:
Commerce Core
Version:
7.x-1.x-dev
Component:
User experience
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Apr 2011 at 18:44 UTC
Updated:
18 Nov 2011 at 20:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
rszrama commentedAnother option would be for the edit link from the product View to have a ?destination=admin/commerce/products query string. I've thought about doing that, but because those links are generated by the local actions system in the menu I wasn't sure how to go about it easily.
Comment #2
hunziker commentedI create a patch that implements a combination of both approaches. I add a "destination" parameter to all operations links.
Then I add a button to the editing form of the product. If the user click this button, the destination is reset. So the customer is not redirect back to the product listing.
Comment #3
Bojhan commentedUX improvements require screenshots to be reviewed.
Comment #4
hunziker commentedHere is a screenshot of the buttons. The rest is hard to demonstrated as screenshot.
Comment #5
Bojhan commentedI think its good to just show the "flow" you changed with more full-sized screenshots. The buttons themselves are hard to review, other than why do we handle this usecase? We dont do that for content editing? I am a bit confused which button one would choose to go back to overview.
Comment #6
hunziker commentedHere are more screenshots.
Comment #7
rszrama commentedYeah, I may just be down with using the destination link without requiring the "continue edit" button, as once you've made changes, there really isn't any reason to stay on this form.
Comment #8
hunziker commentedok. Then add only the changes in commerce_product_ui.module.
Comment #9
rszrama commentedAlrighty, I started with that but ultimately moved the destination setting into the Views field handler for product operations links. I'm not sure where else product contextual links might be used and didn't want to accidentally add destinations to all of them. I updated the default View to have the option set by default.
Commit: http://drupalcode.org/project/commerce.git/commitdiff/4e4f654