#18 Merged
Repository
Deleted repository
Branch
default (d2bb123e122d)
Repository
diefenbach/django-lfs-0.7 django-lfs-0.7
Branch
default

some fixes and improvements

Author
  1. Radim Novotny avatarRadim Novotny
Reviewers
Description
No description

Comments (3)

  1. Kai Diefenbach repo owner

    Thanks for the pull request!

    I have some concerns:

    when we add the image to the order detail page (I'm not sure whether we should at all on the management interface), the SKU should have column of its own and the other columns of a row should be vertical aligned to the top.

    I actually don't like the different colors per state on the order overview, as the state is already displayed within an own column, but I like the idea to have a class per state, so that integrators are able to provide there own css if they want.

    What do you think?

  2. Radim Novotny author

    If it is not a major problem I'd like to leave the product image in the management interface. It helps me to make quick overview of the ordered products. I'll add another column for the image and set the vertical alignment.

    I will remove the provided CSS for order overview from the lfstheme.

Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.