Skip to content

Commit 6d4761f

Browse files
committed
refactor: Encapsulate supported_genders in component
Move CountryConfig.supported_genders call into component method so the view doesn't need to know about CountryConfig directly.
1 parent 93d9eb0 commit 6d4761f

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

app/components/progress_tab/breakdown_table_component.html.erb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@
77
<%= breakdown[:all] %>
88
</p>
99
</div>
10-
<% CountryConfig.supported_genders.each_with_index do |gender, index| %>
11-
<% is_last = index == CountryConfig.supported_genders.size - 1 %>
10+
<% supported_genders.each_with_index do |gender, index| %>
11+
<% is_last = index == supported_genders.size - 1 %>
1212
<div class="d-flex ai-center jc-space-between <% if !is_last || include_bottom_border %>pb-3px bb-grey-mid<% end %> mb-4px">
1313
<p class="m-0px p-0px pl-16px ta-left fw-normal fs-16px c-grey-dark">
1414
<%= t("progress_tab.genders.#{gender}") %>

app/components/progress_tab/breakdown_table_component.rb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,4 +18,8 @@ def initialize(title:, breakdown:)
1818
def include_bottom_border
1919
title != t("progress_tab.diagnoses.hypertension_and_diabetes")
2020
end
21+
22+
def supported_genders
23+
CountryConfig.supported_genders
24+
end
2125
end

0 commit comments

Comments
 (0)