Commit dd93079a authored by Phil Hughes's avatar Phil Hughes

Changed helper method to check for none on params

Moved if statements around in view
parent 0bc9008e
......@@ -62,7 +62,7 @@ module TodosHelper
end
def todos_filter_empty?
todos_filter_params.all? {|key, value| value.nil?}
todos_filter_params.values.none?
end
def todos_filter_path(options = {})
......
......@@ -82,12 +82,9 @@
- elsif current_user.todos.any?
.todos-all-done
= render "shared/empty_states/todos_all_done.svg"
%h4.text-center
- if todos_filter_empty?
%h4.text-center
Good job! Looks like you don't have any todos left.
- else
There are no Todos to show.
- if todos_filter_empty?
%p.text-center
Are you looking for things to do? Take a look at
= succeed "," do
......@@ -95,6 +92,9 @@
contribute to
= link_to "merge requests", merge_requests_dashboard_path
or mention someone in a comment to assign a new todo automatically.
- else
%h4.text-center
There are no todos to show.
- else
.todos-empty
.todos-empty-hero
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment