-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[RFC] Using tabs for channel pricing #11529
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For me, it should be merged ASAP :D
src/Sylius/Bundle/AdminBundle/Resources/views/Product/Tab/_details.html.twig
Show resolved
Hide resolved
abba513
to
2ae8bce
Compare
Hey Max, can you rebase once again? Tests are probably failing due to bc break at Symfony end (see #11542) |
src/Sylius/Behat/Context/Ui/Admin/ManagingProductVariantsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Context/Ui/Admin/ManagingProductVariantsContext.php
Outdated
Show resolved
Hide resolved
e2429ae
to
918e7f1
Compare
…xt.php Co-authored-by: Łukasz Chruściel <lchrusciel@gmail.com>
I rebased the feature on a working master. So if it breaks now this should be my fault. |
One thing I am not sure about is the message about the product being not active in this channel. Which would in turn also solve the issue #11559 . |
Thanks, @mamazu! 🎉 |
According to your doubts, let's iterate over it on the master. It already looks much better :D |
Proposal
Channel prices in a simple product get displayed on the product page under the list of channels. So I propose that they will be made into a tabbed view so that they are more easy to manage. If you have a lot of channels (where the product is not active for example) you have a lot of values to scroll through.
Before
After