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

ロゴの変更 #385

Merged
merged 1 commit into from
Oct 29, 2024
Merged

ロゴの変更 #385

merged 1 commit into from
Oct 29, 2024

Conversation

Futadaruma
Copy link
Contributor

@Futadaruma Futadaruma commented Oct 25, 2024

User description

UIと同様にロゴと大学名の変更を行いました


PR Type

enhancement


Description

  • フッターのロゴをtraP_logo_full_white.svgに変更し、著作権表示を更新しました。
  • ヘッダーのロゴをデバイスに応じて変更し、モバイルではtraPortfolio_dashboard_logo_icon.svg、PCではtraPortfolio_dashboard_logo_full.svgを使用するようにしました。
  • ロゴの幅をモバイルとPCで異なる値に設定しました。

Changes walkthrough 📝

Relevant files
Enhancement
PageFooter.vue
フッターのロゴと著作権表示の更新                                                                                 

src/components/Layout/PageFooter.vue

  • ロゴ画像を新しいファイルに変更
  • 著作権表示の年数を更新
+2/-2     
PageHeader.vue
ヘッダーのロゴ表示をデバイス対応に変更                                                                           

src/components/Layout/PageHeader.vue

  • モバイルとPCで異なるロゴを表示
  • ロゴの幅をデバイスに応じて変更
+5/-3     

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

ロゴのパス変更
フッターのロゴのパスが変更されています。新しいロゴファイルがプロジェクトに存在し、正しくリンクされているか確認する必要があります。

レスポンシブロゴ
ヘッダーのロゴがデバイスに応じて変更されるように設定されています。異なるデバイスでの表示確認と、新しいロゴファイルの存在確認が必要です。

Copy link
Contributor

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Score
Enhancement
画像の幅を動的に調整して、レスポンシブデザインを向上させます。

isMobile 条件に基づいて画像の幅を設定していますが、width 属性の値がハードコードされているため、レスポンシブデザインの観点から見直すことをお勧めします。

src/components/Layout/PageHeader.vue [29]

-:width="isMobile ? '48' : '248'"
+:width="isMobile ? '50%' : '100%'"
Suggestion importance[1-10]: 7

Why: The suggestion to use percentage-based widths enhances responsiveness, which is beneficial for different screen sizes. This change improves the user experience by ensuring images scale appropriately, making it a valuable enhancement.

7
Possible issue
エイリアスの使用に関する問題を解決します。

ロゴの画像パスに使用されているエイリアス@が正しく機能するか確認してください。場合によっては、相対パスまたは絶対パスに変更する必要があります。

src/components/Layout/PageFooter.vue [3]

-<img src="/@/assets/traP_logo_full_white.svg" :class="$style.logo" />
+<img src="../assets/traP_logo_full_white.svg" :class="$style.logo" />
Suggestion importance[1-10]: 3

Why: The suggestion to verify the alias '@' usage is valid, as incorrect path resolution can lead to broken image links. However, changing to a relative path may not be necessary if the alias is correctly configured, making this suggestion of moderate importance.

3

Copy link
Collaborator

@Pugma Pugma left a comment

Choose a reason for hiding this comment

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

よさそうです!
こっちの変更も対応してくれてありがとうございました

@Futadaruma Futadaruma merged commit 77932f1 into main Oct 29, 2024
11 checks passed
@Futadaruma Futadaruma deleted the fix/changeLogo branch October 29, 2024 01:28
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