[WIP] Improved order details page (backend)

[kayue]

Please give me some quick feedback :)

Beside the obvious visual changes, I also added a payment table at the bottom. This could be useful right?

Old

New

[stloyd]

@kayue Nice! I but I'm not sure that showing images is good idea TBH =)

[arnolanglade]

@kayue 👍

[jjanvier]

I like it. But why did you remove the "Manage orders" button and all the details (total, item total, tax total, shipping total) ? It gives a quick overview of the order without having to scroll, don't you think ?

[stloyd]

@jjanvier They are not removed (oh, ok, only that button =)) but I can agree that the order item list should be at the end, IMO it's more important to see details like payment/shipping etc. than contant of order =)

[kayue]

@stloyd The picture is a very useful piece of information for store operator to identify the product. I run a store with over 18,000 cloths naming like this:

• M.V.P. X NEIGHBORHOOD BLACK NHMV.2 “C-TEE.SS.” T-SHIRT
• M.V.P. X NEIGHBORHOOD WHITE NHMV.2 “C-TEE.SS.” T-SHIRT
• M.V.P. X NEIGHBORHOOD NAVY NHMV.WOOL SHIRT

In our case the name doesn't mean much for our operators.

@jjanvier I removed them purely because they are duplicated. Basically I want the order page look similar to a printed invoice.

I have a question: is one order can only have one payment and one shipment? Multiple shipment is not possible at the moment right? If yes I think I can put payment and shipment status to the top easily. Maybe just use two labels.

[kayue]

@stloyd @jjanvier What do you think about the design below?

[kayue]

Any thoughts?

[stloyd]

@kayue I like it, I would just put the labels in Payment/Shipment with pull-right and different colors (this one right now is to bright IMO)

[aniltc]

can we download this changes ?

[aniltc]

It seems you have directly done these changes in sylius.Is it possible to override this page and make the necessary changes in a seprate bundle for my application ?

[stloyd]

@aniltc Please read the docs of Symfony to learn how to do this: http://symfony.com/doc/2.3/cookbook/bundles/inheritance.html, your question has nothing to do with this enhancement.

[aniltc]

Thanks.I am aware of overrding concepts.Here the code chnaged in sylius itself.I would like to know sylius approved thos code and can I use this code ?

[stloyd]

@aniltc Sylius is released with MIT license, so any contributions that are merged into core are being released with same license, so till you hold the original license headers and copyrights in used files (you can edit them of course, by original author details & license/copyrights must be hold), till you can use them within your code.

[kayue]

@stloyd Updated the design.

@aniltc You don't need approval if you don't want to contribute your changes to the Sylius core. In my company's project we do extend the WebBundle, and we only contribute part of our changes back to Sylius's core (for example this one).

[stloyd]

@kayue Looks good for me!
@pjedrzejewski @jjanvier Anything missing here?

[jjanvier]

great for me too 👍

[pjedrzejewski]

Looks very nice, thank you a lot @kayue. @stloyd wsup with this phpspec error on 5.3? Do you know anything?

[Richtermeister]

Pretty! Thanks @kayue