Skip to content

feat:三要素の単語帳を作る機能を追加 - #31

Open
KoshiroIto wants to merge 5 commits into
mainfrom
three-element-flashcards
Open

feat:三要素の単語帳を作る機能を追加#31
KoshiroIto wants to merge 5 commits into
mainfrom
three-element-flashcards

Conversation

@KoshiroIto

Copy link
Copy Markdown
Collaborator

My単語帳 三要素の単語帳(三角柱型)を作成しました。
行った作業は以下です。
・三角柱状のカードを作るコンポーネントを追加
 ・三つの要素が三角柱のそれぞれの面に出るようにした。
 ・左右どちらかをクリックしたときに次の要素へ切り替わるようにした。
・作成した単語のリストが三要素のものだった場合、暗記学習を開始した際に上で作成
 した三角柱の単語カードで表示されるようにした。

@KoshiroIto
KoshiroIto requested review from Tatsu723 and tknkaa August 21, 2026 04:13

@tknkaa tknkaa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

全体的にはよさそう!
細かいところだけコメントした!

borderRadius: "16px",
border: "1px solid rgba(0, 0, 0, 0.18)",
borderTop: "2px solid rgba(0, 0, 0, 0.3)",
backgroundColor: "#ffffff",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここはインラインではなく、className 側に出してダークモードにも対応するようにしてほしい!
それ以外にもインラインにする必要がないもの(borderRadius, padding, border, boxShadow)は className 側に出そう!

const is3DMode = columns.length === 3;

// 3D三角柱のそれぞれの面に入れるコンテンツの準備
const primarySense = current?.data.senses[0] || {};

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

これだと多義語でも最初の意味しか表示されないので、余裕があったら別 PR で意味でページネーションするのがいいと思う!

width: "100%",
height: "100%",
// 上から見下ろした時に奥の面が透けて重ならないように設定
backfaceVisibility: "visible",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

これは多分 hidden の間違いかな?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants