Skip to content

Conversation

@y33550336
Copy link

@y33550336 y33550336 commented Jun 28, 2025

User description

close #298
localhostが整備中で開けなかったのでページ見れてないです


PR Type

Enhancement


Description

  • ContentHeaderからdetailプロパティ削除

  • 詳細テキストとNewボタンをサブヘッダーに移動

  • .headerContainerをフレックスレイアウト化

  • .subHeaderContainerと.detailText追加


Changes diagram

flowchart LR
  UA["UserAccounts.vue"]
  CH["content-header"]
  SH["subHeaderContainer"]
  DB["detailText\n& New button"]

  UA -- "contains" --> CH
  CH -- "below header" --> SH
  SH -- "contains" --> DB
Loading

Changes walkthrough 📝

Relevant files
Enhancement
UserAccounts.vue
ヘッダーのサブコンテナ追加とスタイル調整                                                                         

src/pages/UserAccounts.vue

  • content-headerからdetailプロパティを削除
  • detailTextとNewボタンを.subHeaderContainerでラップ
  • .headerContainerにdisplay:flexとalign-items追加
  • .subHeaderContainerと.detailTextのスタイル定義を追加
  • +20/-8   

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link
    Contributor

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Impact
    General
    ヘッダー要素を左右に分散

    ヘッダーコンテナ内でメインのcontent-headerとサブヘッダーを左右に分離するため、justify-content:
    space-between;
    を追加してください。

    src/pages/UserAccounts.vue [55-58]

     .headerContainer {
       display: flex;
    +  justify-content: space-between;
       align-items: center;
     }
    Suggestion importance[1-10]: 6

    __

    Why: Adding justify-content: space-between to .headerContainer correctly spaces the main header and subheader elements, improving the layout.

    Low
    ヘッダー幅の自動調整

    メインヘッダーが横並びレイアウト内で適切に幅を伸縮できるように、flex: 1;を追加して余白を確保してください。

    src/pages/UserAccounts.vue [70-72]

     .header {
    +  flex: 1;
       margin-bottom: 2rem;
     }
    Suggestion importance[1-10]: 3

    __

    Why: Applying flex: 1; to .header is a minor styling tweak that may be unnecessary when using justify-content: space-between and could lead to unintended stretching.

    Low
    ディレクティブのスペース削除

    Vue のディレクティブでは通常スペースを付けずに記述するため、: の前後のスペースを削除してコードスタイルを統一してください。

    src/pages/UserAccounts.vue [23-32]

    -<div :class = "$style.subHeaderContainer">
    -  <p :class = "$style.detailText">アカウント情報を変更します。</p>
    +<div :class="$style.subHeaderContainer">
    +  <p :class="$style.detailText">アカウント情報を変更します。</p>
       <link-button
         :to="{ name: 'UserAccountsNew' }"
         type="primary"
         icon="mdi:account"
       >
         New
       </link-button>
     </div>
    Suggestion importance[1-10]: 2

    __

    Why: This is a trivial formatting change that does not affect functionality and only has minimal impact on code consistency.

    Low

    @Futadaruma
    Copy link
    Collaborator

    プレビューができない状態で作業させてしまいすみません
    現在の実装だと以下のようになっているので変更をお願いします
    mobile view切り替わり直後の画面幅
    スマホサイズの画面幅

    画面幅が一定以下のときに適応させるメディアアットルール(@media)を使って、要素が縦一列に並ぶようにするとうまくいきそうかなと思います

    このコミットでは/user以下の表示ができると思うのでこのコミットの先で変更をしてプレビューを見て作業してもらえると助かります

    @y33550336
    Copy link
    Author

    Type Check が落ちているのは SNS のアカウント追加(Mixi2, Blue Sky)のところでエラーはいているらしいです

    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.

    sp: ContentHeaderの構造を変える

    3 participants