tommy19970714 / pictogram-san

Everyone can be Pictogram-san
MIT License
16 stars 5 forks source link

ローディング文言を追加する #41

Closed toffyui closed 3 years ago

toffyui commented 3 years ago

概要

実装内容

const LoadingTexts: string[] = [
  'Loading...',
  'Wait a moment...',
  'We are preparing now...',
  'Give us some seconds',
  'Sorry for waiting...',
  'Please keep waiting...',
  'Just some minutes, please',
  'You are almost there...',
]

初期値はLoadingなのはわかるのでWait a moment.を初期値にしてます。 これをランダムで取得。 ピクトグラムみたいに前回のものは表示しないようにしても良いですが、ローディング数秒なのでそこまでする必要もないかとやってないです。

useEffect内でisMountedを返したりしてるのは、setIntervalをuseEffectで使うとunmountのwarningがでるのでそれを防ぐためです。

image

さすがに1.5 * 5秒以内にはローディング終わるだろうということで7.5秒後にタイマーをとめます。

2秒間隔だと少し遅く感じました

画面確認

Image from Gyazo

github-actions[bot] commented 3 years ago

Visit the preview URL for this PR (updated for commit 68edc90):

https://pictogram-san--pr41-feat-change-loading-8tcc9po7.web.app

(expires Tue, 10 Aug 2021 14:23:51 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

tommy19970714 commented 3 years ago

切り替わるのが少し早すぎませんか?

toffyui commented 3 years ago

ちょっと早くなったと思ったんですけど、変わってないですかね? 今の状態でもデグれるわけではないので、このままマージします。