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

Zoom tools button too small #137

Closed anderserla closed 9 years ago

anderserla commented 9 years ago

In some occations the zoom tool buttons get rendered too small

ghost commented 9 years ago

Felet kan reproduceras om man byter ut ../resources/ext-theme-oep/oepTheme-all.css i /dev/debug.html mot https://kartatest.e-tjansteportalen.se/libs/ext-theme-oep/oepTheme-all.css att göra det motsatta verkar inte avhjälpa problemet. Vidare undersökning för att hitta grundorsaken pågår.

ghost commented 9 years ago

Om man kopierar in https://kartatest.e-tjansteportalen.se/libs/ext-theme-oep/oepTheme-all.css försvinner felet vilket tyder på att felet inte har med cssen att göra egentligen.

Felet uppstår högst troligen pga. kod i ZoomTools.js som innebär att den styr parametern scale på Ext-komponenterna baserat på om den hittar css klassen .oep-tools eller inte. Jag har hittat att denna kod skrevs med syftet att ZoomTools ska se bra ut både med oep temat och Ext JS default tema (Neptune). Troligen hittar inte koden .oep-tools om CSSen inkluderas från annan domän än den sida som kör koden.

För att komma tillrätta med felet finns följande alternativ:

  1. Lägg OpenEMap på samma server som verksamhetssystem
  2. Ta bort anpassning för Neptune
  3. Hitta annan lösning för att det ska fungera för både oep tema och Neptune
  4. Fixa oep tema att fungera både med medium och large
ghost commented 9 years ago

Hittat belägg för mitt antagande att det är cross domainproblematik: http://stackoverflow.com/questions/3211536/accessing-cross-domain-style-sheet-with-cssrules

anderserla commented 9 years ago

Om jag lägger in oepTheme-all.css inuti SItevisionsidan för brandlarm, så försvinner problemet -> CORS problem.

anderserla commented 9 years ago

Fixed in 1.5.0-hf.2