Skip to content

ユーザーを一覧するところでアイコンをlazy loadする #4268

Open
@mehm8128

Description

通知設定は#4062 で無理やりちょっと速くしたけど、アイコンをlazy loadすれば多分解決する
他にもナビゲーションバーのユーザー一覧とかでも速くなる

2つ壁があって、

  1. imgタグにloading='lazy'をつけるとなぜか型エラーになる
    https://github.com/traPtitech/traQ_S-UI/pull/4141/files#diff-e313ad9f03b774bebd98b91a97efa5d4bcdfda44d3d7fc37be2356eb61a36f95
    で直してるのでマージされたら直る。lazy loadを先に実装したかったらコピペすればOKだけど、レビューもしてほしい

  2. UserIconコンポーネントの構造がよくない
    ユーザーモーダルのアイコンのはみ出している部分をクリックすると閉じないように修正した #4190
    でちょっと言及してくれるので直してくれるかもしれないし直してくれないかもしれない
    ただのアイコンとボタンとしてのアイコンを同じコンポーネントで扱っているのと、それによってbackground-imageにsrcをしているのがよくない
    これによってlazy loadができなくなっているので、アイコンコンポーネントをimgタグで作ってlazy loadできるようにして、それを使うようなアイコンボタンコンポーネントを別で作るとよさそう

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions