search

VNG-Realisatie / Begrafenisplanner

3 stars 5 forks source link

Bug: Als gebruiker vind het verwarrend om een begrafenis te melden bij de vng #100

Open rubenvdlinde opened 4 years ago

rubenvdlinde commented 4 years ago

Maar er staat nog wel een vng favicon in beeld

favicon

@rjzondervan dit moeten we oprollen door het in de proto applicatie inregelbaar te maken en vervolgens in het wrc in te regelen

YorickGroeneveld commented 4 years ago

image Bij ons lijkt een andere Favicon getoond te worden en we denken dat dit samenhangt met het svg logo.

matthiasoliveiro commented 4 years ago

ik krijg een 500 spookje nu als FavIcon. Granted het is niet de VNG maar dit is nog niet goed genoeg.

Hier moet een westfriesland favicon komen

rjzondervan commented 4 years ago

Ik heb gisteren de nginx container gereset. Herladen of forced reload zou dit moeten weergeven.

PatrickCastenmiller commented 4 years ago

De VNG favicon is er nog steeds

rjzondervan commented 4 years ago

Hoi Patrick, zou je kunnen aangeven waar je het VNG favicon krijgt? Ik heb alle frontends nagekeken, en zo lang ik geen errors trigger krijgt ik de westfriesland favicon... Het kan zijn dat als jij nog wel een VNG favicon krijgt dat er ergens een browser cache vast zit...

PatrickCastenmiller commented 4 years ago

@rjzondervan In de (volgens mij) oude edge image

rjzondervan commented 4 years ago

Zou je voor de grap eens willen kijken wat er gebeurt als je je cache leeggooit of in een InPrivate-window de planner opent? Dit kan haast niets anders zijn dan een vastzittende cache.

PatrickCastenmiller commented 4 years ago

Nou, daar komt de clou :-) image

PatrickCastenmiller commented 4 years ago

oh, dit is in demo. ik probeer het ook nog even in dev

PatrickCastenmiller commented 4 years ago

Daar is geen favicon, dus ook geen VNG) image

rjzondervan commented 4 years ago

Hmm, dan weet ik niet helemaal wat hier gebeurt, maar ik vermoed dat de oude edge het format van de westfriesland-favicon niet slikt. Nu heb ik hier geen oude versie van edge om dit te testen (alles is al door Microsoft omgezet naar de nieuwe versie van edge) waar de favicon wel het westfriesland logo is...

Ik ga vanmiddag heel snel even kijken of ik hier toevallig iets over terug kan vinden.

rjzondervan commented 4 years ago

Oke, dat was sneller uit te vissen: Punt is hier dat Edge het type favicon dat is gebruikt (svg) pas sinds versie 80 ondersteund, waar de oude Edge (op basis van EdgeHTML) een versie heeft die lager is dan 74. Dit is dus een stukje browsercompatibility met oudere browsers, dan is de vraag denk ik vooral hoe veel tijd we willen besteden aan het repareren van een kleine grafische bug voor oudere browsers.

PatrickCastenmiller commented 4 years ago

Het is niet nodig om oude browserversies hier op te ondersteunen. Het Favicon is nuttig maar ook weer niet zo nuttig. We kunnen deze naar demo deplorabele en na testen sluiten