favicon toegevoegd (#22)

dsprenkels / kn-next

Edge for a student's club homepage

0 stars 2 forks source link

favicon toegevoegd (#22) #40

Closed dsprenkels closed 10 years ago

aykevl commented 10 years ago

Mooie favicon. Een paar opmerkingen:

Deze gaat tot 64x64 pixels, is dat niet een beetje veel?
PNG is veel kleiner, en is ondersteund in IE11+ en alle andere browsers. Of PNG-binnen-ICO. Dan kun je PNGs van verschillende groottes opsturen, maar ik weet niet hoeveel browsers dat ondersteunen (ik meen me te herinneren dat Opera dat niet goed pakte).
Het heeft mijn voorkeur als de favicon in de img/ directory komt, bij de andere afbeeldingen. Dat is ook handiger voor de rewrite rules (hoef je geen uitzondering voor favicon.ico te maken).
Voor oudere Internet Explorer versies is het nodig "shortcut icon" te gebruiken in plaats van "icon", maar ik weet niet precies vanaf welke versie dat is.
Het kan misschien een optie zijn de kleinste versie (16x16) handmatig in te kleuren, dan zijn de lijnen wat strakker. Zie bijvoorbeeld wat ik gepost heb in de bugreport (issue #22).

dsprenkels commented 10 years ago

Ik ben het op veel punten niet echt met je eens:

Ik kan de 64x64 eruit nuken, dat is misschien wel een goed idee.
Alle browsers onsdersteunen (voorzover ik weet) een ICO-format, dus het lijkt me in verband met cross-compatibility onhanding om een PNG erin te zetten, omdat het sneller is. Dit maakt voor een icon van 32x32 pixels niet uit, vergeleken met bijvoorbeeld de headerfoto.
Zelfs zonder icon-regel checkt een browser op /favicon.ico voor een favicon. Dit is slecht gedrag, maar het lijkt me beter dit te ondersteunen. Ook al moet je er een regel voor bouwen.
Ik denk dat de 16x16 er nu ook prima uitziet.

aykevl commented 10 years ago

Ik heb het net gecontroleerd. Als het een icoontje is van 16x16 en een van 32x32, die samen in een ICO zitten, dan is die (na gzip!) 48 bytes groter dan de PNG-in-ICO versie. Dus het is het inderdaad echt niet waard. Het klopt dat praktisch alle browsers het ICO formaat ondersteunen (al zijn er hier en daar wel wat kleine probleempjes*).
Als het goed is, checken browsers die een favicon in de HTML zien niet op /favicon.ico. Ik denk nog steeds dat het beter is de favicon in /img te zetten en een rewrite rule te gebruiken, voor consistency.
Dan houden we de 16x16 versie zoals die is. Dit kan altijd nog eenvoudig later aangepast worden (daarmee verandert het design niet).

Is er een reden dat je eerst de 32x32 versie hebt en daarna de 16x16 versie? Ik heb ze zelf net omgedraaid (met icotool). (Ik probeerde hem te uploaden maar GitHub snapt geen .ico bestanden)

* Sommige browsers pakken de 16x16 versie als ze de 32x32 versie zouden moeten hebben of andersom.

dsprenkels commented 10 years ago

Ik zal de 64x64 nuken. Ik heb hem met The Gimp gemaakt, en daar is de volgorde voor zover ik weet nog random.

Je hebt me overtuigd over de locatie van het favicon.

dsprenkels commented 10 years ago

Mergen?