Implementera Fäll ut/Fäll in på komin

[magnivar]

Ta Fäll ut/Fäll in funktionaliteten från Gymnasiewebbarna och impemetera på komin. Först i TEST och sedan förbereda redaktörer för rätt användande.

[EllinorHallett]

Tyvärr verkar jag inte kunna logga in på test för komin med de inloggningsuppgifterna jag fick av dig @magnivar

[tomasrobertsson]

Efter senaste flytt av komin, från produktion till test ändrades inte lösenordet utan det är samma på prod som test.

[EllinorHallett]

Nu ligger funktionaliteten på plats bortsätt från att det saknas ett klassnamn i SiteVision som sedan ska pekas ut på ytan "Center & Dropdownklass" samt att koppla dekorationsmallen. Detta på grund av en bugg i SiteVision som inte tillåter mig att skapa dessa under webbplatsinställningar > Dekorationer. @magnivar vem tar kontakten med SiteVision för att rappotera denna buggen och få dem att åtgärda detta?

[magnivar]

@EllinorHallett jag ser gärna att du förklarar för dem för att de snabbare förstår vad som måste göras.

[EllinorHallett]

Jag har skickat ett mail till supporten och bett dem skicka svar till mig med dig, @magnivar som CC.

[magnivar]

Jag har inte fått någon respons från SV om dettta men det kanske är @badwiz som tar bollen vidare?

[badwiz]

@bjornhutter Kan du se om du skulle kunna göra denna insats på Komin-test? Alltså se till att menyn fungerar som på Gymnasiewebbarna? Men om det inte fungerar (enligt Ellinor) så återkoppla så tar jag det med SiteVision.

[badwiz]

Vi har kollat runt på detta och ser att vi kan få till en liknande meny på 16 timmar på Komin.

[badwiz]

Vi gör detta på test-webben innan det hamnar på Komin.

[badwiz]

Anders Israelsson kommer att kika lite på detta om det finns tid när han håller på med Förvaltningssidorna. Annars lägger vi tid på detta efter Förvaltningssidorna.

[badwiz]

http://latinskolan.malmo.se/utbildningar/estetiska-programmet.html Exempel på Fäll ut / Fäll in, accordion-funktion på innehållet.

[henrikekelof]

Hej @magnivar + @badwiz ! Det här var inte en SiteVisionbugg, det här handlade om att det låg ett par CSS-regler med felaktiga class-namn. Jag har fixat så att den här funkar på test nu: http://161.52.13.184/edit/4.79ff2ce11606aebb8172a3f3

[henrikekelof]

@magnivar Fråga: Vill du att jag stänger ett sådant här ärende när det är fixat, eller vill du kolla först och själv stänga sen?

[magnivar]

@henrikekelof Assigna det på oss (mig) när du är klar så kollar vi och godkänner för produktionssättning och sen test av oss igen för acceptans... Det känns mest rätt eller har @badwiz någon annan tanke?

[badwiz]

@magnivar @henrikekelof Håller med Magnus. Vi gör klart och sedan lägger vi över ärendet på Malmö (Magnus eller någon annan om vi vet vem det är).

[magnivar]

Ser det ut så här hos er också?

[henrikekelof]

Ja, ungefär. Jag tänkte att det inte tillhörde det här ärendet att snygga till den, men jag kan se till att den ser lite bättre ut :)

[henrikekelof]

Nu ska den se lite finare ut också! Kolla om det här känns mer OK:

[magnivar]

@elilun14 @fredrikwittbjer Kan ni kolla sidan och ge feedback? https://komin.test.malmo.se/4.79ff2ce11606aebb8172a3f3.html

[olajoh1]

Designen följer inte WAG

[elilun14]

Funktionen i sig fungerar, men färg, typsnitt och storlek på texten i rubriken behöver anpassas till vad som gäller för Komin (vilket bör finnas dokumenterat här på Github någonstans...?).

[henrikekelof]

Inga problem! Jag uppfattade det som att den såg rätt ut och att det bara var problem med att den inte fungerade. Men jag kan göra om utseendet enligt WAG som jag hittat här: https://malmostad.github.io/wag-intranet-v3/

[henrikekelof]

Testsidan är uppdaterad med ny funktion/utseende.

Jag gjorde så att fäll ut-/in-funktionalitet är inaktiverad i redigeringsläget så att det blir lättare för redaktören att se texten och redigera den.

