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

signup, group registerにおいてアイコンをギャラリーから選択できるようにし、アイコンが円形になるように表示させた #136

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

KuuTokinaga
Copy link
Contributor

@KuuTokinaga KuuTokinaga commented Nov 27, 2023

対応するissue

概要

  • ユーザ登録とグループ登録画面でのアイコンをギャラリーから設定できるようにし、円形に表示させるようにした

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

  • ユーザ登録、グループ登録画面でアイコン画像の場所をタップすると、ギャラリーにアクセスし、そこで画像を選択するとアイコン画像としてそれが反映される
    -↑の画面で表示される画像はスケールが調整され、円形に表示される

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

Screenshot_2023-11-27-16-53-13-49
Screenshot_2023-11-27-16-53-50-80_02aa7fb97b79c031ca96fd56759b472e
Screenshot_2023-11-27-16-53-39-66
Screenshot_2023-11-27-16-53-29-00_02aa7fb97b79c031ca96fd56759b472e

その他

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.

コード確認しました!
動作は問題ありませんでしたが,現在推奨されていないコードの処理が見られたので
そこの修正をお願いします!

android:id="@+id/imageAccount"
android:layout_width="0dp"
<androidx.cardview.widget.CardView
android:id="@+id/icon"
Copy link
Contributor

Choose a reason for hiding this comment

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

iconだと他のViewとの差別化をわかりやすくするためにユニークな命名にしましょう!

android:src="@drawable/img_account"

<androidx.cardview.widget.CardView
android:id="@+id/icon"
Copy link
Contributor

Choose a reason for hiding this comment

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

「fragment_signup.xml」と同じようにiconだと他のViewとの差別化をわかりやすくするためにユニークな命名にしましょう!

private fun openGallary() {
val intent = Intent(Intent.ACTION_PICK)
intent.type = "image/*"
startActivityForResult(intent, REQUEST_GALLARY_TAKE)
Copy link
Contributor

Choose a reason for hiding this comment

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

startActivityForResult()は現在推奨されていない処理なので,
以下の記事を参考にして修正してみてください.
1. startActivityForResultがDeprecatedになった対応方法
2. アクティビティの結果を取得する

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.

[add]ユーザー登録とグループ登録でアイコンを選択・登録できるようにする
2 participants