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

roleを表す色枠が出ていないユーザアイコンがある #7516

Closed
SuzukaHori opened this issue Mar 11, 2024 · 6 comments
Closed
Assignees

Comments

@SuzukaHori
Copy link
Contributor

SuzukaHori commented Mar 11, 2024

概要

日報一覧ページ/reportsのミニアイコンなど、一部のユーザアイコンでroleを表す色枠が表示されていません。


/reportsのミニアイコン

スクリーンショット 2024-03-11 17 19 19

productsのミニアイコン

スクリーンショット 2024-03-11 17 42 43

/external_entriesの普通のアイコン

スクリーンショット 2024-03-11 17 49 33

/articlesのミニアイコン

スクリーンショット 2024-03-11 17 46 32

※このページに関しては外の人向けなので、仕様のような気もします🤔


詳細には見れていないので、このほかにもあるかもしれません。

補足

こちらのIssueに取り組んでいる最中に、仕様ではなくバグであると確認しました。

/pagesのミニアイコンにも枠が表示されていませんが、こちらに関しては上のIssueで対応します。

環境

OS: MacOS
ブラウザ: chrome
chromeバージョン: 122.0.6261.69

@machida
Copy link
Member

machida commented Mar 12, 2024

📝

現状の実装

  • reports は react
    • react を外したら実装
  • products は react
    • react を外したら実装
  • external_entries は vue
    • vue を外したら実装

除外

  • articlesのミニアイコン
    • これは外向けなのであえて枠を表示しない。

@komagata komagata added 2 and removed 1 labels Mar 13, 2024
@hirano-vm4
Copy link
Contributor

hirano-vm4 commented Mar 17, 2024

@komagata @machida

お疲れ様です!こちら現在、取り組んでいますが、 @machida さんのコメントで以下のようにありますが

現状の実装
reports は react
react を外したら実装
products は react
react を外したら実装
external_entries は vue
vue を外したら実装

現在は、React・Vueの状態ですが、現状のままバグを修正という認識で大丈夫でしょうか?言葉の通りreact を外したら実装 vue を外したら実装だと今はステイという捉え方もできたので念の為確認になります🙏

すぐに非React化・非Vue化しないようならそんなにかからず、修正自体はできるかと思うので修正に取り掛かります!

@machida
Copy link
Member

machida commented Mar 17, 2024

@hirano-vm4 質問ありがとうございます!今回はvueとreactのままでお願いしますー

@hirano-vm4
Copy link
Contributor

@machida

ありがとうございます🙏承知いたしました!

Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@hirano-vm4
Copy link
Contributor

こちら、本番環境での動作を確認したためcloseします。

@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

4 participants