ExternalLink accept other props

keboola / indigo-ui

Indigo UI library, its CSS and React components. Also Styleguide app showing usages of this library.

https://indigo.keboola.com

MIT License

5 stars 0 forks source link

ExternalLink accept other props #310

Closed natocTo closed 5 years ago

natocTo commented 5 years ago

ujovlado commented 5 years ago

Asi by som to neriesil takto zhurta, to pouzitie v kbc-ui moze byt kludne zle.

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a

V skratke:

disabled na anchore nefunguje, aj tak sa da kliknut
onClick je takisto kravina, kvoli pristupnosti https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a#onclick_events

natocTo commented 5 years ago

OK. Disabled tam bylo nakombinováno s CSS tuším a pointer-events: none;. Ale pokud to tam půjde vyřešit jinak tak toto není potřeba no.

ujovlado commented 5 years ago

No asi je to tam zbastlene, co sa toho anchoru tyka.

ujovlado commented 5 years ago

Este k tomu ...restProps ... imho je lepsie povolovat konkretne props, ako vsetko. Potom prave zabranis nezmyslom ako napr. ten disabled a podobne. T.j. clovek tam skusi dat disabled, ale nic sa nestane, tak mu mozno trkne, ze je dovod, ze komponenta ma len konkretne props ;)