Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NJ 190 - update household rent/own screen with description and reveals #5302

Merged
merged 2 commits into from
Jan 7, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 35 additions & 8 deletions app/views/state_file/questions/nj_household_rent_own/edit.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -5,22 +5,49 @@

<% content_for :card do %>
<h1 class="h2"><%= title %></h1>
<p><%= t(".label", filing_year: current_tax_year) %></p>
<p><%= t(".description_html") %></p>
aloverso marked this conversation as resolved.
Show resolved Hide resolved
<%= form_with model: @form, url: { action: :update }, local: true, method: "put", builder: VitaMinFormBuilder do |f| %>
<div class="white-group">
<%= f.cfa_radio_set(
:household_rent_own,
label_text: t(".label"),
label_text: t(".label_html", filing_year: current_tax_year),
collection: [
{ value: :own, label: t(".homeowner") },
{ value: :rent, label: t(".tenant") },
{ value: :both, label: t(".both") },
{ value: :neither, label: t(".neither") },
],
legend_class: "sr-only"
aloverso marked this conversation as resolved.
Show resolved Hide resolved
{ value: :own, label: t(".homeowner_html") },
{ value: :rent, label: t(".tenant_html") },
{ value: :both, label: t(".both_html") },
{ value: :neither, label: t(".neither_html") },
]
) %>
</div>

<div class="reveal">
<button class="reveal__button"><%= t('.helper_1_header') %></button>
<div class="reveal__content">
<p><%= t('.helper_1_description') %></p>
</div>
</div>

<div class="reveal">
<button class="reveal__button"><%= t('.helper_2_header') %></button>
<div class="reveal__content">
<p><%= t('.helper_2_description') %></p>
</div>
</div>

<div class="reveal">
<button class="reveal__button"><%= t('.helper_3_header') %></button>
<div class="reveal__content">
<p><%= t('.helper_3_description') %></p>
</div>
</div>

<div class="reveal">
<button class="reveal__button"><%= t('.helper_4_header') %></button>
<div class="reveal__content">
<p><%= t('.helper_4_description_html') %></p>
</div>
</div>

<% if params[:return_to_review].present? %>
<%= hidden_field_tag "return_to_review", params[:return_to_review] %>
<% end %>
Expand Down
21 changes: 15 additions & 6 deletions config/locales/en.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3224,12 +3224,21 @@ en:
where_to_find_helper_heading: Where can I find property taxes paid?
nj_household_rent_own:
edit:
both: Both a tenant and a homeowner
homeowner: Homeowner
label: In %{filing_year}, which were you?
neither: Neither
tenant: Tenant (If you own a mobile home that is located in a mobile home park, you are considered a tenant.)
title: You might be eligible for the New Jersey Property Tax credit or deduction!
both_html: "<b>Both</b>: I was a homeowner for part of the year, and I was a renter for part of the year."
description_html: Your "main home" is where you primarily lived at any given time (e.g., a house, a condo, or an apartment). <b>You can’t have multiple main homes at the same time, but you may have different main homes throughout the year if you moved.</b>
helper_1_description: Yes. This tax break is designed to support people in New Jersey whose money contributes to property taxes. (Renters do this by paying rent, contributing to the cost of property taxes paid by the property owners.)
helper_1_header: Are you sure this applies to rent paid?
helper_2_description: No. Please answer the questions in this section for your main home only. You can’t claim this tax break for a vacation home, a "second home," or a property you own and rent to someone else.
helper_2_header: Can I claim for a vacation home or a property I rent out to someone?
helper_3_description: If you owned a mobile home in a mobile home park, you are considered a renter. Your site fees are what you paid in rent.
helper_3_header: What if I lived in a mobile home?
helper_4_description_html: The property tax <b>deduction</b> reduces your taxable income. We will deduct your share of property taxes paid or $15,000, whichever is less. For renters, we’ll consider 18% of rent paid as your share.
helper_4_header: How much is this deduction or credit worth?
homeowner_html: I was a <b>homeowner</b> for at least part of the year
label_html: How would you describe your living situation in %{filing_year} <b>as it pertains to your main home(s)</b>?
neither_html: Other
tenant_html: I was a <b>renter</b> for at least part of the year
title: You may qualify for a property tax deduction or credit on your main home, whether you owned or rented it.
nj_ineligible_property_tax:
edit:
continue_text: Click continue to see what else we can guide you to qualify for.
Expand Down
21 changes: 15 additions & 6 deletions config/locales/es.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3184,12 +3184,21 @@ es:
where_to_find_helper_heading: Where can I find property taxes paid?
nj_household_rent_own:
edit:
both: Tanto inquilino como propietario
homeowner: Dueño de casa
label: En %{filing_year}, ¿quién era usted?
neither: Tampoco
tenant: Inquilino (si usted es propietario de una casa móvil ubicada en un parque de casas móviles, se le considera inquilino).
title: "¡Usted podría ser elegible para el crédito o deducción del impuesto a la propiedad de New Jersey!"
both_html: "<b>Both</b>: I was a homeowner for part of the year, and I was a renter for part of the year."
description_html: Your "main home" is where you primarily lived at any given time (e.g., a house, a condo, or an apartment). <b>You can’t have multiple main homes at the same time, but you may have different main homes throughout the year if you moved.</b>
helper_1_description: Yes. This tax break is designed to support people in New Jersey whose money contributes to property taxes. (Renters do this by paying rent, contributing to the cost of property taxes paid by the property owners.)
helper_1_header: Are you sure this applies to rent paid?
helper_2_description: No. Please answer the questions in this section for your main home only. You can’t claim this tax break for a vacation home, a "second home," or a property you own and rent to someone else.
helper_2_header: Can I claim for a vacation home or a property I rent out to someone?
helper_3_description: If you owned a mobile home in a mobile home park, you are considered a renter. Your site fees are what you paid in rent.
helper_3_header: What if I lived in a mobile home?
helper_4_description_html: The property tax <b>deduction</b> reduces your taxable income. We will deduct your share of property taxes paid or $15,000, whichever is less. For renters, we’ll consider 18% of rent paid as your share.
helper_4_header: How much is this deduction or credit worth?
homeowner_html: I was a <b>homeowner</b> for at least part of the year
label_html: How would you describe your living situation in %{filing_year} <b>as it pertains to your main home(s)</b>?
neither_html: Other
tenant_html: I was a <b>renter</b> for at least part of the year
title: You may qualify for a property tax deduction or credit on your main home, whether you owned or rented it.
nj_ineligible_property_tax:
edit:
continue_text: Click continue to see what else we can guide you to qualify for.
Expand Down
2 changes: 1 addition & 1 deletion spec/features/state_file/complete_intake_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -725,7 +725,7 @@ def continue
def choose_household_rent_own(household_rent_own)
case household_rent_own
when "homeowner", "tenant", "both", "neither"
choose I18n.t("state_file.questions.nj_household_rent_own.edit.#{household_rent_own}")
choose strip_html_tags(I18n.t("state_file.questions.nj_household_rent_own.edit.#{household_rent_own}_html"))
else
throw "not a valid choice"
end
Expand Down
Loading