DRB-IT / blacktiger

Blacktiger server code written in Java.
2 stars 1 forks source link

Kosmetisk: IE9 viser ekstra knapper, og view ikke pænt hvis vinduet er smalt på IE9 & IE11 #27

Closed xToMMeRx closed 11 years ago

xToMMeRx commented 11 years ago

Når siden vises i IE9 dukker der nogle ekstra knapper op (som ikke er på IE10 og IE11 og som ikke er nødvendige) f.eks. til valg af sang i musik playeren, og til valg af antal minutter på rapport siden (hvor knappen i øvrigt står helt forskudt i forhold til linjen)

Derudover hvis man editerer navne mens vinduet f.eks. er snappet til siden og kun er halv bredde, så falder teksten med navnet uden for feltet, og godkendt og annuller knapperne ligger derfor oven i navnet. (Dette er galt på både IE9, 10, 11)

lyttere-ie9 ie11 rapport-ie9 ie11

xToMMeRx commented 11 years ago

De ekstra knapper der dukker op på IE9, vises også på IE8 (nyeste browser man kan køre på XP, som mange stadig kører i rigssalene)

IE8 ser ud til slet ikke at tilpasse siden, som f.eks. at indskrænke felterne når man laver vinduet mindre.

michaelkrog commented 11 years ago

ok.. Jeg havde egentlig forsøgt at fjerne musikafspilleren i browseren der ikke understøtter Audio objektet i javascript, hvilket i bund og grund vil sige IE før version 9. Man kan slet ikke bruger musikafspilleren i IE8.

IE8-brugere skal selvfølgelig kunne bruge telesal, men jeg vil foreslå at vi anbefaler at XP-brugere installerer Chrome hvis de vil have fuld gavn af den nye brugerflade.

kasperdyrvig commented 11 years ago

Den nye brugerfalde benytter CSS3 og HTML5. CSS3 har den nye funktion "media-query" som holder øje med hvor stor skærmen er og benytter så det css-kode der passer til. Helt ligesom på jw.org. Vi har lagt tracking på v2, så vi kan få data om hvad rigssalene faktisk bruger. Men jeg håber da at menighederne i det store hele følger Rigssalsforeningens anbefalingen om at køre Windows 7 med den nyeste IE browser...?

kasperdyrvig commented 11 years ago

Et bud på en løsning i forbindelse med navneændring, så vi helt undgår kampatibilitetsproblemer, kunne være simpelt hen at brug javascripts window.prompt() funktion (http://www.w3schools.com/js/js_popup.asp (se under "Prompt Box")). Simpelt, enkelt, en smule grimt... men det virker.

Jeg tænker lidt videre angående tal-inputs

kasperdyrvig commented 11 years ago

En lille ide: Man kan tilføje et meta tag som tvinger IE til at bruge den nyeste standarder browseren har tilgængelig. Tilføj meta http-equiv="X-UA-Compatible" content="IE=edge" i headeren (jeg kan ikke indsætte koden ordenligt her). Det vil ikke løse alt, men måske noget...

xToMMeRx commented 11 years ago

Det kunne være skønt hvis alle menighederne kørte med nyeste IE på Windows 7 eller nyere, men det er desværre kun ca. 30% der gør det i.

Der kan jo være mange grunde til de ikke kører nyeste version. Windows 7 var ikke var udkommet da vi implementerede Telesal. Det er ikke alle steder de har nogen der har forstand på IT, eller også glemmer dem som er teknisk ansvarlig at opdatere pc'en. Nogen steder har en minipc som f.eks. ASUS Eee, som kører nogenlunde på WinXP, men ville drukne med Win7. Det er jo også et spørgsmål om at hver menighed skal investere nogle penge i ny pc, eller i ny Windows licens.

Her er lidt statistik fra telesal serveren i søndags: 1% kørte Win 8 med IE10 29% kørte Win 7 med IE10 14% kørte Win 7 med IE9 4% kørte Win 7 med IE8 9% kørte Win 7 med IE7 5% kørte Vista med IE9 9% kørte Vista med IE7 9% kørte WinXP med IE8 5% kørte WinXP med IE7 9% kørte Chome 3% kørte Firefox 1% kørte Opera 1% kørte Safari

Jeg tror desværre ikke vi kommer helt udenom at skulle sikre at websitet kører ordentligt på i hvert fald de mest brugte browsere. @paf61 hvad siger du? Skal vi stille nogle krav til menighederne i forhold til browsere?

kasperdyrvig commented 11 years ago

Jeg blander mig lige igen... Jeg vil mene at man i det mindste kan sende en påmindelse ud til de teknisk ansvarlige om det ville være en rigtig god ide at de sørger for at have det nyest tilgængelige installeret. Også af sikkerhedshensyn.

Vi skal i hvert fald have dem der bruger IE7 til at opgradere! Den kan vi overhovedet ikke understøtte med den nye brugerflade. Og IE8 understøtter ikke media-queries, så dem der kan installere en nyere version vil få en meget bedre oplevelse ved at opgradere.

Jeg ved at nogle få menigheder kører med Linux maskiner. Hvordan det nye design klarer sig i browsere her ved jeg ikke endnu.

michaelkrog commented 11 years ago

Dem der kører linux kører enten Chrome, Firefox eller Opera. Hvad angår Firefox og Chrome, så er jeg meget sjældent stødt på betydelige forskelligheder mellem Windows, Linux og Mac. Derhjemme har jeg dog kun testet på Mac(Firefox, Safari, Chrome) og en smule på Windows(IE8 og IE9).

xToMMeRx commented 11 years ago

Jeg har lige drøftet browsere og styresystem med Peter.

Vi er blevet enige om at vi skal supportere og teste af det virker og ser optimalt ud på Windows 7 IE10 og fremad(dvs. Win8 IE11 og Win8.1 IE12) Funktionalitetsmæssigt skal websitet virke på WinXP IE8 (nyeste browser til XP), men om det ser helt korrekt ud/står rigtigt på siden er ikke vigtigt da vi officielt kun vil supportere Win7 og frem.

Derfor kan vi skippe punktet med de ekstra knapper på IE9, men blot fokusere på teksten der falder uden for kassen på IE10 og nyere.

kasperdyrvig commented 11 years ago

Vil vi gøre noget for at gøre de teknisk ansvarlige i menighederne opmærksom på dette? Vi kan nemt ligge en besked på status siden, men så kan alle der sidder ved mixeren i pricippet begynde at pille og det er de teknisk ansvarlige nok ikke interesseret i... Flyttet til #32

michaelkrog commented 11 years ago

@webspas Kan du ikke oprette et nyt issue med spørgsmålet om en besked på statusside?

Så kan vi få det her issue tilbage i fokus: Teksten falder udenfor kassen ved editering når vinduet er smalt.

kasperdyrvig commented 11 years ago

Jo, det er gjort. Jeg tester lidt senere om det er et problem der opstår i de browsere vi understøtter. Indtil da kan du/I tænke over mit forslag som er nævnt 9 kommentarer oppe, omkring brug af window.prompt().

michaelkrog commented 11 years ago

Jeg stemmer for window.prompt(). Det vil samtidig også gøre det til en bedre oplevelse på en mobil enhed eller tablet.

michaelkrog commented 11 years ago

window.prompt() er taget i anvendelse med https://github.com/DRB-IT/blacktiger/issues/41. Så denne kan lukkes når https://github.com/DRB-IT/blacktiger/issues/41 er lukket.