wechta / najdisi

Najdi.si agregator novic
0 stars 0 forks source link

nova komponenta: menu vrstica - Kategorije - #1 #11

Closed bostjans closed 6 years ago

bostjans commented 6 years ago

Prosim za oceno dela:

vajtim commented 6 years ago

Kako tocno si predstavljas: menu opcije se preberejo iz specificnega elementa?

Jaz bi najraje imel json, ki bi vseboval array objectov s key-ema "title" in "link".

Ok glede 15 ur. Ker so pa ure zelo na tesno oz. se zna zgodit, da bova potrebovala tudi kaksno vec, bi samo opozoril, da bo kakrsnakoli sprememba oz. dodatna zelja obracunana naknadno.

Urediva lahko do ponedeljka, 18.6.

bostjans commented 6 years ago

"menu opcije se preberejo iz specificnega elementa" .. pomeni, da bi strukturo in podatke za menu prebral iz enega <ul> elementa (po nekem ID-ju). To je meni najlazja opcija za narest. In ker si bolj vesc Javascript-a .. bi bil predlog, da se podpre taka izvedba. Verjetno najbolj smiselno:

Primer .. elementov, ki jih imam danes na izvorni html strani (stran, ki se najprej poservira):

<nav class="menuSampl">
    <div class="hidden" id="navCategory">
        <ul id="navCategory" class="topnav">
            <li><a shape="rect" title="Novice" href="http://novice.najdi.si">N</a></li>
            <li id="categoryTop-vroce-zgodbe"><a href="/kategorija/vroce-zgodbe" id="categoryFacetSelection-vroce-zgodbe">Vroče zgodbe</a></li>
            <li id="categoryTop-slovenija"><a href="/kategorija/slovenija" id="categoryFacetSelection-slovenija">Slovenija</a></li>
            <li id="categoryTop-regionalne-novice"><a href="/kategorija/regionalne-novice" id="categoryFacetSelection-regionalne-novice">Regionalne novice</a></li>
            <li id="categoryTop-svet"><a href="/kategorija/svet" id="categoryFacetSelection-svet">Svet</a></li>
            <li id="categoryTop-gospodarstvo"><a href="/kategorija/gospodarstvo" id="categoryFacetSelection-gospodarstvo">Gospodarstvo</a></li>
            <li id="categoryTop-sport"><a href="/kategorija/sport" id="categoryFacetSelection-sport">Šport</a></li>
            <li id="categoryTop-kronika"><a href="/kategorija/kronika" id="categoryFacetSelection-kronika">Kronika</a></li>
            <li id="categoryTop-zanimivosti"><a href="/kategorija/zanimivosti" id="categoryFacetSelection-zanimivosti">Zanimivosti</a></li>
            <li id="categoryTop-avtomobilizem"><a href="/kategorija/avtomobilizem" id="categoryFacetSelection-avtomobilizem">Avtomobilizem</a></li>
            <li id="categoryTop-znanost-in-it"><a href="/kategorija/znanost-in-it" id="categoryFacetSelection-znanost-in-it">Znanost in IT</a></li>
            <li class="has-submenu">
                <a href="#c"><strong>Več</strong></a>
                <div class="submenu3" style="display: none;">
                    <ul class="submenuroot">
                        <li id="categoryMore-kultura"><a href="/kategorija/kultura" id="categoryFacetSelection-kultura">Kultura</a></li>
                        <li id="categoryMore-lepota-in-zdravje"><a href="/kategorija/lepota-in-zdravje" id="categoryFacetSelection-lepota-in-zdravje">Lepota in zdravje</a></li>
                        <li id="weatherLink"><a href="http://vreme.alpha.najdi.si">Vreme</a></li>
                        <li id="sourceListLink"><a href="http://novice.alpha.najdi.si/seznamvirov">Seznam virov</a></li>
                    </ul>
                </div>
            </li>
        </ul>
    </div>
</nav>

.. in bi iz id = navCategory zgradil JsonData podatke za menuKomponenta().

Yes/No/Maybe?

vajtim commented 6 years ago

Torej da sparsam strukturo obstojecega menija, ga vzamem kot input za react cmp in vrnem responsive menu? :D

Pa si ti preprican, da bo ta (skrit?!) menu zgrajen, se preden se bo react menuCmp inicializirala? :D Uglavnem... to ni prava pot.

bostjans commented 6 years ago

Ok .. damo zacet z verzijo 1:

vajtim commented 6 years ago

Dodana je nova komponenta showMenu. Potreben array s trenutnim menijem strani http://novice.alpha.najdi.si/ komponenta ze vsebuje. Po potrebi se ga lahko povozi cez prvi parameter.

Za prikaz pa zadosca: `najdisiComponents.showMenu(null, document.getElementById("menu"));

bostjans commented 6 years ago

Ok. Prosim za naslednje tocke/dopolnitve:

T1. A lahko dopolnis se (osnovni) README.md s primeri uporabe .. za menu komponento:

T2. Preveri -> Popravi style.css

Problem je v novem (zadnjem) style.css file-u (torej dopolnitve za "menu vrstico") .. ce ga vkljucim -> mi podere kar nekaj obstojecih gradnikov na "news" najdiSi straneh. Zdaj .. ne vem niti kako k temu "problemu" pristopit .. -> Prosim predlagajte.

markoBlagotinsek commented 6 years ago

Kaj tocno pa podre?

bostjans commented 6 years ago

Recimo na Alpha/DEV okolju imas na desni strani eno kopico raznih gradnikov, ki si vertikalno (lepo) sledijo:

Ce pa vkljucim novo style.css datoteko dobim -> malo zmedo, oz. se doloceni box-i zacnejo prekrivati ali pa neustrezno prikazovati. Glej sliko .. -> primer box-a: NAJBOLJ BRANO screenshot_2018-07-30 najdi si

markoBlagotinsek commented 6 years ago

Mislim, da mi je jasno kje je problem. Bom preveril.

vajtim commented 6 years ago

Readme je posodobljen.