Okay, is this the last point on your list? I will rewrite that and
retest then.

LZ

On Mon, Dec 16, 2013 at 03:07:38AM -0800, Dominic Cleal wrote:
> > @@ -1,6 +1,21 @@
> >  <div class="fields">
> >    <%= selectable_f f, :pool_name, compute_resource.storage_pools.map(&:name), { }, :class => "span2", :label => _("Storage Pool") %>
> > -  <%= text_f f, :capacity, :class => "input-mini", :label => _("Size (GB)") %>
> > -  <%= select_f f, :format_type, %w[RAW QCOW2],:downcase, :to_s, { }, :class => "span2", :label => _("Type"),
> > +  <%= text_f f, :capacity, :class => "input-mini", :label => _("Size (GB)"), :onchange => 'capacity_edit(this)' %>
> > +  <%
> > +    active = 'Size'
> > +    active = 'None' if f.object.allocation.starts_with? '0'
> > +    active = 'Full' if f.object.allocation == f.object.capacity
> 
> @lzap readonly shows as a greyed out field in both Firefox and Chrome, seems to be exactly what you need here
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/theforeman/foreman/pull/1073/files#r8363662

-- 
Later,

 Lukas "lzap" Zapletal
 irc: lzap #theforeman