On 06/28/2011 01:46 PM, jprovazn(a)redhat.com wrote:
From: Jan Provaznik<jprovazn(a)redhat.com>
Added class "checkbox_table" to mapping lists (this class is required for
using
global delete confirm hook). Class name is little bit confusing, but this view
will be reworked anyway.
---
src/app/views/realms/_mapping.haml | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/app/views/realms/_mapping.haml b/src/app/views/realms/_mapping.haml
index dea5cda..f89cc8b 100644
--- a/src/app/views/realms/_mapping.haml
+++ b/src/app/views/realms/_mapping.haml
@@ -5,7 +5,7 @@
- unless @backend_provider_targets.empty?
%h2 Realm '#{(a)realm.name}' is mapped to providers:
- %ul
+ %ul{:class => 'checkbox_table'}
- @backend_provider_targets.each do |backend|
%li
%input{:name => "id[]", :type => "checkbox",
:value => backend.id, :id => "backend_id_#{backend.id}"}
@@ -13,7 +13,7 @@
- unless @backend_realm_targets.empty?
%h2 Realm '#{(a)realm.name}' is mapped to realms:
- %ul
+ %ul{:class => 'checkbox_table'}
- @backend_realm_targets.each do |backend|
%li
%input{:name => "id[]", :type => "checkbox",
:value => backend.id, :id => "backend_id_#{backend.id}"}
I checked
BZ and I saw that you wrote that bug is fixed by commit
0ca429ae536e1d15b9b2f480381e96a433f790d8 (Jirka's patch Ensure that
tables are cleanly styled.), so I suppose that this your patch is obsolete.
NACK.