luci-base: cbi: rework ListValue and MultiValue templates

[project/luci.git] / modules / luci-base / luasrc / view / cbi / lvalue.htm

diff --git a/modules/luci-base/luasrc/view/cbi/lvalue.htm b/modules/luci-base/luasrc/view/cbi/lvalue.htm
index ac63e9e..61759fd 100644 (file)
--- a/modules/luci-base/luasrc/view/cbi/lvalue.htm
+++ b/modules/luci-base/luasrc/view/cbi/lvalue.htm
@@ -1,18 +1,42 @@
+<%
+       local i, key
+       local br = self.orientation == "horizontal" and '&#160;' or '<br />'
+%>
+

 <%+cbi/valueheader%>
 <% if self.widget == "select" then %>
 <%+cbi/valueheader%>
 <% if self.widget == "select" then %>

-       <select class="cbi-input-select" data-update="change"<%= attr("id", cbid) .. attr("name", cbid) .. ifattr(self.size, "size") %>>
-       <% for i, key in pairs(self.keylist) do -%>
-               <option id="cbi-<%=self.config.."-"..section.."-"..self.option.."-"..key%>"<%= attr("value", key) .. ifattr(tostring(self:cfgvalue(section) or self.default) == key, "selected", "selected") .. attr("data-index", i) .. attr("data-depends", self:deplist2json(section, self.deplist[i])) %>><%=striptags(self.vallist[i])%></option>
-       <%- end %>
+       <select class="cbi-input-select" data-update="change"<%=
+               attr("id", cbid) ..
+               attr("name", cbid) ..
+               ifattr(self.size, "size")
+       %>>
+               <% for i, key in pairs(self.keylist) do -%>
+                       <option<%=
+                               attr("id", cbid.."-"..key) ..
+                               attr("value", key) ..
+                               attr("data-index", i) ..
+                               attr("data-depends", self:deplist2json(section, self.deplist[i])) ..
+                               ifattr(tostring(self:cfgvalue(section) or self.default) == key, "selected", "selected")
+                       %>><%=pcdata(self.vallist[i])%></option>
+               <%- end %>

        </select>
        </select>

-<% elseif self.widget == "radio" then
-       local c = 0
-       for i, key in pairs(self.keylist) do
-       c = c + 1
-%>
-       <input class="cbi-input-radio" data-update="click change" type="radio"<%= attr("id", cbid.."-"..key) .. attr("name", cbid) .. attr("value", key) .. ifattr((self:cfgvalue(section) or self.default) == key, "checked", "checked") .. attr("data-index", i) .. attr("data-depends", self:deplist2json(section, self.deplist[i])) %> />
-       <label<%= attr("for", cbid.."-"..key) %>><%=self.vallist[i]%></label>
-<% if c == self.size then c = 0 %><% if self.orientation == "horizontal" then %>&#160;<% else %><br /><% end %>
-<% end end %>
+<% elseif self.widget == "radio" then %>
+       <div<%= attr("id", cbid) %>>
+               <% for i, key in pairs(self.keylist) do %>
+                       <label<%=
+                               attr("id", cbid.."-"..key) ..
+                               attr("data-index", i) ..
+                               attr("data-depends", self:deplist2json(section, self.deplist[i]))
+                       %>>
+                               <input class="cbi-input-radio" data-update="click change" type="radio"<%=
+                                       attr("name", cbid) ..
+                                       attr("value", key) ..
+                                       ifattr((self:cfgvalue(section) or self.default) == key, "checked", "checked")
+                               %> />
+                               <%=pcdata(self.vallist[i])%>
+                       </label>
+                       <% if i == self.size then write(br) end %>
+               <% end %>
+       </div>

 <% end %>
 <%+cbi/valuefooter%>
 <% end %>
 <%+cbi/valuefooter%>