| A handful of suggestions - consider putting initial focus on the item description field and possibly using a textarea over input - in alternating rows with white background it's difficult to determine what can be updated - as this is desktop targeted, I'd consider throwing a hover style over the fields which are in fact editable - rather than trying to rework styling of form elements, use the hover effect as described above and the ng-if (or ng-show/ng-hide) to toggle between the input field and text based element - print mode you can still edit, as noted in another comment there really shouldn't be a print mode, you should have a print stylesheet to hide what you don't want on there and format everything else appropriately - consider localStorage for the most recent invoices created, also helps with the "oh shit, I just navigated off the page with 100 line items" Overall good first effort, keep at it! |