Feature/search

[dsntzn]

Nový search v lighter-styleguide

• search funguje tak, že hľadá v path a v title linku v navigácii, každé hľadanie má svoje vlastné skóre a podľa toho ktoré skóre je vyššie to sa uprednostní a podľa toho skóre sa sortuje výsledná množina vyhľadaných liniek, v skratke najvyššie skóre či už v path alebo title je vždy navrchu

• Prepísal som Navigation component do Hookov a zmenil som štýl akým sa pamätá či je kategória aktívna,
  Silno to súvisí s location.pathname ale ovplyvňuje to aj klikanie na kategórie. Pomocou hookov sa mi to podarilo celkom zjednodušiť.
  Existuje len jeden objekt aktívneho stavu kategórie s key: value v tvare url-kategorie: true/false
  Keď sa mení pathname tak sa preparsuje a vložia sa tam všetky kategórie ktoré sú v pathname do objektu ako aktívne kategórie
  Klikanie na kategóriu iba zmení príznak z true na false a opačne, keďže klik na kategóriu nemení pathname tak má vlastnú funkciu na zmenu príznaku

• TODO: Juri tam ešte chcel shortcut na focus do searchu, to tam ešte nie je, našiel som aj pekný hook na používanie shortcutov ale poslať referenciu na ten input ktorý vygeneruje autocomplete je trošku tricky a už nemám čas, predstavoval by som si to na cmd + P ako v VS code

• TODO: Juri tam chcel ešte aj query string ako parameter v url na to som úplne zabudol.

• TODO: možno by bolo fajn aj nejak naštýlovať ten search

• TODO: rozmýšľal som aj že by bolo fajn mať ten search umiestnený niekde inde nie v navigácii lebo keď sa navigácia scrolluje tak search sa stratí z dohľadu. Nech je niekde fixne by som chcel a veľký. Ale na tom sa môžme tiež dohodnúť radšej. viem že sa robil ten sidebar teraz tak do neho to nejak pekne napasovať potom možno.

• BUG: všimol som si ešte aj že ak niečo vyhľadám a potom stlačím esc a potom skúšam písať ďalej už sa mi nevyznačuje prvá možnosť čo mi troška vadí. Neviem tiež čím to je.

[adammockor]

Tento else je tu zbytocny :)

[adammockor]

Velmi pekne citatelne nakodene :) Aj sumar spisany!

Mergol som sem master s tym novym sidebarom. Vidim tam jeden problem, ze ten list sa teraz z nejakeho dovodu zobrazuje cez input. Vies to prosim este kuknut?

Inak je to podla mna good to go a todos mozu pockat podla mna. Idealne ich prosim zapis do Asany :)

[dsntzn]

Napravil som poziciu toho listu a odstranil else. Todo spisem do Asany asap.

…

On Nov 25, 2019, at 16:28, Adam Močkoř ***@***.***> wrote:  Velmi pekne citatelne nakodene :) Aj sumar spisany! Mergol som sem master s tym novym sidebarom. Vidim tam jeden problem, ze ten list sa teraz z nejakeho dovodu zobrazuje cez input. Vies to prosim este kuknut? Inak je to podla mna good to go a todos mozu pockat podla mna. Idealne ich prosim zapis do Asany :) — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[dsntzn]

chcel som dať do hlavičky ešte jeden search nech nemusí človek otvárať menu kvôli hľadaniu ale nefungovalo to dobre na mobile. nazoomovalo to na input a to vyrolované menu bolo nepoužiteľné, tak som to rovno revertol

[adammockor]

Dakujem za super pracu!