[bug] Notification icons verkeerd

[.oisyn]

In Chrome 52.0.2743.82 m

Als ik de elementen inspecteer dan lijkt me dat de coordinaten verkeerd zijn. De eerste notification heeft een background-position van -20px -108px door de ".notificationType.thumbsup" rule. Dat moet -20px -126px zijn.

[Inspector]

.oisyn schreef op maandag 15 augustus 2016 @ 16:47:
[afbeelding]

In Chrome 52.0.2743.82 m

Als ik de elementen inspecteer dan lijkt me dat de coordinaten verkeerd zijn. De eerste notification heeft een background-position van -20px -108px door de ".notificationType.thumbsup" rule. Dat moet -20px -126px zijn.

Die sprite is aangepast laasts en de cache gecleared. Ik denk dat hij nog in je localcache zit. Kun je hem eens hard refreshen?

[.oisyn]

Heb ik al 5x gedaan

.edit: nu met devtools "empty cache and hard reload" gedaan en nu werkt het wel . Wel vreemd dit, zou het kunnen dat er verkeerde metadata wordt gereturned door jullie servers?

[ Voor 80% gewijzigd door .oisyn op 15-08-2016 17:30 ]

[crisp]

.oisyn schreef op maandag 15 augustus 2016 @ 17:27:
Heb ik al 5x gedaan

.edit: nu met devtools "empty cache and hard reload" gedaan en nu werkt het wel . Wel vreemd dit, zou het kunnen dat er verkeerde metadata wordt gereturned door jullie servers?

Het is een background-image in css. Wellicht dat die niet op een harde reload reageren?

Reminder voor onszelf om de volgende keer wel de bestandsnaam aan te passen iig

[AW_Bos]

Waarom niet met een GET-parameter werken in de bestandsnaam die je per iteratie ophoogt. Dan zit bijna niemand meer met verouderde sheets en dus ook sprites.

Of is dit al weer achterhaald?

[crisp]

AW_Bos schreef op maandag 15 augustus 2016 @ 19:53:
Waarom niet met een GET-parameter werken in de bestandsnaam die je per iteratie ophoogt. Dan zit bijna niemand meer met verouderde sheets en dus ook sprites.

Of is dit al weer achterhaald?

Voor de stylesheets zelf doen we dat al (timestamp). Voor plaatjes binnen een stylesheet zou dat inderdaad ook wel werken. Filenaam aanpassen is echter wel meer 'safe' volgens mij.

[Raven]

crisp schreef op maandag 15 augustus 2016 @ 19:17:
Reminder voor onszelf om de volgende keer wel de bestandsnaam aan te passen iig

Semi-offtopic: Misschien ook handig om dat met profielfoto's te doen? Daar zijn af en toe ook wel eens cache-issues mee.

[crisp]

Raven schreef op maandag 15 augustus 2016 @ 20:24:
[...]

Semi-offtopic: Misschien ook handig om dat met profielfoto's te doen? Daar zijn af en toe ook wel eens cache-issues mee.

Ja, ware het niet dat dat een heel ander stuk (verouderde) code is... En iets met prio en schaarse tijd enzo...

[Raven]

crisp schreef op maandag 15 augustus 2016 @ 20:34:
[...]

Ja, ware het niet dat dat een heel ander stuk (verouderde) code is... En iets met prio en schaarse tijd enzo...

Maar het kan geen kwaad daar een ticket voor te maken, als die er al niet is

[crisp]

Raven schreef op maandag 15 augustus 2016 @ 20:35:
[...]

Maar het kan geen kwaad daar een ticket voor te maken, als die er al niet is

Die is er al (al heel lang...)

[Hero of Time]

Hebben anderen met de notificatie iconen overigens bij de gelezen notificatie van een quote nog een pixel van het ongelezen icoon zichtbaar?

Wat ik zie, maar dan 13x ingezoomd:


Voor m'n custom css, ik pas alleen de achtergrondkleur aan.

[Raven]

crisp schreef op maandag 15 augustus 2016 @ 20:40:
[...]

Die is er al (al heel lang...)

Ik durf het haast niet te vragen, hoeveel jaar?

[.oisyn]

Hero of Time schreef op maandag 15 augustus 2016 @ 20:42:
Hebben anderen met de notificatie iconen overigens bij de gelezen notificatie van een quote nog een pixel van het ongelezen icoon zichtbaar?

Wat ik zie, maar dan 13x ingezoomd:
[afbeelding]

Voor m'n custom css, ik pas alleen de achtergrondkleur aan.

Yup same here

[Hero of Time]

.oisyn schreef op maandag 15 augustus 2016 @ 21:05:
[...]


Yup same here

[afbeelding]

Had even snel m'n custom uitgeschakeld en zag 't daar ook. Maar ik zie ook waar het eigenlijk hoort:

Hier mist een pixel. Het zou mij niet verbazen als het met de andere iconen ook zo is, alleen valt het hier meer op.

[crisp]

Hero of Time schreef op maandag 15 augustus 2016 @ 20:42:
Hebben anderen met de notificatie iconen overigens bij de gelezen notificatie van een quote nog een pixel van het ongelezen icoon zichtbaar?

Wat ik zie, maar dan 13x ingezoomd:
[afbeelding]

Voor m'n custom css, ik pas alleen de achtergrondkleur aan.

Die heb ik zojuist lokaal opgelost

[Hero of Time]

crisp schreef op dinsdag 16 augustus 2016 @ 10:54:
[...]

Die heb ik zojuist lokaal opgelost

Top!

[crisp]

En staat ook online inmiddels