WebGL 2.0 がサポートされているかチェックする #390
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
naogify
requested review from
a team,
keichan34,
sugama-satsuki and
shi-works
and removed request for
a team
sugama-satsuki
approved these changes
Apr 23, 2024
keichan34
requested changes
Apr 24, 2024
There was a problem hiding this comment.
手法をちょっと変更したいです、、。
今は新しい webgl2 コンテキストを新しく作ろうとしているけど、この手法ではいくつかの問題があるのかなと思います:
- 余計にコンテキストを作ってしまう（例えば Chrome は16個まで、古い順から削除される)
- これが通ったといっても地図が表示されるか、というのも完全にわかるわけではない。。
ので、理想的な実装は、地図を初期化しようとするけど、何かしらの問題で失敗した時にエラーを表示する。例えば、コンテキストが起動できなくて new maplibregl.Map(...) で例外が発火されたら、それをキャッチしてユーザーにわかりやすい表示にする。
今回、alertで出してるけど、alertだと複数地図ある時にスパムっぽくなるし、場合によって地図がメインじゃないサイトに邪魔になる可能性にもなるので、地図があるはずの場所にHTMLで表示してください。
イメージとしてはGoogleMapsのエラーメッセージ（リクエスト上限足したらエラー画面が出るような設定はembed APIでも入っているはず、、）の方が親切だと思います。
src/lib/geolonia-map.ts
Outdated
| @@ -46,6 +46,12 @@ export default class GeoloniaMap extends maplibregl.Map { | |||
| private __styleExtensionLoadRequired: boolean; | |||
|
|
|||
| constructor(params: string | GeoloniaMapOptions) { | |||
|
|
|||
| if (!isWebgl2Supported()) { | |||
| console.warn('[Geolonia] WebGL 2.0 is not supported, or graphics acceleration is disabled.'); | |||
There was a problem hiding this comment.
これは明示的に入れているので、eslintのワーニングが出ないようにコメントを入れてください
When an error is thrown in renderGeoloniaMap(), subsequent maps will fail to render and show the error dialog. Additionally, the try/catch was shortened to include only the new map (super(options)) and re-throws the initial error object, because at that point the map is useless and should not be used normally. If the error was not rethrown, code using the map wouldn't know if the map was valid or not.
|
以下のように地図表示エリアに警告を表示するようにしました。 PC モバイル 確認方法 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WebGL 2.0 がサポートされているかチェックする機能を追加しました。
WebGL 2.0 を無効にした場合、Chrome でグラフィックアクセラレーションを無効にした場合は以下の表示がされます。
Closes #389