poff-bnff / web2021

1 stars 0 forks source link

optimeeritud pildiversioonide kasutamine PUGides #466

Open jaanleppik opened 2 years ago

jaanleppik commented 2 years ago

Seotud issue: https://github.com/poff-bnff/web2021/issues/308

PUGid, mis kasutavad pilte:

Alustaks Artiklitega? Uudise leht ja uudiste listi leht?

mpennaste commented 2 years ago

@jaanleppik @LiisKasper Palun värskendage ja seletage lahti, et kus mida kasutama peaks. St kas _small on nüüdsest siis thumbnaili asemel (kui jah, siis oleks üldse tegelikult mõistlik jättagi ta thumbnail nimega, töötaks ka tagasiulatuvalt) ja kas siis artikli lahti klikates taustaks originaal või kuidas see mõeldud on, et kuhu need big ja medium lähevad jne?

jaanleppik commented 2 years ago

Small on jah ses mõttes praegu väikseim versioon. Aga ma ei julgeks esailgi üht veel teisega asendada, kuna ma ei saa päris aru, mis tingimustel need thumbnail versioonid tehakse jne. E teeks ikka kaks rada: seniste pildifailide kasutamise rada + uute versioonide kasutamise rada.

Artikli juures: Originaali ei kasutagi. Artikli avamisel _big versioon Artiklite listis e uudiste listi lehel _small versioon

PS! Katmata on versioon, et toimetaja tahab kasutada F_ prefiksiga pilti artikli juures...

mpennaste commented 2 years ago

Kuna vanadel piltidel neid versioone pole, siis kasutab nende puhul originaali ehk nii nagu praegu?

jaanleppik commented 2 years ago

jah, kui pole loodud antud PUGi jaoks sobivaid uusi versioone pildist, siis kasutab originaali edasi. Sest ega me senistest piltidest praegu polegi teinud kõiki uusi versioone ehk ei teegi.

mpennaste commented 2 years ago

Artiklitega peaks olema nüüd okei. Soovi korral saab localis testida harus #466---Pictures

Ehk artiklivaated kui ka listivaated (olemasolul) järgnevatel artiklitüüpidel:

jaanleppik commented 2 years ago
jaanleppik commented 2 years ago

e nii uudiste listi vaates kui artikli vaates võiks esimene valik olla _big versioon (keelepõhine ja siis teisena default), siis järgmiseks _big_16_9 versioon (keelepõhine ja siis teisena default)

mpennaste commented 2 years ago
  • uudise juurde peaks saama ka F_ prefiksiga piltide _16_9 versioone kasutada. Esimene valik on tavaline, _small listis ja _big artikli vaates, teine valik _small_16_9 listis ja _big_16_9 artikli vaates

See oleks hea kui oleks lähteülesandes, sest praegu pean absoluutselt kõik vähemalt 17 faili uuesti üles otsima ning _16_9 versioonid valikuks lisama. St lähteülesanne võiks sisaldada algselt kogu olulist infot.

Tegelen siis ka hero ja viiekaga, see tõesti ununes enne. Panen samas stiilis ja järjekorras nagu kirjeldasid + siis tagasiulatuv neile mil neid pildiversioone pole:

  1. image_keel _small versioon
  2. image_default _small versioon
  3. image_keel _small_16_9 versioon
  4. image_default _small_16_9 versioon
  5. image_keel originaal
  6. image_default originaal

@jaanleppik On see järjestus korrektne? :)

mpennaste commented 2 years ago

Palun täpsustada üle ka filmide, persoonide ja tiimide info selles osas, et mis versioonid ning kus esimesena, teisena, kolmandana jne enne kui ma seda edasi toimetan.

jaanleppik commented 2 years ago

Film / Cassette

LISTI VAADE

  1. _small_16_9
  2. _small

FILMI VAADE, STILLS

  1. _big_16_9
  2. _big

Filmi vaade / Director

_med_sq _small

jaanleppik commented 2 years ago

Article

LISTI VAADE

  1. _small / keel
  2. _small / default
  3. _small_16_9 / keel
  4. _small_16_9 / default
  5. _small_p / keel
  6. _small_p / default

