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

Fix/change logo #225

Merged
merged 3 commits into from
Oct 12, 2024
Merged

Fix/change logo #225

merged 3 commits into from
Oct 12, 2024

Conversation

Futadaruma
Copy link
Contributor

@Futadaruma Futadaruma commented Oct 11, 2024

User description

追加したtraPortfollio のロゴを、traPortfolio_logo_full.svgtraPortfolio_logo_icon.svgとしました
traPortfolio_ui_logo~~とした方が良ければ変更します

ロゴの.svgファイルを.icoファイルに変換しましたが、(faviconとしてみる分には問題ないですが)画質が悪くなっています
より良い変換の方法を知っていたら教えて欲しいです
変換の際は、imagemagicを使ってmagic {元ファイル}.svg -quality 100 -transparent white -define icon:auto-resize=16,32,48,64,128 favicon.icoとしてます


PR Type

enhancement


Description

  • フッターのロゴをtraP_logo_full_white.svgに変更しました。
  • ヘッダーのPC用ロゴをtraPortfolio_logo_full.svgに、モバイル用ロゴをtraPortfolio_logo_icon.svgに変更しました。

Changes walkthrough 📝

Relevant files
Enhancement
PageFooter.vue
フッターのロゴファイルを変更                                                                                     

src/components/Layout/PageFooter.vue

  • ロゴファイルをtraP_logo_full_white.svgに変更
+1/-1     
PageHeader.vue
ヘッダーのロゴファイルを変更                                                                                     

src/components/Layout/PageHeader.vue

  • PC用ロゴをtraPortfolio_logo_full.svgに変更
  • モバイル用ロゴをtraPortfolio_logo_icon.svgに変更
  • +2/-2     

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

    @Futadaruma Futadaruma requested a review from a team October 11, 2024 14:02
    Copy link

    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

    Asset Path
    フッターのロゴのパスが正しいか確認してください。パスには "@/assets/traP_logo_full_white.svg" が使用されていますが、このパスがプロジェクトの構造と一致しているか確認が必要です。

    Asset Path
    ヘッダーのロゴのパスが正しいか確認してください。パスには "@/assets/traPortfolio_logo_full.svg" と "@/assets/traPortfolio_logo_icon.svg" が使用されていますが、これらのパスがプロジェクトの構造と一致しているか確認が必要です。

    @Futadaruma Futadaruma removed the request for review from a team October 11, 2024 14:03
    Copy link

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Score
    Possible issue
    エイリアスの使用法を確認し、必要に応じて修正してください。


    ロゴのパスにエイリアスを使用していますが、Vueプロジェクトでのエイリアスの正しい使用法を確認してください。エイリアスが正しく設定されていない場合、ロゴが表示されない可能性があります。

    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]: 7

    Why: The suggestion to verify the alias usage is valid, as incorrect alias configuration can lead to issues with asset loading. However, it is not a direct code change but a recommendation to ensure correct setup, which slightly reduces its impact.

    7

    @Futadaruma Futadaruma requested review from Kaki256 and sh0go07 October 11, 2024 14:10
    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.

    ありがとう!!!

    favicon の画質に関しては問題ないです!
    複数のサイズの生成をしてくれているので、場面に合わせて大きな解像度が使われると思います
    VSCode とかで見ると favicon として見るよりも表示領域が大きいので、それで劣化して見えたかもしれません

    一つだけ、修正をお願いします
    従来の traP のロゴと新規の traPortfolio のロゴでは縦横比が異なるので、PageHeader.vue 内部での

    <img
        :src="isMobile ? spLogo : pcLogo"
        alt="traP"
        :width="isMobile ? '48' : '343'"
        height="48"
    />

    の部分を変更してほしいです

    @Futadaruma Futadaruma requested a review from Pugma October 12, 2024 14:43
    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.

    ok です!
    最後細かなサイズ調整もやってくれてありがとうございました

    @Pugma Pugma merged commit 6937915 into master Oct 12, 2024
    8 checks passed
    @Pugma Pugma deleted the fix/change_logo branch October 12, 2024 14:50
    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