Commit 0bc9008e authored by Phil Hughes's avatar Phil Hughes

Fixed todos empty state when filtering

Closes #24127
parent 717b8a76
...@@ -61,6 +61,10 @@ module TodosHelper ...@@ -61,6 +61,10 @@ module TodosHelper
} }
end end
def todos_filter_empty?
todos_filter_params.all? {|key, value| value.nil?}
end
def todos_filter_path(options = {}) def todos_filter_path(options = {})
without = options.delete(:without) without = options.delete(:without)
......
...@@ -83,14 +83,18 @@ ...@@ -83,14 +83,18 @@
.todos-all-done .todos-all-done
= render "shared/empty_states/todos_all_done.svg" = render "shared/empty_states/todos_all_done.svg"
%h4.text-center %h4.text-center
Good job! Looks like you don't have any todos left. - if todos_filter_empty?
%p.text-center Good job! Looks like you don't have any todos left.
Are you looking for things to do? Take a look at - else
= succeed "," do There are no Todos to show.
= link_to "the opened issues", issues_dashboard_path - if todos_filter_empty?
contribute to %p.text-center
= link_to "merge requests", merge_requests_dashboard_path Are you looking for things to do? Take a look at
or mention someone in a comment to assign a new todo automatically. = succeed "," do
= link_to "the opened issues", issues_dashboard_path
contribute to
= link_to "merge requests", merge_requests_dashboard_path
or mention someone in a comment to assign a new todo automatically.
- else - else
.todos-empty .todos-empty
.todos-empty-hero .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