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

Bildtext under video har för stort avstånd #355

Open magnivar opened 7 years ago

magnivar commented 7 years ago

Micke tycker det är för stort avstånd mellan video och bildtext under på denna sida: http://malmo.se/Kommun--politik/Socialt-hallbart-Malmo/Kommission-for-ett-socialt-hallbart-Malmo/Uppdrag.html

Det är styrt av http://assets.malmo.se/external/v4/malmo.css .bc-video-box{position:relative;height:0;padding-bottom:56.25%;margin-bottom:1.5em}

Påverkar det andra ställen?

martent commented 7 years ago

Det är svårt att matcha upp den kod som SV lägger i kontexten här med CSS-selectors utan att det negativt påverkar andra scenarier, t.ex. video utan bildtext. Eftersom HTML-koden kring video skapas av den byggda portleten sv-webbvideo-inline ligger det nog närmast till hands att i redigeringsläget för portleten ha ett frivilligt fält för en bildtext och att generera ut en enkel och stabil HTML-kod som man med några rader i sitevision_assets kan formatera.

Genomför man detta bör avstånd och textformattering vara densamma som för bilder med bildtext, se förlaga:

http://malmostad.github.io/prototypes/external/body-copy-2/

martent commented 7 years ago

Ett alternativ som inte kräver någon ändring i portleten sv-webbvideo-inline är att skapa en dekoration för inline video med bildtext som publicisten får applicera när hen lägger in den. Då är det lätt att matcha upp CSS-selectors på korrekt sätt.

Bouvet har eventuellt någon ännu bättre idé.

magnivar commented 7 years ago

VI kanske löser detta med ett nytt Video-Element.