WebGL 2.0 がサポートされているかチェックする #390
Conversation
naogify
requested review from
a team,
keichan34,
sugama-satsuki and
shi-works
and removed request for
a team
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 モバイル 確認方法 |
| try { | ||
| renderSingleMap(item.target); | ||
| } catch (e) { | ||
| console.error('[Geolonia] Failed to initialize map', e); // eslint-disable-line |
There was a problem hiding this comment.
ここに、なんでre-throwをしていないかをコメントなどで説明してください
There was a problem hiding this comment.
@keichan34
遅くなりましたが、以下のようにコメントを追加しました。お手隙の際に確認お願いしても良いでしょうか？
https://github.com/geolonia/embed/blob/alert-check-webgl/src/lib/render.ts#L73
https://github.com/geolonia/embed/blob/alert-check-webgl/src/lib/render.ts#L93
WebGL 2.0 がサポートされているかチェックする機能を追加しました。
WebGL 2.0 を無効にした場合、Chrome でグラフィックアクセラレーションを無効にした場合は以下の表示がされます。
Closes #389