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/入出金リストのレスポンシブ対応 #297

Merged
merged 5 commits into from
Sep 24, 2024

Conversation

anko9801
Copy link
Contributor

@anko9801 anko9801 commented Sep 18, 2024

User description

  • モバイルのときの表のレイアウトのデザインを作成し、実装しました
  • 複数のタグが並んでいるときテキストの折り返しをせず、改行する

PR Type

enhancement


Description

  • タグ表示を改善し、テキストの折り返しを防ぐためにflex-wrapwhitespace-nowrapを追加しました。
  • トランザクションテーブルのレイアウトを変更し、異なる画面サイズでのレスポンシブ性を向上させました。
  • PencilIconEditButtonに置き換え、編集機能を改善しました。
  • モバイルビュー用に新しいレイアウトを追加し、flexgapを使用しました。
  • トランザクションページのメインコンテナから最小幅の制約を削除しました。

Changes walkthrough 📝

Relevant files
Enhancement
TagsGroup.vue
Improve tag display responsiveness and prevent text wrapping

src/components/shared/TagsGroup.vue

  • Added flex-wrap to the container div for better responsiveness.
  • Added whitespace-nowrap to tag divs to prevent text wrapping.
  • +2/-2     
    TransactionTable.vue
    Enhance transaction table responsiveness and edit functionality

    src/components/transactions/TransactionTable.vue

  • Modified grid layout for better responsiveness on different screen
    sizes.
  • Replaced PencilIcon with EditButton for edit functionality.
  • Added a new layout for mobile view using flex and gap.
  • +37/-5   
    TransactionsPage.vue
    Simplify layout constraints for transaction page                 

    src/pages/TransactionsPage.vue

    • Removed minimum width constraint from the main container.
    +1/-1     

    💡 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: 3 🔵🔵🔵⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Key issues to review

    スタイル変更
    flex-wrapwhitespace-nowrapの追加により、タグの表示が改善されましたが、これによる他のスタイルへの影響を確認してください。

    レイアウト変更
    グリッドレイアウトの変更があります。特にgrid-cols-[minmax(120px,1fr)_70px_repeat(2,minmax(120px,1fr))_3fr_50px]の適用により、異なる画面サイズでの表示がどのように変わるかを検証する必要があります。

    最小幅削除
    最小幅の制約を削除したことで、特にモバイルビューでのレイアウトがどのように影響を受けるかを確認してください。

    @anko9801 anko9801 requested review from mehm8128 and reiroop and removed request for mehm8128 September 18, 2024 10:25
    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.

    めちゃいいと思います!ありがとうございます!
    nits 横幅770px~840pxくらいのときに横にスクロールできるようになってるなーと気づいたんですが、あんまり困らないですし、そもそもこの幅の端末ってあんまりないのでほんとに気にしなくてもいいかなーって感じです。
    あと要素全体がrouter-linkで囲まれてるのでeditbuttonじゃなくてもよさそう?って思ったんですが、こちらは使う上では特に影響がないのでもっと気にしなくて良さそうって感じです。こちらって何か理由とかあるんですか?

    @anko9801
    Copy link
    Contributor Author

    横スクロールは保留させて頂いて EditButton を押すとモーダルが開くんですがそれは別の PR でするので今は大丈夫です 🙇

    @anko9801 anko9801 merged commit 4ccc15f into main Sep 24, 2024
    6 checks passed
    @anko9801 anko9801 deleted the feat/responsive_transaction_list branch September 24, 2024 12:45
    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.

    2 participants