-
-
Notifications
You must be signed in to change notification settings - Fork 19
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[#1472] Replace "Room" with "Section" #1474
Conversation
locales files with Section and Sections Co-authored-by: Zee Spencer <zspencer@users.noreply.github.com> Co-authored-by: Ana <anaulin@users.noreply.github.com> Co-authored-by: Dalton Pruitt <daltonrpruitt@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like it! Thank you!
@@ -29,7 +29,7 @@ | |||
|
|||
<div> | |||
<%- if current_room && policy(current_room).edit? %> | |||
<%= link_to t('icons.edit'), [:edit, current_space, current_room], class: 'no-underline', aria: { label: "Configure Room"} %> | |||
<%= link_to t('icons.edit'), [:edit, current_space, current_room], class: 'no-underline', aria: { label: "Configure Section"} %> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe not for this PR, but it would be nice to pull this out to config/locales/rooms/en.yml
and place it somewhere like rooms.edit.link_to
(you'll want to figure out where we put theedit.link_to
text in other spots)
@@ -27,7 +27,7 @@ | |||
|
|||
<footer> | |||
<%= link_to [:new, space, :room] do %> | |||
<span class="icon --add" role="img" aria-label="Add"></span>Add New Room | |||
<span class="icon --add" role="img" aria-label="Add"></span>Add New Section |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could live at rooms.new.link_to
in locales/rooms/en.yml
@@ -10,7 +10,7 @@ | |||
|
|||
<fieldset> | |||
<header> | |||
<h3>Rooms</h3> | |||
<h3>Sections</h3> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if we have a pattern yet for how we localize headers of pages but I wish we did.
#1472
Replace "Room" with "Section" in customer facing UI.
Rooms List Page
create room form
edit room form
Tests
Confirmed pages render on local with text changes.
Can create and edit a room/section.
馃悰 cannot Delete a room/section