sakabar / hinemos

Website for Blindfolded speedcubing players
MIT License
1 stars 1 forks source link

レターペアに空白文字を許容する #858

Closed sakabar closed 2 years ago

sakabar commented 2 years ago

kitsさんからの要望 https://twitter.com/kits/status/1501231437982228480

空白を許していなかった理由は明確ではない。 空白を含むかどうかによる表記揺れを嫌った可能性があるが、 現状で空白以外でも表記ゆればかりになっているため空白で事態が劇的に悪化することはなさそう。

→ レターペアに関する機能で、空白を許せるように修正していく

sakabar commented 2 years ago
git grep -l -i 'letterPair' 

src/css/registerLetterPair.css
src/html/letterPairQuiz.html
src/html/letterPairTable.html
src/html/mypage.html
src/html/registerLetterPair.html
src/html/threeStyle/quiz.html
src/html/transformFromAnalysis.html
src/html/tutorialAnalysisForBeginner.html
src/html/tutorialMemorizationForBeginner.html
src/js/components/pages/MemoTrainingResultPage/index.js
src/js/components/pages/MemoTrainingStatsPage/index.js
src/js/components/templates/MemoTrainingResultTemplate/index.js
src/js/components/templates/MemoTrainingStatsTemplate/index.js
src/js/letterPairQuiz.js
src/js/letterPairTable.js
src/js/letterPairTableUtils.js
src/js/modules/memoTraining.js
src/js/modules/memoTrainingResult.js
src/js/modules/memoTrainingStats.js
src/js/registerLetterPair.js
src/js/threeStyleQuiz.js
src/js/transformFromAnalysis.js
test/letterPairQuiz.js
test/registerLetterPair.js
webpack.config.js
sakabar commented 2 years ago
**/hinemos_stg/hinemos%git grep -l -i 'letterPair' | grep -F '.js' 
src/js/components/pages/MemoTrainingResultPage/index.js
src/js/components/pages/MemoTrainingStatsPage/index.js
src/js/components/templates/MemoTrainingResultTemplate/index.js
src/js/components/templates/MemoTrainingStatsTemplate/index.js
src/js/letterPairQuiz.js
src/js/letterPairTable.js
src/js/letterPairTableUtils.js
src/js/modules/memoTraining.js
src/js/modules/memoTrainingResult.js
src/js/modules/memoTrainingStats.js
src/js/registerLetterPair.js
src/js/threeStyleQuiz.js
src/js/transformFromAnalysis.js
test/letterPairQuiz.js
test/registerLetterPair.js
webpack.config.js
sakabar commented 2 years ago

HTMLのページと、Reactのページの2つに分けて考える。

まずHTMLに紐づくJS

1. src/js/registerLetterPair.js
2. src/js/letterPairQuiz.js
3. src/js/letterPairTable.js
4. src/js/letterPairTableUtils.js
5. src/js/threeStyleQuiz.js
6. src/js/transformFromAnalysis.js
sakabar commented 2 years ago
  1. src/js/registerLetterPair.js

登録時にAPI側でスペースを外している部分を修正する。 取得時には何もしていないので修正不要。

sakabar commented 2 years ago
  1. src/js/letterPairQuiz.js 修正不要。そのまま表示しているだけ
sakabar commented 2 years ago
  1. src/js/letterPairTable.js 取得時はそのまま表示しているので修正不要 登録時にスペースを消しているので修正する。
sakabar commented 2 years ago
  1. src/js/letterPairTableUtils.js 何もしていないので修正不要

  2. src/js/threeStyleQuiz.js 取得したレターペアを , でつなげているだけなので問題なし

  3. src/js/transformFromAnalysis.js 同上。

sakabar commented 2 years ago

次、React関連

src/js/components/pages/MemoTrainingResultPage/index.js src/js/components/pages/MemoTrainingStatsPage/index.js src/js/components/templates/MemoTrainingResultTemplate/index.js src/js/components/templates/MemoTrainingStatsTemplate/index.js

src/js/modules/memoTraining.js src/js/modules/memoTrainingResult.js src/js/modules/memoTrainingStats.js

sakabar commented 2 years ago

・ありうる2文字の組み合わせをletterPairTableUtilsの関数を使って取得 ・レターペアを , で結合

これしかしていないので、問題なし。