Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/interactive changes #115

Merged
merged 19 commits into from
Aug 26, 2019
Merged

Feature/interactive changes #115

merged 19 commits into from
Aug 26, 2019

Conversation

DusanTuzinsky
Copy link

Pocistil som kod od veci co netreba alebo sa opakovali,
potom som vyrobil filter na nepotrebné malé komponenty ako BR keď sa vypisuje pri badgeoch a podobne.
Vytvoril som presúvanie dát cez context a vytvoril nové menšie komponenty aby bol kód prehľadnejší ktoré používajú context.
Posunul som state pre show code do interactive z preview. Aby tlačidlo show code nemizlo a bolo na jednom mieste. Jediná vec ohľadom toho je že takto teraz ak je viac komponentov vykreslených vľavo v interactive tak jedno tlačidlo zobrazuje kód pre všetky naraz. Neviem či je to ok. Dá sa to prežiť.

@adammockor
Copy link

K tej zmene show code. Vyzera to tak ze showCode momentaln zije v state v oboch komponentoch. Preco vlastne? A trochu ma zarazilo ze to dobre reaguje aj bez toho aby Interact handloval zmenu showCode v componentDidUpdate (ked sa updatne isCodeShown v Preview). Tak to mi pride take zmetocne, no mozno sa len zle pozeram.

A ked si uz vyrabal male komponenty, mohol si ich kludne poslat aj do samostatnych suborov :)

@DusanTuzinsky
Copy link
Author

Mas pravdu netrwba to mat v state Interactu asi. to mi vypadlo on to ten preview aj tak vzdy posle do interactu ked sa zmeni. To handlovanie neviem ci nie je prave preto lebo je to state? Netusim.

Mohol by som to dat do novych suborov len som nevedel kam ich strcit tie subory lebo preview ma svoj vlasny priecinok tak ci v priecinku preview vytvarat este podpriecino pre Interact a atm davat nove componenty alebo priamo tam? Trosku som bol tiez aj v casovom sklze a toto bola rychlejsia varianta v tom momente pre mna prehladnejsia. Lebo komponenty boli pekne pri sebe a este som ich za jazdy aj dost menil.

Mozem sa na to este pozriet ked bude cas. Dik za feedback.

@DusanTuzinsky
Copy link
Author

popremiestňoval som componenty do vlastných súborov vo foldri Interact, presunul som tam aj ten Interact component.

vymazal som aj handleShowCode z Interactu. Nerobilo to nič nikde. zabudol som na to.

@DusanTuzinsky
Copy link
Author

trošku som to vyčačkal nech je to krajšie uhladenejšie.

@adammockor adammockor merged commit 6f49dc5 into master Aug 26, 2019
@adammockor adammockor deleted the feature/interactive-changes branch August 26, 2019 13:00
adammockor pushed a commit that referenced this pull request Oct 2, 2019
* create one generateProps method instead of three

* interactive title as component

* fix badge display name

* keep one button to show code, simplify code

* simplify renderInput method, create input components

* move interact to new folder

* move components to corresponding files

* rename styled wrapper

* remove handleShowCode from interact

* add spacing to group form fields

* method renderInput to PropFormField compnent

* add tooltip icon styles

* use PropFormField

* new styles for component name button

* keep text children component visible all the time

* move styled input to propInput component file

* style select and checkbox fields

* fix key on map items

* add text for comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants