malmostad / sitevision_assets

Sitevision specific assets used in addition to Global Assets
http://malmostad.github.io/wag-external-v4/
GNU Affero General Public License v3.0
0 stars 1 forks source link

<br> saknas mellan bilder i kontaktrutan på Komin? #328

Closed Svempan closed 7 years ago

Svempan commented 9 years ago

Henrik, jag tror du förstår problemet i texten från Tove nedan. Du kan också kolla med Mårten om du behöver:

"Jag har frågor om kontaktboken och kontaktmodulens visningsbild på Komin men vet inte i nuläget vart jag vänder mig. I kontaktmodulen tillkom för en tid sedan funktionen att kunna välja visningsbild på person i kontaktrutan. Detta fungerar fint så länge det är en bild eller bilder i rätt format (beskurna till kvadratiskt format likt vinjettbilden som visas i en kontaktlista!) innan de läggs upp i kontaktboken. Vill jag visa flera bilder i kontaktrutan och flera bilder är ”för höga” så störs layouten helt. Detta borde fungera per automatik och korrelera med kontaktboken så att bilder visas på samma sätt oavsett format. Vem vänder jag mig till för att ställa frågor om detta?

Sida där bilder i kontaktboken funkar – men bara för att båda bilderna är kvadratiska: https://komin.malmo.se/Forvaltningar/Grundskoleforvaltningen/Stod-och-service-i-grundskoleforvaltningen/Kommunikation-och-marknadsforing/Webbarbete.html

För övrigt ser det mestadels ut så här när visningsbilder visas, trots litet olika format, på flera personer i kontakt: https://komin.malmo.se/Forvaltningar/Grundskoleforvaltningen/Var-organisation/Utbildningsomraden/Utbildningsomrade-Oster.html https://komin.malmo.se/Forvaltningar/Grundskoleforvaltningen/Var-organisation/Utbildningsomraden/Utbildningsomrade-Samordning-och-stod.html "

hryd commented 9 years ago

Deployed fix i komin-test, återskapade en av sidorna du länkade, se här http://komin.test.malmo.se/4.2b56f4113ec05ff84f1f5cd.html.

Testa och återkoppla @Svempan.

olajoh1 commented 8 years ago

Sven, du måste återkoppla!

Svempan commented 8 years ago

Ser bra ut. Kör ut i prod vid tillfälle.

hryd commented 8 years ago

Ute i produktion. @olajoh1 @Svempan

olajoh1 commented 8 years ago

bilderna linjerar ej med texten samt att namn renderas före bild

nikjon76 commented 8 years ago

För att inte skriva över befintliga klasser i CSS behöver vi uppdatera malmo.css som ligger i Global Assets.

Svempan commented 8 years ago

Kan ni göra det?

susjon11 commented 8 years ago

Mårten tittar på det

martent commented 8 years ago

Jag har gjort en ändring i Global Assets för detta.

@nikjon76, följande måste göras i Sitevision och sitevision_assets för att formateringen ska bli korrekt där:

För att se ordningen på markupen, fetstilen och korrekt linjering av bilderna, se prototypen: http://malmostad.github.io/prototypes/internal/clean/

Olas och Susannes bilder har vita pixlar överst i bilden vilket kan vara lite vilseledande för top-linjeringen, kolla jespers lite längre ned.

magnivar commented 8 years ago

Kollar om det fortfarande är aktuellt.

tomasrobertsson commented 7 years ago

Jag måste medge att jag här inte vet vad det är ni vill ska ske. eller rättare sagt jag kan på komin inte se hur det ser ut idag (pga av att jag via VPN inte får några formtmallar alls på den sidan/kan inte ladda några externa resurser) och därmed inte heller se vad det är som ska ändras. Testar på test-komin och lägger in formatmallarna och JS direkt där, då får jag format och kod som körs men på motsvarande sida där http://161.52.13.184/Forvaltningar/Grundskoleforvaltningen/Var-organisation/Utbildningsomraden/Utbildningsomrade-Oster.html laddas inte data från https://webapps06.malmo.se/avatars-test/fahzej/small.jpg dvs jag tror sidan saknar den miniatyr som problemet rör.

Dvs jag behöver en ny problembeskrivning, en bild/sida som visar på felet (där ni visar vad som är fel) och då även på hur det ska se ut istället. Eller att ni sätter upp så att er testerver, när jag går via VPN, når er asset-server och även webapps06.

martent commented 7 years ago

@magnivar Be ITS se till så att Thomas når den publika servern assets.malmo.se när han är på VPN.

tomasrobertsson commented 7 years ago

Och även webapps06 tror jag bestämt då ni har resurser även där.

magnivar commented 7 years ago

Har beställt access nu till båda.

magnivar commented 7 years ago

Tomas ska nu ha access till både assets och webapps06

tomasrobertsson commented 7 years ago

Tackar, nu flyter sidorna på mycket bättre.

tomasrobertsson commented 7 years ago

Denna har jag nog löst men oklart för mig hur jag gör deploy av css i ert system. Vet att vi gick igenom det vid startmötet men bäst att repetera. Ändringen är i två steg. 1: Flytta på bildelementet i sv_contacform 2; Lägga till bold till vcard h2 a i CSS.

men hur deployar jag ny css?

martent commented 7 years ago

Du bygger en war-fil och deployar som servlet i SV. Kort + fullständig dokumentation:

https://github.com/malmostad/sitevision_assets/

tomasrobertsson commented 7 years ago

Då ligger det uppe på komin test. Som vanligt vid CSS-ändring måste man göra en refresh av sidan för att den ska laddas. https://komin.test.malmo.se/Forvaltningar/Grundskoleforvaltningen/Var-organisation/Utbildningsomraden/Utbildningsomrade-Oster.html exempelvis. Ser väl ut som ni önskat? @magnivar och @martent ber om ursäkt för att jag inte hunnit detta tidigare.

martent commented 7 years ago

Ser rätt ut på sidan och i koden. Man borde nog göra ett mer provocerande exempel för att testa av det helt, med flera kontakter med olika format på bilderna typ de som finns på:

http://malmostad.github.io/prototypes/internal/clean/

tomasrobertsson commented 7 years ago

Har ni en sida med det så... annars lägg upp en och se hur det blir. Ni är mer bekanta med funktionen/användningen av er webb. Jag tittar på det på måndag.

tomasrobertsson commented 7 years ago

Två dummy-bilder https://komin.test.malmo.se/4.2b56f4113ec05ff84f1f5cd.html hittar ingen sida, lyckas inte skapa en sida, där det finns riktiga bilder på test.

tomasrobertsson commented 7 years ago

http://komin.test.malmo.se/testarea/kontakt.html

tomasrobertsson commented 7 years ago

Tagit bort fixen, 158b418, även. Påverkade inte sidan.

magnivar commented 7 years ago

Är lite osäker på om denna är kvar i test eller ute i prod. @tomasrobertsson @Svempan

tomasrobertsson commented 7 years ago

denna ligger på komin.malmo.se Inga ändringar gjorda som ska påverka på externa webben så har inte byggt/publicerat något där.

magnivar commented 7 years ago

@elilun14 Kan du bekräfta att detta är löst så kan vi stänga ärendet? äntligen...

elilun14 commented 7 years ago

Ok att stänga detta. @magnivar

magnivar commented 7 years ago

@elilun14 du kunde ju stängt också... ska formulera mig annorlunda nästa gång 💃