stayradiated/Nitro — pull request #125 by mlms13 (2012-07-23 11:21:40+00:00)
Title: Minor sorting and interface-related fixes

These changes are small enough that I decided to group them in a single pull request, but if you'd rather review/pull them individually, I can make that happen.

When adding a new task, or editing the date/priority of an existing task, the list wasn't re-sorted automatically, so the newly created/edited task might be out of order. Commit 7325edd fixes that by refreshing the list when a task is saved. In a perfect world, this re-ordering would be animated so that you can visually see where a task is moving. That would be a lot more work, though, and this change is still an improvement even without animation, imo.

995895f is pretty self-explanatory. With the sort order set to "Due Date", this change uses "priority" as a secondary criteria, putting higher priority tasks ahead of lower priority tasks when they have the same due date.

The final change is purely cosmetic. The 40px of padding below each `ul` in the task list could cause unnecessary scroll bars in the task list if the list was within 40px of the bottom of the viewing area. This change reduces that number to 20px (only for the last `ul` in the list), which matches the padding on the right.