ARTIKLI VAADE

  1. _big / keel
  2. big/ default
  3. _big_16_9 / keel
  4. _big_16_9 / default
  5. _big_p / keel
  6. _big_p / default
jaanleppik commented 2 years ago

Esileht / HERO artikkel

ESILEHE VAADE

  1. _big / keel
  2. _big / default
  3. _big_16_9 / keel
  4. _big_16_9 / default
  5. _big_p / keel
  6. _big_p / default

Esileht / Viiekas

ESILEHE VAADE

  1. _med / keel
  2. _med / default
  3. _med_16_9 / keel
  4. _med_16_9 / default
  5. _big_p / keel
  6. _big_p / default
jaanleppik commented 2 years ago

Team / festivalTeam

  1. _small_sq
  2. _small

Team / jury / guest

  1. _med_sq
  2. _small
mpennaste commented 2 years ago

Tänud! Tegin siis sellise tööriista: image_prioritizer.js Peaks lihtsustama PUG süsteemi ning parem enne ehitust pildivalik ja kõik muu loogika võimalusel ära teha, et build peaks vähemaga tegelema. PS! Üleliigne media mida ei kasutata kustutatakse yamlist, on ka buildil vähem kraami millega tegeleda.

Seda kasutavad hetkel artiklite fetcherid mis saavad järjekorra sellele tööriistale edastamiseks domain_specificust (domain_specifics.yaml). Seal on määratud formaadipõhised eelistused mille sisu kasutatakse eelisjärjekorras vastavalt ülalt alla:

Kui ühtegi pilti formaatidega pole kasutatakse järgmiste alt esimest originaali mis olemas:

Kui pildi formaat või selle puudumisel originaalpilt leitakse, on see saadaval objekti all alati primaryImage nimega (nt self.primaryImage või val.primaryImage). Kõik eilsed 17 PUG faili panin seda kasutama, ning kui seda väärtust ei ole, asendab PUG pildi placeholderiga.

Hero jms hetkel ei jõudnud, need jms hiljem.

mpennaste commented 2 years ago

Hero, Viiekas, Team, Jury, Guest peaks nüüd olema ka olemas. @jaanleppik , ( @LiisKasper )

  1. Tiim ja Jury

