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

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

anko9801
Copy link
Contributor

@anko9801 anko9801 commented Aug 28, 2024

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

    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
    src/pages/GroupsPage.vue Outdated Show resolved Hide resolved
    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

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

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

    Successfully merging this pull request may close these issues.

    グループ一覧ページ
    3 participants