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

[refactor]グループ一覧ページをCompose LazyGirdに置き換え #133

Merged
merged 6 commits into from
Dec 18, 2023

Conversation

yuuhanasaki39
Copy link
Contributor

@yuuhanasaki39 yuuhanasaki39 commented Nov 6, 2023

対応するissue

関連PR

  1. CompileSDKを上げる
  2. Compose導入とRecyclerViewの置き換え ← いまここ
  3. データクラスの変更
  4. 作品詳細画面の修正

概要

  • データクラスWorkDataListの変更対応が大変そうなので、
    ネックになっているRecyclerViewをCompose LazyGridに置き換えた

  • Compose導入に合わせてKotlin バージョンを1.8.21→1.8.10にした

意図する動作内容(または変更点)

  • 「グループのボトムアイコン」→「新しくグループを登録する」→「グループ作品一覧」と進んでUIが表示されること
  • スクロールできること

元から↓

  • タブの切り替えができない
  • アイテム押下後に詳細ページに飛ばない

スクリーンショット(UI作成,変更時)

Before After

その他

@yuuhanasaki39 yuuhanasaki39 marked this pull request as ready for review November 13, 2023 06:48
@yuuhanasaki39 yuuhanasaki39 self-assigned this Nov 13, 2023
@yuuhanasaki39 yuuhanasaki39 changed the title グループ一覧ページをCompose LazyGirdに置き換え [Refactoring]グループ一覧ページをCompose LazyGirdに置き換え Nov 13, 2023
@yuuhanasaki39 yuuhanasaki39 changed the title [Refactoring]グループ一覧ページをCompose LazyGirdに置き換え [refactor]グループ一覧ページをCompose LazyGirdに置き換え Nov 13, 2023
Copy link
Contributor

@yuhei-yama yuhei-yama left a comment

Choose a reason for hiding this comment

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

以下の環境動作確認しました!
1つコメントしているのでそこの部分を追加してもらえれば問題ないと思います!

端末:Pixel7a
OSバージョン:14

implementation(platform("androidx.compose:compose-bom:2023.01.00"))
implementation("androidx.compose.material:material")
implementation("androidx.compose.foundation:foundation")
implementation("androidx.compose.ui:ui")
Copy link
Contributor

Choose a reason for hiding this comment

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

Buildの際に以下の依存関係を追加しないとエラーが出たので追加をお願いします!
implementation("androidx.compose.ui:ui-android:1.5.4")

Copy link
Contributor

@yuhei-yama yuhei-yama left a comment

Choose a reason for hiding this comment

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

確認しました!
コメントで指摘したか箇所は別のプルリクで対応しましょう!

@yuuhanasaki39 yuuhanasaki39 merged commit bb2ea5b into develop Dec 18, 2023
1 check passed
@yuuhanasaki39 yuuhanasaki39 deleted the fix/recycle-to-compose branch December 18, 2023 06:03
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.

[fix] Data classの命名変更に合わせたRecyclerViewの修正
2 participants