Tekitasin hetkel fetcheris nendest kõigist piltidest ajutise media objekti millest siis eelnevas kommentaaris mainitud script õige kraami välja toimetab. Hetkel panin viisil, et persooni all ei ole formaate, aga kui on, saab parandada. Järjekord nagu kõigil kirjeldatud domain_specificus.

  1. Trio pildid kas ja mis järjekorra järgi?
  2. Industry persons pildid kas ja mis järjekorra järgi?
  3. Ajalugu ( #454 )
jaanleppik commented 2 years ago

Tiim ja Jury: Personi all korratavusele ei ole osanud rakendust mõelda, sellepärast pole see korratav. @Team on olnud mingi fantaasia, et võiks olla mitu pilti. Aga ega me picture@Team pilte kusagil mitut ei kasuta, nii et põhim vib selle kah yhekordseks teha. Praegu jah see picture@Team ei lange formaatide tegemise alla kudagi. Sinna peaks üles laadima R_ prefiksiga pildi, et saada ruudukujulised versioonid, nagu näopildilt ootame.

Trio

  1. _small_16_9
  2. _small
  3. _big_p

Industry Persons pildid

  1. _big
  2. _big_16_9
mpennaste commented 2 years ago

Hetkel tirisin arhiivi haru Pictures harule peale, et ka arhiivi harus teha persoonidele jne pildiasjad korda ja panin toimima. Triod tegin ka ära.

Tekkis küsimus pildiformaatide kohta järgmistel:

mpennaste commented 2 years ago

Tirisin ka FE-History-Publisher harusse Pictures haru. Ja ka dev serverisse. Saab seal vajadusel koos arhiiviga testida ja buildida kõike ja vaadata.

mpennaste commented 2 years ago

Koos arhiiviga nüüd live, sulgen.

jaanleppik commented 2 years ago

Arhiivis olevatel filmidel ei leite pilte: filmide listi vaates on pildid olemas, filmide fetailises vaates mitte: näide: https://poff.ee/film/ulbolson-ueksi-koigi-vastu/

mpennaste commented 2 years ago

See peaks OK olema nüüd buildis valmis (https://build.poff.ee/film/ulbolson-ueksi-koigi-vastu/)

jaanleppik commented 2 years ago

eri pildiversioonide loomise ja kasutamisega on katki läinud FB jaoks jagamisel pildi kaasa panemise funktsionaalsus. Siin peame kah siis valima, milline versioon artikli või filmi jagamisel FB'sse kaasa panna. Kus koodis seda tehakse? Kas Jaan saab juhendamisel seda modida?

mpennaste commented 2 years ago

Tõsi, selle feature'i olemasolu läks üldse meelest tol ajal kui neid pilte sai muditud.

Otsisin üles, see on failis ssg\source\_templates\head_common.pug ridadel 11-22. Seda siis includetakse iga saidi head templatesse. Neil ridadel kontrollitakse sõltumata sellest kas tegu artikli või filmiga või millegi muuga, et millist pilti FB või Twitteri tarbeks meta tagi'desse panna. Kuna nüüd sai need nimetused muudetud, siis peab üksikvaate YAMLidest uurima ja need asjad matchima panema.

@jaanleppik anna teada kas soovid ise pusida või proovin ise joonde ajada selle, ma kindel ei ole, aga mul võib olla läheks kiiremini.

jaanleppik commented 2 years ago

Palun paneme siis artikli / filmi jagamisel Facebookis sellises järjekorras pildiversioonide eelistused: _small_16_9 _small

mpennaste commented 2 years ago

Kas siin on vajalik üldse formaate sel puhul just eraldi arvestada, ma isiklikult paneks selle sama põhilise pildi mis oleme juba üksikule artiklile või filmile esimeseks kuvapildiks valinud. FB ja Instagram ilmselt niikuinii ise toimetavad neid ka. Vähemalt FB tundub laseb mingist süsteemist veidi läbi: image

Muidu peaks kõik jälle läbi käima ja mingit eriloogikat looma hakkama.

jaanleppik commented 2 years ago

Nõus, kui FB tõesti need pildid ise töötleb automaatselt.

mpennaste commented 2 years ago

Kassett, film, artikkel, industry person, programm, industry project/event - need peaks nüüd olema OK pärast lehtede full buildi. Lisaks ka filmi directori pildiformaat.

jaanleppik commented 2 years ago

paistab, et puudu on veel versioonide eelistamise loogika frontpagecourse moodilite jaoks (domain_specifics Priority osa frontpagecourse jaoks). E poster versioonid _big_p ja _small_p luuakse media librarysse, aga PUGi jaoks ei ole neid kirjeldatud ja builditud?

jaanleppik commented 2 years ago

frontpagecourse_filmikool.pug ja frontpagecourse_discamp.pug _big_p _small_p

jaanleppik commented 2 years ago

tiimide lehel avastasin, et kui laen üles R_ prefiksiga pildi, ja lisan selle persooni juurde, siis tiimi lehel ei kasutata pildi ruudukujulist versiooni, vaid pildi algsete proportsioonidega versiooni.

https://assets.poff.ee/img/R_simona_stenberg_q_555805b8e0.jpeg https://build.poff.ee/poffi-meeskond/

ja ma ei saa aru, kuhu ma domain_specificus pean juurde kirjutama neid versioone...

jaanleppik commented 2 years ago

siin näide: Ester Kuntu https://build.oyafond.ee/laureaadid/

mpennaste commented 2 years ago

Siia ka, see siis peaks järgnev olema mis soovime:

juryAndGuestImagePriority:
    - - _med_sq
      - pictureAtJury
    - - _small_sq
      - pictureAtJury
    - - _small
      - pictureAtJury
    - - _med_sq
      - personPicture
    - - _small_sq
      - personPicture
    - - _small
      - personPicture

personPicture siis Strapis subteam memberi all olematu asi, st koodis välja mõeldud muutuja, et saaks sealt persooni alt pildi võtta mida kasutada kui pictureAtJury/pictureAtTeam pilt puudu. Erinevalt siis pictureAtJury'st mis reaalne Strapis olev meediaväli.