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

カードの新規作成 #48

Merged
merged 19 commits into from
Sep 8, 2023
Merged

カードの新規作成 #48

merged 19 commits into from
Sep 8, 2023

Conversation

kawanolk
Copy link

Ticket

変更内容

カードの新規作成

確認方法

npm run devで実行

@kawanolk kawanolk added feature 新しい機能の追加 nomerge 作業中/マージしないで labels Aug 27, 2023
@kawanolk
Copy link
Author

デザインに合わせたカードのみ作成しています。
現在のコンポーネントではデザインに合わせた表示ができないため以下の改良が必要
・マイルストーン、issuesごとに分けるコード
 ->マイルストーンとissuesが連結してるため分離する必要有

以下の残りタスク
・レスポンシブ化
・右寄せ
・タイトルと説明
・カード内に表示するデザインの取り込み

@takecchi
Copy link
Member

takecchi commented Aug 27, 2023

@kawanolk

・レスポンシブ化
・右寄せ

これに関しては別のタスクで切り出してやるので一旦考えなくて大丈夫かなと。

・マイルストーン、issuesごとに分けるコード
 ->マイルストーンとissuesが連結してるため分離する必要有

Fragment(<></>)を使用することで解決できそうだったのでコミットしました。

@takecchi
Copy link
Member

ローディング中はスケルトンを使うと良いかなと思いました。
https://mui.com/material-ui/react-skeleton/

@kawanolk kawanolk added the help wanted 助けをください label Aug 31, 2023
@kawanolk
Copy link
Author

APIからstateとlabel_urlを取得して、card.tsxで任意(open or closed)のアイコンの設定とラベル表示させたいけど、分からない
->data.mapを使うと思っている

@kawanolk kawanolk removed the help wanted 助けをください label Sep 1, 2023
@kawanolk
Copy link
Author

kawanolk commented Sep 3, 2023

変更内容

カードの作成

確認方法

npm run devで実行

スケルトンが表示されること
カード内コンテンツが表示されること

`;

const Milestones = () => {
const { data } = useMilestones();
Copy link
Member

Choose a reason for hiding this comment

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

useMilestones()でエラーが返却された場合どうなりますか

Copy link
Author

Choose a reason for hiding this comment

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

考慮していなかったので、エラーが起きた際に非表示になるようにします。

padding-bottom: 300px;
background-color: rgba(255, 255, 255, 0.6);
backdrop-filter: blur(10px);
`;
Copy link
Contributor

@AyumuAkimoto AyumuAkimoto Sep 5, 2023

Choose a reason for hiding this comment

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

widthは%とmax-widthを併記すると画面サイズが変わってもデザインが崩れにくくて良いので以下のような感じにしてみて欲しいです。

const StyledContainer = styled('div')`
  width: 100%;
  max-width: 645px;
  min-height: 300px;

あくまで例なのでdevツールとかで画面サイズ比とかいじくり回してデザイン崩れないようによしなにやってください。
他のコンポーネントも同じ容量で出来るだけstyledcomponentを使うようにしてみてください。

Copy link
Author

@kawanolk kawanolk Sep 7, 2023

Choose a reason for hiding this comment

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

CSSだと適用されなかったのでBoxコンポーネントでラッパーしたら適用されたことを思い出しました。
代替手段を探してみます

Copy link
Author

Choose a reason for hiding this comment

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

記述方法を間違えてただけでしたので解決しました。

Copy link
Contributor

Choose a reason for hiding this comment

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

%使って欲しいというのは極端な例にはなるけど以下の画像みたいにクソデカ画面で見た時とかに画面サイズに合わせないで決めうちの数字になってしまうのが困るという話でした。
あとデザイン的にも下の透過している部分に対してフェーズの方のボックスは横幅いっぱいにレイアウトされると思うのでそこも再現してほしい。
スクリーンショット 2023-09-08 16 28 53

@takecchi takecchi merged commit 83cec50 into main Sep 8, 2023
3 checks passed
@takecchi takecchi deleted the feature/issues-41 branch September 8, 2023 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 新しい機能の追加 nomerge 作業中/マージしないで
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants