Feature #29053
closed
Add check/unchek all icon in "Email notifications" section on "My account" page
Added by Go MAEDA over 6 years ago.
Updated over 6 years ago.
Description
"Email notifications" section on "My account" page, you can select which project notifications to receive if you choose "For any event on the selected projects only...".
you cannot select/deselect all projects at once because there is no check/uncheck all icon which you can see various pages in Redmine.
I think also "Email notifications" section should have check/uncheck all icon when users select projects by using checkboxes.
Files
I think that you can realize that feature by adding this one line.
diff --git a/app/views/users/_mail_notifications.html.erb b/app/views/users/_mail_notifications.html.erb
index 7d253f7e8d..9dc4355ec4 100644
--- a/app/views/users/_mail_notifications.html.erb
+++ b/app/views/users/_mail_notifications.html.erb
@@ -18,6 +18,7 @@
) + ' ' + h(project.name)
)
end %>
+ <p><%= check_all_links('notified-projects') %></p>
<%= hidden_field_tag 'user[notified_project_ids][]', '' %>
<p><em class="info"><%= l(:text_user_mail_option) %></em></p>
<% end %>
This is a nice feature for those who need to see many projects.
- Target version set to 3.4.7
Mizuki ISHIKAWA wrote:
I think that you can realize that feature by adding this one line.
[...]
Thank you for posting the patch so quickly.
Setting target version to 3.4.7.
Mizuki ISHIKAWA, maybe it would be better to use a green tick icon which is used on other pages. What do you think about this idea?
Go MAEDA wrote:
Mizuki ISHIKAWA, maybe it would be better to use a green tick icon which is used on other pages. What do you think about this idea?
I suggested #29053#note-1 because I did not have the idea of enclosing with a fieldset.
The green tick icon is used frequently on other screens, so I think that the layout you suggested is better.
By applying the following code, #29053#note-4 layout can be realized.
diff --git a/app/views/users/_mail_notifications.html.erb b/app/views/users/_mail_notifications.html.erb
index 7d253f7e8d..96340dc055 100644
--- a/app/views/users/_mail_notifications.html.erb
+++ b/app/views/users/_mail_notifications.html.erb
@@ -7,6 +7,7 @@
:onchange => 'if (this.value == "selected") {$("#notified-projects").show();} else {$("#notified-projects").hide();}'
) %>
</p>
+<fieldset class="box" id="notified-project-ids"><legend><%= toggle_checkboxes_link("#notified-project-ids input[type=checkbox]") %><%=l(:label_project_plural)%></legend>
<%= content_tag 'div', :id => 'notified-projects', :style => (@user.mail_notification == 'selected' ? '' : 'display:none;') do %>
<%= render_project_nested_lists(@user.projects) do |project|
content_tag('label',
@@ -21,6 +22,7 @@
<%= hidden_field_tag 'user[notified_project_ids][]', '' %>
<p><em class="info"><%= l(:text_user_mail_option) %></em></p>
<% end %>
+</fieldset>
<%= fields_for :pref, @user.pref do |pref_fields| %>
<p>
- Target version changed from 3.4.7 to 4.1.0
I slightly updated the patch in #29053#note-6. The previous patch shows an empty "Projects" box when other than "For any event on the selected projects only..." is selected.
diff --git a/app/views/users/_mail_notifications.html.erb b/app/views/users/_mail_notifications.html.erb
index 7d253f7e8..85842de56 100644
--- a/app/views/users/_mail_notifications.html.erb
+++ b/app/views/users/_mail_notifications.html.erb
@@ -7,7 +7,8 @@
:onchange => 'if (this.value == "selected") {$("#notified-projects").show();} else {$("#notified-projects").hide();}'
) %>
</p>
-<%= content_tag 'div', :id => 'notified-projects', :style => (@user.mail_notification == 'selected' ? '' : 'display:none;') do %>
+<%= content_tag 'fieldset', :id => 'notified-projects', :style => (@user.mail_notification == 'selected' ? '' : 'display:none;') do %>
+ <legend><%= toggle_checkboxes_link("#notified-projects input[type=checkbox]") %><%=l(:label_project_plural)%></legend>
<%= render_project_nested_lists(@user.projects) do |project|
content_tag('label',
check_box_tag(
- Copied to Feature #29080: Add check/uncheck all icon in "Copy" section on copy_project page added
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Target version changed from 4.1.0 to 4.0.0
- Resolution set to Fixed
Committed. Thank you for improving Redmine.
Also available in: Atom
PDF