wechta / najdisi

Najdi.si agregator novic
0 stars 0 forks source link

generalno: od narocnika: posamezne novice morajo biti klikabilni - #1 #9

Closed bostjans closed 6 years ago

bostjans commented 6 years ago

Prejeta informacija od narocnika:

Metapodatki posamezne novice morajo biti klikabilni. Premik s kurzorjem na podčrta tekstklik na »gospodarstvo« naj vrne prikaz kategorije gospodarstvo; klik na »Marketing magazin« naj vrne vse vsebine tega medija. Klik na »pravkar objavljeno« ne vrne ničesar.

Ok, tole bo potrebno dopolniti. Mislim, da velja za vse Komponente.

Torej .. za Category:

Torej .. za Vir:

Prosim za povratno inforamcijo:

markoBlagotinsek commented 6 years ago

V tem primeru pridemo nazaj do situacije, ko se ta text, ki ni vec en string, ne bo vec lepo zlagal v vrstice, sploh kadar so kategorije, ki sestavljajo breadcrumb vrstico, daljse, kot je predvideno v dizajnu.

bostjans commented 6 years ago

Ok. Zahteva je, da je link. Mimo tega ne moremo .. Da to dejansko vpliva na "zlaganje texta" .. nisem predvidel. Torej .. moj IMHO: just do it;

wechta commented 6 years ago

Če so zahteve takšne naj se potem naredi link, samo Bostjan prosim da potem obvestis naročnika glede potencialnih tezav..

vajtim commented 6 years ago

@bostjans Category_vrednost in Vir_vrednost bosta torej dodatna keya v json?

bostjans commented 6 years ago

Category_vrednost in Vir_vrednost bosta torej dodatna keya v json?

Ne. Zakaj? Saj že imaš vse podatke .. Za: "Category_vrednost" polja se vzame JSON field: "category" (primer: "category": "Slovenija") Za: "Vir_vrednost" polja se vzame JSON field: "channelUrl"

vajtim commented 6 years ago

Urejeno po navodilih.

Verjetno bi bilo pravilno le "window.location.host +" in ne "window.location.protocol + window.location.host +". Ce je tako, popravite v "src/common/sourceList.jsx".

bostjans commented 6 years ago

Tole se ni reseno.

Daj Wechta prever, ce lahko to "na brzino" popravis. (popravite v "src/common/sourceList.jsx")

Za Category link se trenutno (na Alpha okolju) nastavi na:

Moral bi se pa nastaviti na link:

Dve stvari nekako ne stimata:

Pri linku za Vir .. pa ne stima dejansko generiran link.