Defect #11
Minor rendering problem within Safari at admin/mail_options...
| Status: | Closed | Start: | ||
| Priority: | Normal | Due date: | ||
| Assigned to: | - | % Done: | 0% |
|
| Category: | - | |||
| Target version: | - | |||
| Affected version: | Resolution: | |||
Description
In views/admin/mail_options.rhtml :
Added a self-clearing DIV so that fieldset border actually contains the checkboxes:
Original:
<% actions.keys.sort.each do |group_id| %>
<fieldset style="margin-top: 6px;"><legend><strong><%= l(Permission::GROUPS[group_id])
%></strong></legend>
<% actions[group_id].each do |p| %>
<div style="width:170px;float:left;"><%= check_box_tag "action_ids[]", p.id, p.mail_enabled?
%>
<%= l(p.description.to_sym) %>
</div>
<% end %>
</fieldset>
<% end %>
Changed:
<% actions.keys.sort.each do |group_id| %>
<fieldset style="margin-top: 6px;"><legend><strong><%= l(Permission::GROUPS[group_id])
%></strong></legend>
<% actions[group_id].each do |p| %>
<div style="width:170px;float:left;"><%= check_box_tag "action_ids[]", p.id, p.mail_enabled?
%>
<%= l(p.description.to_sym) %>
</div>
<% end %>
<div style="clear: both;"></div>
</fieldset>
<% end %>
It's the simple <div style="clear: both"></div> that's been added.
It's a small thing. ;) Thanks for this cool, cool project!
History
2007-03-06 17:28 - Jean-Philippe Lang
Thanks for your submission.
Do you have the same problem on the role permissions screen ?
because it's the same kind of layout.
Regards,
Jean-Philippe
2007-03-06 17:32 - Chris Grant
This problem is present within the role permissions screen, as
well. I looked
around to find if there were similar problems and I guess I missed
this screen.
You can add the self-clearing DIV to that layout, as well.