On Fri, 2010-10-22 at 18:34 +0200, Jakub Steiner wrote:
- removed all the line noise. Using whitespace as a separator.
- bug: removing a packae seems to remove all
- restructured the styling to be in the #build context
- use links for package names and include a hidden field so the package selection is actually stored.
- Use .live() for package removal and dropdowns
- readd functionality - Add all packages if the collection checkbox is checked.
- show package details when clicking on a name. Always shown when nojavascript
ACK, merged in with Jan's patch and pushed both. Here is a list of things I think are remaining to be cleaned up and/or fixed. I don't think it is very likely that they can all be fixed before the 3:00 Eastern time (US) build cutoff, but if any are quick fixes that could be snuck in I;d be happy to get any that we can.
* Styling for pagination links
* Styling for 'add selected' button
* If you add software when scrolled down to the bottom of list, you can't see that your software was added w/o scrolling back up. Perhaps quick initial fix cloud be to move intine list under 'package selection'?
* If you add a collection, there is no way to 'unadd' the whole collection', at least that I can see
* It is not clear how to close the 'detail box' when you click on an item in the inline list of packages. The first couple times, I hit the 'x', thinking it was close, when it was actually 'delete'.
* Clicking off of the detail box to close it only works if you are in the main content area. If you try, for instance, to click in the whitespace to the left, it does not close the box.