search

Sundsvallskommun / OpenEMap-WebUserInterface

Open eMap user interface for the main map client
GNU Affero General Public License v3.0
4 stars 5 forks source link

Map tiles get white #58

Closed anderserla closed 10 years ago

anderserla commented 10 years ago

Do the following

  1. http://194.71.24.168/OpenEMap-1.0.3/
  2. Choose map config Anna2
  3. Turn on layer Planområde
  4. Zoom in using mouse wheel -> ERROR Tiles of layer Planområde covers background map with white tiles. See image: image
anderserla commented 10 years ago

If I press F12 to open debug window it looks correct: image

anderserla commented 10 years ago

I've tried with Användningsbestämmelse and Väghållare also, and it works in the same way. It only seem to occur when the web browser window is maximized.

anderserla commented 10 years ago

Same problem occured when I choosed the polygon tool: image

ghost commented 10 years ago

Detta fel påverkar så vitt jag vet endast Chrome och introducerades för några versioner (av Chrome) sedan. Jag tror det går att komma runt...

ghost commented 10 years ago

Problemet beskrivs här: https://github.com/openlayers/openlayers/issues/1181. Jag kan få det att funka om man tar bort vissa CSS klasser från OpenLayers egen CSS.

anderserla commented 10 years ago

Är det möjligt att skriva över CSS:en med en egen CSS, så att vi slipper göra förändringarna i OpenLayers CSS? Bäst vore ju också om det gick att göra den justeringen enbart för de som surfar in med Chrome.

ghost commented 10 years ago

Jag tror det ska kunna gå, dock mer jobb att få det att overrida enbart för Chrome.

anderserla commented 10 years ago

Då ser jag tre olika lösningar:

  1. Göra korrigering i OL:s CSS, som påverkar samtliga Webbläsare - ingen GPU-acceleration
  2. Göra korrigering i vår CSS, som påverkar samtliga Webbläsare - ingen GPU-acceleration
  3. Se till att enbart Chrome använder den korrigerade CSS:en - GPU-acceleration för alla webbläsare utom Chrome

Det jag undrar då är hur påverkar det prestandan? Kan du tidsskatta de tre varianterna?

ghost commented 10 years ago
  1. 1h
  2. 1-2h
  3. 4h

Jag tror att negativ prestandaskillnad ej kommer att märkas förutom på högupplösta mobila enheter, där panoreringen kan bli mindre mjuk.

anderserla commented 10 years ago

Jag har pratat med Anna, och vi bestämde oss för att gå på lösningsförslag 2.

ghost commented 10 years ago

Plötsligt kan jag inte reproducera felet längre och kan inte hitta någon förklaring till det... (inget ändrat, samma version av Chrome).

anderserla commented 10 years ago

Har du felsökningsverktyget uppe? Då fungerar det. Det är bara när man har expanderat Chrome till hela fönstret och inte har felsöknings/utvecklingsverktyget uppe som det blir fel.

ghost commented 10 years ago

Jo jag har testat med hela fönstret utan debug.. och tidigare kunde jag koppla felet till att det börjar hända efter fönstret är över en viss storlek.

Nu märker jag dock att http://extmaptest.sundsvall.se/ verkar strula med att leverera bilder vilket kanske påverkar det hela. Får "Bad request" som svar hela tiden nu.

anderserla commented 10 years ago

Nu är extmaptest uppe och rullar igen

ghost commented 10 years ago

Tyvärr kan jag fortfarande inte reproducera felet längre.

anderserla commented 10 years ago

Min skärmupplösning är 1920*1200.

ghost commented 10 years ago

Jag har 2560x1440. Det märkliga är att jag kunde reproducera felet för 2 dagar sedan.

anderserla commented 10 years ago

Underligt. Både jag och Anna har problemet fortfarande. Du har inte fått någon patch till Chrome? Jag kör Version 34.0.1847.131 m. Kanske kan du testa med min upplösning och se ifall det gör någon skillnad?

ghost commented 10 years ago

Kunde reproducera felet på Windows och senaste Chrome. Fixat CSS override workaround för verifiering.

ghost commented 10 years ago

Notera att workaround är gjord i /src/main/webapp/resources/OpenEMap.css - jag ser att det finns /resources/css/OpenEMap.css också vet inte om den är relevant. Borde inte behöva ha duplicerad data i repot, det skapar bara oreda.

anderserla commented 10 years ago

Fixen funkar!

anderserla commented 10 years ago

Strukturen är kraftigt förändrad och det är därför viktigt att jobba mot developbranchen, eller releasebranchen. I dessa grenar ligger CSS:erna enbart under resources/css.

ghost commented 10 years ago

Stämmer inte, i develop branches finns följande filer: /src/main/webapp/resources/OpenEMap.css /resources/css/OpenEMap.css

Dessutom finns det någon rest från innan namnbytet: /resources/css/map-client.css

Det är överhuvudtaget (gäller även OpenEMap-Search-backend) mer rörigt än förut tycker jag så jag har lite svårt att förstå varför omstrukturering gjorts.