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

RangeErrorを解消し、スマホからhash付きのコメントURLにアクセスできるようにする #6771

Merged
merged 1 commit into from
Jul 31, 2023

Conversation

gogutan
Copy link
Contributor

@gogutan gogutan commented Jul 30, 2023

Issue

概要

iPhoneのSafariからhash付きのコメントURLにアクセスすると

257038731-d201d4d8-5d55-49b0-8fc2-2251ab2a7aef_PNG__1170×2532_

のようなエラーが発生し、クラッシュしていました。

原因を探るためPCのChromeでアクセスすると、

70【WebAP】「Webを支える技術」読了🎉___FBC

クラッシュはしないのですが、上記のように

created() {
window.onpopstate = function () {
location.replace(location.href)
}
this.getFootprints()
},

window.onpopstateにて無限ループが発生していることが分かりました。

少し調査したところ、このような記述は

created() {
// ブラウザバック・フォワードした時に画面を読み込ませる
window.onpopstate = function () {
location.replace(location.href)
}
this.getNotificationsPerPage()
},

にて、ブラウザバック/フォワード時にページャーを読み込ませるために必要なもののようです。
しかしfootprints.vueではこのような挙動は不要のため、この3行を単純に削除してエラーを解消したいと考えています。

変更確認方法

  1. bug/range-errorをローカルに取り込む
  2. (ユーザー1操作)日報上で、ユーザー2へのメンションを含むコメントをする。
  3. (ユーザー2操作)通知タブを開き、メンションされた日報コメントにアクセスする。
  4. (ユーザー2操作)アクセスした日報ページで検証ツールのConsoleを開き、Uncaught RangeErrorが発生していないことを確認する。

Screenshot

変更前

70【WebAP】「Webを支える技術」読了🎉___FBC

変更後

_development__テストのお知らせ___FBC

@gogutan
Copy link
Contributor Author

gogutan commented Jul 30, 2023

@komagata @machida
スマホからアクセス時に少し不便だったためPRを作ってみました、お手隙の際にご確認をお願いいたします🙇‍♂️

@machida machida requested a review from komagata July 31, 2023 06:41
@machida
Copy link
Member

machida commented Jul 31, 2023

@gogutan おおー、すごく助かります!!これ、原因を調べきれてなくて、未対応だったんだけど、すぐに解決したいやつでした!!これのせいでスマホの Chrome を使ってました。。。

@komagata レビュー、マージお願いしますー🙏

@komagata
Copy link
Member

@gogutan PR非常に助かります!!!!!

@github-actions github-actions bot mentioned this pull request Jul 31, 2023
16 tasks
@gogutan
Copy link
Contributor Author

gogutan commented Jul 31, 2023

早速ご確認ありがとうございます、良かったです!
@kasumi8pon も一緒に調べてくれました😄

@komagata
Copy link
Member

komagata commented Aug 1, 2023

@kasumi8pon さんもありがとうございます〜😭

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants