search

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

Personbild i kontakt #283

Closed olajoh1 closed 9 years ago

olajoh1 commented 9 years ago

Skapa möjlighet att publicera personbild i kontaktrutorna på komin och malmo.se.

Mårten designar lösning.

@hryd

martent commented 9 years ago

@hryd, den REST-retur du får från Kontaktboken i test har nu profilbilds-data. Wikin uppdaterad:

https://github.com/malmostad/intranet-dashboard/wiki/Contacts-API-v1#get-a-single-employee

martent commented 9 years ago

@Svempan klargör att behovet inte finns på malmo.se. Detta löser problemet med åtkomst till bilder från malmo.se vi annars haft. Profilbilder visas alltså endast i Komin-version av portletten och den är valbar för redaktören som de andra fälten.

martent commented 9 years ago

Finns nu i prototypen på: http://malmostad.github.io/prototypes/internal/clean/

Lösningen är gjord med responsiva bilder vilket innebär att en webbläsaren som stöder detta väljer bildstorlek beroende på skärmdensitet, den satta layouten som påverkar bildbredden på sidan, bandbredd, användarpreferenser eller vad webbläsaren valt att stödja. Fallback är 180px-bredd-bilden.

@hryd Du får url:erna till de olika bildformaten i API-responsen, se ovan. Markupen finns på sidan. Stilsättning ligger i Global Assets.

olajoh1 commented 9 years ago

Tveksam till storleken på bilden. Det vanliga på komin är att användaren visar namn, ev titel, tele och mailadress. Hur ser det ut med endast dessa uppgifter och hur ser det ut om man har två kontaktpersoner på en sida?

martent commented 9 years ago

Ola vill ha mindre bild.

martent commented 9 years ago

Nu är den liten i alla bredder.

Jag vet inte om du har börjat implementera @hryd men det är en liten diff: https://github.com/malmostad/prototypes/commit/37dfaf24e3ae745a1321b9253fc5affe1efc18b1

Över till @hryd för implementation.

martent commented 9 years ago

Jag ger nycklar till @hryd

martent commented 9 years ago

skickat till @hryd

hryd commented 9 years ago

Deployed i testmiljöerna. @Svempan @martent

Svempan commented 9 years ago
  1. Jag kan tyvärr inte testa i mobilen. Min VPN funkar inte där. Men borde inte allt innehåll i kontaktrutan ligga under bilden i mobilt läge så det blir uppdelat över och under?
  2. Bilden blir jättelik i Explorer 11.
hryd commented 9 years ago
  1. Lösningen följer prototypen, förutom att personnamnet ligger precis ovanför bilden och inte i samma nivå som personbilden. Jag kan förklara varför imorgon på ett möte. Här är prototypen: http://malmostad.github.io/prototypes/internal/clean/
  2. Fixat nu!
Svempan commented 9 years ago
  1. Men kolla vad som händer här i mobilt läge: http://komin.test.malmo.se/Arbetsfalt-verksamheter/Inget-arbetsfalt.html Min långa titel gör nog att det blir väldigt mycket luft till vänster om bilden.
hryd commented 9 years ago

Ja, det blir lite märkligt. Jag uppdaterade CSS:en med ett förslag på att bilden läggs under rubriken vid en viss bredd på webbläsaren, hur ser det ut? @Svempan @olajoh1

Svempan commented 9 years ago

Men det blev ju jättebra!

hryd commented 9 years ago

Kan du även testa för olika personer, med eventuellt ännu längre titlar? @Svempan

Svempan commented 9 years ago

Jag letar förtvivlat efter längre än "informationssekreterare". Återkommer.

hryd commented 9 years ago

Det uppstår egentligen bara problem om personers titlar är ett och samma ord och längre än t.ex. "Informationssekreterareloremips". Om titeln består av fler ord sker det ju en radbrytning. Det kanske är rimligt att anta att det är extremfall och att lösningen fungerar? @Svempan

Svempan commented 9 years ago

Överförmyndarhandläggare! Har lagt in det under mig på samma sida. Hon har dock ingen bild inlagd.

Svempan commented 9 years ago

I produktion i augusti.

hryd commented 9 years ago

Deployed. @Svempan