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.

Sign up for GitHub

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

Jump to bottom

feat/グループ一覧ページのレイアウトを Figma に揃える #268

Merged
merged 18 commits into from
Sep 24, 2024
Merged

feat/グループ一覧ページのレイアウトを Figma に揃える #268

merged 18 commits into from
Sep 24, 2024

Conversation

anko9801
Copy link
Contributor

@anko9801 anko9801 commented Aug 28, 2024 •
edited by github-actions bot
Loading

User description

fix: #164

PR Type

enhancement

Description

  • グループ一覧ページのレイアウトをFigmaに合わせて大幅に改善しました。
  • グリッドレイアウトを導入し、グループ情報（名前、詳細、予算）を見やすく表示します。
  • 管理者向けの「グループの新規作成」ボタンの位置を調整しました。

Changes walkthrough 📝

Relevant files
Enhancement
GroupsPage.vue
グループ一覧ページのレイアウトとスタイルの改善                                                                   

src/pages/GroupsPage.vue

  • グループ一覧ページのレイアウトをFigmaに合わせて調整
  • グリッドレイアウトを使用してグループ情報を表示
  • 管理者向けの新規作成ボタンの位置を変更
    		•  +18/-16 

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    @github-actions GitHub Actions
    Copy link

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No key issues to review

    @anko9801 anko9801 requested a review from reiroop August 28, 2024 07:08
    @anko9801 anko9801 self-assigned this Aug 28, 2024
    reiroop
    reiroop reviewed Aug 30, 2024
    View reviewed changes
    src/pages/GroupsPage.vue Outdated Show resolved Hide resolved
    reiroop
    reiroop reviewed Aug 30, 2024
    View reviewed changes
    Copy link
    Contributor

    @reiroop reiroop left a comment

    Choose a reason for hiding this comment

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

    must 画面比率をスマホくらいにすると右側が画面外に出てしまうので、その対応またはissueに起こすのをお願いしたいです。

    src/pages/GroupsPage.vue Outdated Show resolved Hide resolved
    @reiroop
    Copy link
    Contributor

    reiroop commented Aug 30, 2024

    他はいいと思います！綺麗に描かれていてありがたいです

    reiroop
    reiroop approved these changes Sep 21, 2024
    View reviewed changes
    Copy link
    Contributor

    @reiroop reiroop left a comment

    Choose a reason for hiding this comment

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

    LGTMです！ありがとうございます
    nits 画面幅が390pxを切ったあたりから左右にスクロールできるようになってしまうのと、グループ一覧の文字が画面の左端と接してしまうのが少し気になるかなーって感じです。重要ではないですが将来的に対応したいのでissueにしてしまってもいいですか？

    @reiroop reiroop mentioned this pull request Sep 21, 2024
    Merged
    @anko9801
    Copy link
    Contributor Author

    ページネーションバーをレスポンシブ化しました。レビューお願いします。

    reiroop
    reiroop approved these changes Sep 23, 2024
    View reviewed changes
    Copy link
    Contributor

    @reiroop reiroop left a comment

    Choose a reason for hiding this comment

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

    LGTMです！pagenation barもすごくいい感じでありがたいです！
    細かいところなんですが、気になったところをいじってみたのでよければ見てくださると嬉しいです！

    src/pages/GroupsPage.vue Outdated Show resolved Hide resolved
    src/pages/GroupsPage.vue Outdated Show resolved Hide resolved
    @anko9801 anko9801 merged commit 7ed8c7b into main Sep 24, 2024
    6 checks passed
    @anko9801 anko9801 deleted the feat/group_list branch September 24, 2024 12:31
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Reviewers

    @reiroop reiroop reiroop approved these changes

    Assignees

    @anko9801 anko9801

    Labels
    Review effort [1-5]: 2
    Projects
    None yet
    Milestone
    No milestone
    Development

    Successfully merging this pull request may close these issues.

    グループ一覧ページ
    3 participants
    @anko9801 @reiroop @mehm8128