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

Stoppa feta rubriker? #294

Closed Svempan closed 9 years ago

Svempan commented 9 years ago

Kan vi spärra för att redaktörerna kan göra feta H1, H2 osv? Just nu slår det igenom till utsidan vad jag kan se. Gäller även malmo.se

hryd commented 9 years ago

Det går att låsa så användare inte kan använda sig av fetstil eller kursivt, men då gäller det alla typer av formatmallar, inte enbart rubriker.

olajoh1 commented 9 years ago

Henrik låser. Görs även för malmo.se

martent commented 9 years ago

@martent tar contexten .body-copy för h1, h2 och p (förutom .preamble) och lägger in i malmostad/global_assets

martent commented 9 years ago

Ser att det redan ligger overrides i Global Assets för rubriker och brödtext som har b och strong i sig:

https://github.com/malmostad/global_assets/blob/master/app/assets/stylesheets/body_copy.scss#L138...L161

Har vi något exempel där någon lyckats feta i rubrik eller brödtext respektive gjort rubrik kursiv i body copy?

Stilsättning av hx på SV-startsidor får göras separat som faksimiler. Eftersom shared_assets ligger importerad fysiskt i sitevision_assets med Bower går det att göra Sass extends på dessa:

https://github.com/malmostad/shared_assets/blob/master/stylesheets/shared/extends.scss

hryd commented 9 years ago

Startsida: http://komin.test.malmo.se/4.6fae280a14db498df1e29fad.html Normalsida: http://komin.test.malmo.se/4.6fae280a14db498df1e29fe9.html

Det slår inte riktigt igenom på .body-copy, important behövs i font-weight:normal. @martent

martent commented 9 years ago

Det är SV:s egna CSS:er som overridar ser det ut som. Lämpligast stället att overrida det, är väl i sitevision_assets. Typ:

.body-copy .sv-text-portlet {
  h1, h2, h4, p {
    strong, strong *  {
      font-weight: normal !important;
    }
  }

  h1, h2, h3 {
    em, em *  {
      font-style: normal !important;
    }
  }
}

Hm, fick jag med allt där? h3 ska vara fet, h4 kursiv och kursiv ska vara tillåtet i p.

För start-page-main får man kanske vara lite mer försiktig så att inte primary-news h1 och andra påverkas eftersom de har en helt annan stil utan endast när de lägger in rubriker och p utanför de sections som redan har en korrekt stil. Eller så funkar det, för det är ju inte strong och em som används i befintliga sections?

hryd commented 9 years ago

Sant, det blir nog säkrast att lägga det i sv-hacks. Det ska funka bra på start-page-main, de andra stylas inte på em eller strong. För att tillägga, ingress ska vara fet men inte kursiv. Jag har uppdaterat testmiljöerna, kika på nedanstående länkar. @martent @olajoh1 @Svempan

Komin Startsida: http://komin.test.malmo.se/4.6fae280a14db498df1e29fad.html Normalsida: http://komin.test.malmo.se/4.6fae280a14db498df1e29fe9.html Malmo.se Normalsida: http://www.test.malmo.se/4.4aa5d54114db4999a9f7c7e.html

martent commented 9 years ago

Helt korrekt med ingressen @hryd.

@Svempan och @olajoh1, det kan krävas en fet-reload i webbläsaren för att det ska slå igenom.

På Kominnormalsidan har ingressen classen introduction och inte preamble, var kommer den classen ifrån?

hryd commented 9 years ago

@martent introduction kommer från SiteVisions formatmallar, likadant som preamble på externwebben.

martent commented 9 years ago

@hryd Men den ska ha få samma format som preamble eller den ska tas bort?

hryd commented 9 years ago

@martent Jag är faktiskt osäker varför den heter introduction (kan vara att det är det tidigare klassnamnet), jag kan byta klassnamnet till preamble på komin.test och stämma av så det ser rätt ut där först, OK?

martent commented 9 years ago

Låter vettigt.

hryd commented 9 years ago

Verkar fungera bra, ändrar från introduction till preamble samtidigt som fixen för denna ticket går ut. Inväntar OK från @olajoh1 eller @Svempan.

Svempan commented 9 years ago

OK. Ser bra ut.

olajoh1 commented 9 years ago

Fetstil i brödtext ersätts med kursiv.

hryd commented 9 years ago

Uppdaterad på testmiljöerna. @olajoh1

olajoh1 commented 9 years ago

Sven, vill du kolla

Svempan commented 9 years ago

Ok, funkar. Hoppas dock vi kan backa bandet här i fall responsen blir alltför negativ.

hryd commented 9 years ago

Det går att ändra tillbaks CSS:en om ni vill det senare.

hryd commented 9 years ago

Deployed. @olajoh1

olajoh1 commented 9 years ago

ok