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

Component/interact #102

Merged
merged 23 commits into from
Mar 26, 2019
Merged

Component/interact #102

merged 23 commits into from
Mar 26, 2019

Conversation

dsntzn
Copy link

@dsntzn dsntzn commented Mar 20, 2019

New component for interaction with component props

packages/styleguide/src/components/Preview/CodeExample.js Outdated Show resolved Hide resolved
},
bgThemeExcludedColors: [],
hasCodePreview: true,
// to discuss if it should be turned on or off
// for debugging is better on
isInteractive: true,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ja by som povedal ze ak to nesposobuje ziadny side effect, ani to neerroruje tak by som to nechal zapnute

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no to by bolo treba otestovať na niečom. na lighteri to neerrorovalo na ničom ale napríklad tooltip dokumentácia keď som sa snažil niečo zmeniť tak padla mám pocit.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A co ty myslis, ako by to bolo lepsie?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No ak to bude na každom Preview táto interaktivita tak by sa to možno ľudia naučili že to existuje. asi je lepšie to nechať ON. uvidíme či s tým bude problém. a keď budv nejakých previews tak to môžme vypnúť len pre tie a podobne.

packages/styleguide/src/components/Preview/Preview.js Outdated Show resolved Hide resolved
packages/styleguide/src/components/Preview/Preview.js Outdated Show resolved Hide resolved
packages/styleguide/src/components/Preview/Interact.js Outdated Show resolved Hide resolved
packages/styleguide/src/components/Preview/Interact.js Outdated Show resolved Hide resolved
packages/styleguide/src/components/Preview/Interact.js Outdated Show resolved Hide resolved
packages/styleguide/src/components/Preview/Interact.js Outdated Show resolved Hide resolved
packages/styleguide/src/components/index.js Outdated Show resolved Hide resolved
packages/styleguide/src/components/Preview/index.js Outdated Show resolved Hide resolved
@adammockor
Copy link

Je to PARADNA praca 👏🏻👏🏻👏🏻.

Pozri prosim este trosku tie komentare. Jedina dolezitejsia vec je len stylisticka, tie chybajuce ciarky na poslednom elemente v poli/objekte. Mas asi inak nastaveny prettier v editore?

Ostatne nie je prilis dolezite, kludne sa to da odpinkat ked uznas za vhodne ;)

@adammockor
Copy link

Este z pohladu pouzivatela:

  1. Pride mi trosku divne ze tlacidlo show code zmizne v interactivnom mode a zobrazi sa o kusok nizsie ako ine tlacitko.
  2. Tym cislam v zatvorke (pocet propov) nerozumiem bez toho, ze som si precital kod.
  3. Pridal by som dokoumentaciu k Preview aspon na interactiveProps.
  4. Zmienil by som sa, ze to nefunguje s Preview s function as children (Preview docs > Adapt to current background color).

@adammockor
Copy link

Screenshot 2019-03-24 at 22 32 52

Taketo BR sa tam objavuju. Mozno keby sa filtrovali ciste html tagy. Je to mozne?

@adammockor
Copy link

A na predchadzajucom screenshote je vidno, ze tooltip je taky na pol vytrceny von. Neviem ci to bol zamer :) Nevyzera to tak :)

@adammockor
Copy link

Tie ostatne veci co su tu budes este riesit, ci sa to odlozi? Ak odlozi, zapisal by som si to.

@dsntzn
Copy link
Author

dsntzn commented Mar 26, 2019

tak posledné body čo treba vyriešiť sú tieto:

Pride mi trosku divne ze tlacidlo show code zmizne v interactivnom mode a zobrazi sa o kusok nizsie ako ine tlacitko.

Taketo BR sa tam objavuju. Mozno keby sa filtrovali ciste html tagy. Je to mozne?

Zmienil by som sa, ze to nefunguje s Preview s function as children (Preview docs > Adapt to current background color).

Porozmýšľať či show interactive ukazovať na všetkých Previews

zapisujem si to.

Čo teraz ďalej? Ostatné som vyriešil tak neviem ako budeme postupovať. asi by som to už skúsil nasadiť na nejaký projekt čo to spraví. mám to spraviť lokálne a potom to prepojíme nejak či rovno to vydáme?

@adammockor
Copy link

Vydame a budeme fixovat ked treba :) Ak mas cas prosim vyrob aj release. Ak nie vecer spravim.

@adammockor adammockor merged commit 9e41ac1 into master Mar 26, 2019
@adammockor adammockor deleted the component/interact branch March 26, 2019 07:31
@adammockor
Copy link

🚢

@DusanTuzinsky
Copy link

Pripravil som release

adammockor pushed a commit that referenced this pull request Oct 2, 2019
* fix proptype definition

* interactive documentation component

* add code example clean up function

* new interact component

* make Preview component interactive

* remove interact element import from docs

* return commas where automatically removed by formatter

* change toggle text

* cleanUpCode function refactor

* change tooltip to prop description

* describe props in comments

* remove prop count

* remove Interact exports

* use styleguide button

* use styleguide colors

* fix commas

* Preview component documentation for interactiveProps

* add mdx heading

* stylistic changes

* remove interactiveProps preview, no use

* change wording
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.

3 participants