WebGL: Warnung wenn deaktiviert - #97
Hidden character warning
JakobMiksch wants to merge 10 commits into
Conversation
JakobMiksch
changed the title
joto
left a comment
joto
left a comment
There was a problem hiding this comment.
Kannst Du einen rebase machen (und nur einen commit), statt den main branch reinzumergen? Dann liest sich die commit-History nachher viel leichter.
Was die Meldung von gitmap mit sagen soll Hidden character warning
The head ref may contain hidden characters: "93-deaktivertes-webgl-sollte-zu-einer-klaren-und-verst\u00e4ndlichen-fehlermeldung-f\u00fchren` hab ich noch nicht verstanden. Mag er Umlaute nicht?
| thumbnail: "oepnv.png", | ||
| }, | ||
| }; | ||
| const setupMap = () => { |
There was a problem hiding this comment.
Kann das nicht eine normale Funktion sein?
There was a problem hiding this comment.
könnte schon, aber in dem Fall ist es egal. const setupMenu = () => {} ist auch schon so definiert. Dann würde ich eher die Funktion die WebGL check auch als Arrow Funktoin machen damit es einheitlich ist
| const canvas = document.createElement("canvas"); | ||
| const gl = canvas.getContext("webgl"); | ||
| return gl instanceof WebGLRenderingContext; | ||
| } |
There was a problem hiding this comment.
Die Funktion ist schon ganz schön verbose für den kleinen Check, den sie macht. Wir liefern das JS ja nachher so aus, da lohnt es sich schon, darauf ein bischen zu achten. Und das Kommentar könte auch weg und stattdessen in die commit msg.
There was a problem hiding this comment.
ich habe die Funktion jetzt möglichst klein gemacht. Wobei ich nicht weiß ob das wirklich ins Gewicht schlägt. Unsere beiden externen Libs haben zusammen schon 1MB Speicher. Ein paar Zeilen mehr Kommentar sind da glaub ich nicht so dramatisch
… 93-deaktivertes-webgl-sollte-zu-einer-klaren-und-verständlichen-fehlermeldung-führen
|
@joto ich habe Änderungen gemacht. Den PR würde ich am Schluss "squashen", damit es nur ein Commit wird. |
2 participants
Uh oh!
There was an error while loading. Please reload this page.