follow on from issue #1859152: Display stock availability as message @Albert Volkman comments:

  • The table should utilize theme_table() instead of manually creating the markup. see comment #1
  • Look into using a dynamic way instead of a fixed set of 5 options?

Comments

MarcElbichon’s picture

MarcElbichon commented

I've tried to use theme_table rather than markup but there is a problem with this (http://drupal.stackexchange.com/questions/27264/drupal-form-themed-as-ta...)
but even with that, settings are not saved because form needs [display][row][field] structure.
I think this is better to use markup.

guy_schneerson’s picture

guy_schneerson commented

Thanks @MarcElbichon its not a real issue anyway happy for it to sty the way it is

guy_schneerson’s picture

guy_schneerson commented
Issue summary: View changes

updated to do

Prashant Singhal’s picture

Prashant Singhal commented
Issue summary: View changes
Prashant Singhal’s picture

Prashant Singhal commented
Title: Imrove code in Commerce Stock Decimal formatter » Improve code in Commerce Stock Decimal formatter