search

RegionHalland / general

0 stars 0 forks source link

Komponent för puffar #84

Closed JohannaOlin closed 5 years ago

JohannaOlin commented 5 years ago

Beskrivning

Kodandet av komponenten Puffar

Skisser

Acceptanskriterier

JOHNREGIONHALLAND commented 5 years ago

http://demo.regionhalland.se/rh-cdn/styleguide/design-editor.php#rh-article__box

Articles information
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. ut enim ad minim veniam, quis nostrud exercitation ullmaco laboris nisi ut aliquap ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolor eu fugiat nulla pariatur.
Articles information
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. ut enim ad minim veniam, quis nostrud exercitation ullmaco laboris nisi ut aliquap ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolor eu fugiat nulla pariatur.
viktor-sarge commented 5 years ago

Assignade John på denna.

viktor-sarge commented 5 years ago

Måste stämma av med @fridabergentoft så att det är rätt skiss jag tittar på. Men några snabba synpunkter på markupen för rh-article__box

Några snabba funderingar på CSS:en

JOHNREGIONHALLAND commented 5 years ago

Ok... Ska gå igenom det och ändra. Ja jag slopar px och kör med rem, em.

När det gäller table är en tillfällig lösning. Då utility som Roland lagt upp från gamla styleguiden är skadad/inte ok. Vilseledande.

Tror bättre att du ser över när det är inlagt direkt på sajten. Håller på fintjustera rh komponenter och den nya utility från tailwin utanför som vgsajten använder.

Jag återkommer så det inte blir missuppfattning.

John Stenlund RGS IT APP (webbutvecklare : -) john.stenlund@regionhalland.se 072 084 28 77 Fiskaregatan 12.

From: Viktor Sarge - Region Halland notifications@github.com Sent: Thursday, December 27, 2018 1:52:45 PM To: RegionHalland/general Cc: Stenlund John RGS IT APP; Assign Subject: Re: [RegionHalland/general] Komponent för puffar (#84)

Måste stämma av med @fridabergentofthttps://github.com/fridabergentoft så att det är rätt skiss jag tittar på. Men några snabba synpunkter på markupen för rh-article__box

Några snabba funderingar på CSS:en

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHubhttps://github.com/RegionHalland/general/issues/84#issuecomment-450145368, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ao-AuUL3eRORjzcCU-ZTjt6j9sMShVy0ks5u9MKdgaJpZM4Y-ewp.

Så behandlar Region Halland dina personuppgifter - https://www.regionhalland.se/dataskydd

viktor-sarge commented 5 years ago

Ok. Lätt till review när du känner dig färdig.

( Men varken basscss eller tailwind blir egentligen del av den nya RH-styleguide med tanke på att vi sagt att vi bygger komponenter istället för det som Frida designar. Utilityklasser använder vi i undantagsfall. )

JOHNREGIONHALLAND commented 5 years ago

Jo jag vet. Utilityklasser ska lämnas ifred. Därför main.min.css som ni la upp från stylie2 var vilseledande. Rh css kommer att heta rh.comp.css

Jag har uppdaterat vg sajten och gått igenom det mer noggrann och använt nya komponenter, enklare för dig att se fungerande komp på sajten än på sidan jag har slängt in.

Efter att har granskat igenom vgsajten och stage-vg på utv i 2 dagar har jag några pålägg som jag gärna vill ta upp. Kommer att pusha uppdateringar när vi är tillbaka.

John Stenlund RGS IT APP (webbutvecklare : -) john.stenlund@regionhalland.se 072 084 28 77 Fiskaregatan 12.

From: Viktor Sarge - Region Halland notifications@github.com Sent: Friday, December 28, 2018 9:00:57 AM To: RegionHalland/general Cc: Stenlund John RGS IT APP; Assign Subject: Re: [RegionHalland/general] Komponent för puffar (#84)

Ok. Lätt till review när du känner dig färdig.

( Men varken basscss eller tailwind blir egentligen del av den nya RH-styleguide med tanke på att vi sagt att vi bygger komponenter istället för det som Frida designar. Utilityklasser använder vi i undantagsfall. )

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHubhttps://github.com/RegionHalland/general/issues/84#issuecomment-450312616, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ao-Aucx9tbMORqw2ircSn2-rhiChaKsiks5u9c-5gaJpZM4Y-ewp.

Så behandlar Region Halland dina personuppgifter - https://www.regionhalland.se/dataskydd

fridabergentoft commented 5 years ago

Länk till komponenter: https://www.figma.com/proto/CMYZeVSIh8UkD2VUQvu3hAHb/Komponentbibliotek-2.0?node-id=110%3A440&viewport=226%2C476%2C0.12392&scaling=min-zoom

brightjennie commented 5 years ago

Denna heter "block" efter figma-skissen. Jag har lagt komponenterna på samma sida med mellanrum för att visa med och utan bild.

viktor-sarge commented 5 years ago

Gott. Utseendet är korrekt, stämt av med Frida.

En sak jag funderar på är de fasta bredderna. Jag tänker mig att vi hellre vill fylla 100% av en container + kanske en maxbredd (=bildens maxbredd) och så låter vi containern skala istället för att blocken själva har färdiga bredder.

Vad tror du om att istället lägga maxbredd på dem, i övrigt fylla ut föräldracontainern och så lägga en demo-container som förälder?

brightjennie commented 5 years ago

Gött att utseendet stämde.

Jag har lagt en demo-div och lagt max-width där. La widht 100% på komponenten. Kika och se om jag uppfattat dig korrekt.

viktor-sarge commented 5 years ago

Grunnat lite. Jag är snarare ute efter att komponenten har en total maxbredd och att den sedan är helt flytande för att skala nedåt. Så får containern istället ta hand om eventuella brytpunkter. Så ingen small, medium, large alls i själva rh-block utan bara en maxbredd där.

Hojta om jag har otur när jag tänker här :)

viktor-sarge commented 5 years ago

Passed QA.

Eventuellt ytterligare tweakande behöver vi inte gör här och nu utan det kan vi själva ta när vi implementerar komponenten på vår webb. Tack!