Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Current look: https://drupalcommerce.org/sites/default/files/product-form.png
Initial feedback is that the store selector is too prominent at the top of the left column.
Since it's "meta" information, it would make sense to move it to the right column.
Ryan later found out that Shopify does exactly this: http://www.ugmonk.com/wp-content/uploads/2015/11/shopify.jpg
So we'd want a Visibility fieldset between the Published part and "Url path settings", expanded by default.
We also need to make sure (via #pre_render callback?) to hide the fieldset if the store selector itself is hidden.
UX questions:
1) Is it "Visibility" or "Visibility settings"? Or "Store visibility" We might want to think about all of the labels on the right.
2) Can we/should we hide the title of the store selector?
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | stores-sidebar2.png | 154.58 KB | bojanz |
| #5 | stores-sidebar1.png | 143.56 KB | bojanz |
| #5 | 2626972-5-move-store-selector.patch | 3.07 KB | bojanz |
Comments
Comment #2
ArtusamakComment #3
ArtusamakComment #4
JulienD CreditAttribution: JulienD as a volunteer commentedI've moved the store field widget to the right column and applied the same rule than the path field widget to hide or not the field. I based the control by checking if the store field #access property is set to True or not.
Ux questions remarks:
At the beginning "Store visibility" made more sens to me, but after thinking to it, I have a preference for "Visibility settings". It is a store option and we need to indicate clearly that checking or not an option will display the product on the selected store. "Visibility" alone does not make a lot of signification. I don't think removing the store title of the field widget is a good choice because this field is mandatory and we will need to handle the asterisk in an other way, resulting in writing complex code for not a lot a benefits.
Comment #5
bojanz CreditAttribution: bojanz at Centarro commentedTweaked Julien's patch to hide the fieldset when the store widget is hidden (that was a tricky one!).
Attaching screenshots.
Comment #7
bojanz CreditAttribution: bojanz at Centarro commentedCommitted, thanks!