-
-
Notifications
You must be signed in to change notification settings - Fork 108
スマートフォン版のDojo表示を地方ごとから都道府県ごとに変更 #1701
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
Conversation
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.
細かい部分で気になった点をコメントしましたが、それ以外は動作も含めて良さそうです!
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.
以下もご対応いただければ🙏
@rakuda-san-desu |
@rakuda-san-desu |
いろいろ含めると何をしているPRなのか分かりづらくなってしまうので、「このPRでやりたい内容」に含まれるかで考えると良さそうです👍 Zoom でお話しした、「1つのコミットに1種類の更新」も、同様の考え方です! |
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.
PRありがとうございます。気になった部分にコメントしました 🙇 対応方法についても相談に乗りますので、気軽にメンションください 〜
@nanophate 動作確認や丁寧なレビューありがとうございます!助かります🙏💖 @nacchan99 対応ありがとうございます!後ほど確認します👀✨ |
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.
Review App での動作も問題なさそうでした...!! 💯 🚀
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.
ヘルパー切り出しなども対応ありがとうございます!
細かい部分で1点コメントしていますが、それ以外良さそうなので Approve しています🙆♀️
Closes #462
概要
スマートフォン版のDojo情報表示を、「地方ごと」から「都道府県ごと」へ変更しました。
変更内容
Dojo
モデルにgroup_by_prefecture
/group_by_prefecture_on_active
を追加home_controller.rb
に@prefectures_and_dojos
を追加app/views/shared/_dojos.html+smartphone.erb
にて都道府県ごとのアコーディオン表示に対応home_controller.rb
にて Dojo数が 1 件のとき"1 Dojos"
と表示される問題に対応label: "#{count} Dojo#{'s' if count != 1}"
という形で文言を整形し、単数形を表示できるように調整変更前

変更後

補足
スマホ表示のレイアウトやスタイルには変更を加えていません。構造のみを置き換えています。