Project

General

Profile

Actions

Defect #11

closed

Minor rendering problem within Safari at admin/mail_options...

Added by Chris Grant about 17 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Affected version:

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!


Files

mail_options.rhtml (765 Bytes) mail_options.rhtml Chris Grant, 2008-02-03 10:48
_form.rhtml (729 Bytes) _form.rhtml Chris Grant, 2008-02-03 10:48
Actions #1

Updated by Jean-Philippe Lang about 17 years ago

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

Actions #2

Updated by Chris Grant about 17 years ago

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.

Actions #3

Updated by Chris Grant about 17 years ago

I've made the appropriate change in views/roles/_form.rhtml and
uploaded the
fixed file. Sorry for not knowing how to submit a formal patch
-- will have to
look into how to do that. Thanks again for this project, it's
great!

Actions #4

Updated by Jean-Philippe Lang about 17 years ago

thanks, i've committed the fix in the repository.

Actions

Also available in: Atom PDF