[Toaster] [review-request] dreyna/IMAGE_INSTALL_append_space_7394

Reyna, David david.reyna at windriver.com
Thu Mar 12 17:17:25 PDT 2015


Hi Belén,

I have completed what I think is the minimal implementation for this fix.

    dreyna/IMAGE_INSTALL_append_space_7394

  * It insures that any non-empty value update has a space prefix.
  * It insures that any non-empty value in the edit box has a space prefix, to guide the user.
  * I also caught an edge case, where I now suppress the "Save" button when only spaces are entered
  * There is also a white space fix

Note:
  * I did not force the leading space in this value to show in the page (i.e. I let the HTML trim it). It would visually offset the value by one position from the other values, and it would require us to do " " substitutions, and I thought that the added complexity not worth the effort. If you really want it I could do either "<pre></pre>" or use the template "{{ image_install_append|replace:"/ / " }}"

- David


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/toaster/attachments/20150313/bc6072ab/attachment.html>


More information about the toaster mailing list