https://komin.test.malmo.se/4.79ff2ce11606aebb8172a3f3.html

Ping: @olajoh1 @elilun14 @fredrikwittbjer

[elilun14]

Det behöver vara ett plus istället för den lilla pilen, för att man ska förstå att det går att klicka och att det finns något "under". (Det må vara att pilen finns i den tidigare komindokumentationen men vi vet att den inte förstås, så använd ett plus.)

[henrikekelof]

@elilun14 Har nu ändrat till ett plus som blir ett kryss för att stänga. Testa igen och kolla om det ser OK ut.

[elilun14]

@henrikekelof Bra. Dock ska "fäll ihop" illustreras som ett minustecken och inte ett X. (Kryss är lika med "ta bort")

[fredrikwittbjer]

@henrikekelof Rubriken måste göras tydligare för att öka call-to-action-värdet. Kan vi feta texten (till exempel via att använda H3)?

[henrikekelof]

@fredrikwittbjer OK, vill ni då att vi avviker från er WAG här? Jag hittar inte någon box där med fet stil https://malmostad.github.io/wag-v4/typography_and_articles/#article-boxes

@elilun14 Håller inte med om att kryss betyder ta bort, men jag ändrar :)

[fredrikwittbjer]

Testa att använda denna, där bakgrundsfärgen ändras. Då kanske inte texten behöver fetas, men man kan ändå urskilja att det är en call to action. :)

[henrikekelof]

Status just nu > Har gjort fet stil i rubrik och +/-.

[henrikekelof]

@fredrikwittbjer @elilun14 Är det OK med senaste utseendet eller vill ni göra ytterligare ändringar?

[magnivar]

@fredrikwittbjer @elilun14 Är det OK med senaste utseendet eller vill ni göra ytterligare ändringar? :)

[elilun14]

Hej! Den ser märklig ut på test, eller har jag inte senaste länken? https://komin.test.malmo.se/4.79ff2ce11606aebb8172a3f3.html

[henrikekelof]

@elilun14 Nej, sorry. Det var en miss av mig. Kolla igen :)

[elilun14]

Tack @henrikekelof - jag tycker det ser bra ut nu. @magnivar @fredrikwittbjer

[fredrikwittbjer]

Håller med! @magnivar @elilun14 @henrikekelof

[henrikekelof]

Fint! Ska jag flytta över det här till komin Prod då @magnivar ?

[magnivar]

Jag gör ett nytt issue för malmo.se så kan @perber14 först komma med sina grafiska (färger) val. Lägg detta på PROD och bolla över till mig så verifierar vi och stänger denna.

[perber14]

@elilun14 @fredrikwittbjer Varför har ni inte samma färg på plattan som de här har? Kommer synas mycket bättre?

[henrikekelof]

Jag valde bort den mörka plattan p.g.a det som står i WAG: "This type of box must not be used in article content" https://malmostad.github.io/wag-v4/other_page_elements/#boxes-for-application-views

För den ljusa boxen står det "A box suitable for related content": https://malmostad.github.io/wag-v4/typography_and_articles/#article-boxes

Det är fullt möjligt att jag har tolkat avsikten fel, men det är anledningen. Vem tar beslutet kring mörk eller ljus?

[elilun14]

@henrikekelof Helt rätt med den ljusa för Komin, så vi kör på den. (@perber14 Säger intranätansvarig :-).)

[perber14]

o7

[henrikekelof]

Nu finns rutan i prod för Komin. Testsida: https://komin.malmo.se/4.132262a160fdc3380f9bd69.html

[magnivar]

@fredrikwittbjer verifierar :)

[fredrikwittbjer]

Ser bra ut! En liten grej bara @henrikekelof: rubriken fetas i redaktörsläget. Går det att fixa?

[henrikekelof]

@fredrikwittbjer Ah, bugg! Tanken är att den ska vara fet i båda lägena enligt det vi pratade om häromdagen.

[henrikekelof]

Nu fixat så inte fet stil faller bort.

[fredrikwittbjer]

hej igen @henrikekelof, förlåt men det måste blivit ett missförstånd. Vi skulle gärna se att den inte har en fetad text. Annars är vi mycket nöjda och anser att så fort texten är tillbaks i regular så blir det slutversionen :)

[henrikekelof]

Testa igen nu. Sorry. Jag hängde inte med i alla turerna.

[elilun14]

Tack, helt rätt så @henrikekelof - slutversion!