telerik / kendo-react

Issue tracker - KendoReact http://www.telerik.com/kendo-react-ui/
https://kendo-react-teal.vercel.app
Other
212 stars 37 forks source link

min and max attributes doesn't work with cell #576

Closed gine closed 4 years ago

gine commented 4 years ago

I'm submitting a...

Current behavior

min and max attributes doesn't work if cell attribute are present

Expected behavior

min and max attributes work with cell attribute too

Minimal reproduction of the problem with instructions

https://stackblitz.com/edit/react-ayvuky?file=app/main.jsx

Environment

Package versions:

├── @capacitor/android@2.0.2
├── @capacitor/cli@1.5.1
├── @capacitor/core@2.0.2
├── @ionic/app-scripts@3.2.4
├── @ionic/react@5.1.0
├── @ionic/react-router@5.1.0
├── @progress/kendo-react-dateinputs@3.13.0
├── @progress/kendo-react-intl@3.13.0
├── @progress/kendo-theme-bootstrap@4.15.0
├── @types/node@12.12.38
├── @types/react@16.9.34
├── @types/react-dom@16.9.7
├── @types/react-router@5.1.7
├── @types/react-router-dom@5.1.5
├── ionicons@5.0.1
├── react@16.13.1
├── react-dom@16.13.1
├── react-redux@7.2.0
├── react-router@5.1.2
├── react-router-dom@5.1.2
├── react-scripts@3.4.0
└── typescript@3.7.4

Browser:

System:

kspeyanski commented 4 years ago

Hello @gine ,

The described behavior is due to the override of the opacity property in the demo. I've modified the provided stacklitz example logic to account for the isDisabled property accordingly: https://stackblitz.com/edit/react-ayvuky-ocsx7c?file=app/main.jsx