Есть такой код: const DropdownItem = ({ onFilterClick = () => {}, activeFilter = '', filterType = '', hasPadding = false, }) => ( <div role="presentation" onClick={onFilterClick} className={`dropdown-item dropdown-item ${hasPadding ? 'pl-5' : ''} ${activeFilter === filterType ? 'active' : ''}`} > {filterType} {(activeFilter === filterType) && <i className="check">✓</i>} </div> ); DropdownItem.propTypes = { onFilterClick: PropTypes.func, activeFilter: PropTypes.string, filterType: PropTypes.string, hasPadding: PropTypes.bool, }; DropdownItem.defaultProps = { onFilterClick: PropTypes.func, activeFilter: '', filterType: '', hasPadding: false, }; export default DropdownItem; Я хочу отказаться от DropdownItem.defaultProps в пользу значений по умолчанию в пропсах: const DropdownItem = ({ onFilterClick = () => {}, activeFilter = '', filterType = '', hasPadding = false, }) => ( ... Но я чувствую, что неправильно прописываю значения по умолчанию. Вопрос: как делать это правильно в моём случае?

ну вроде да

Есть такой код: const DropdownItem = ({ onFilterClick = () => {}, activeFilter = '', filterType = '', hasPadding = false, }) => ( <div role="presentation" onClick={onFilterClick} className={`dropdown-item dropdown-item ${hasPadding ? 'pl-5' : ''} ${activeFilter === filterType ? 'active' : ''}`} > {filterType} {(activeFilter === filterType) && <i className="check">✓</i>} </div> ); DropdownItem.propTypes = { onFilterClick: PropTypes.func, activeFilter: PropTypes.string, filterType: PropTypes.string, hasPadding: PropTypes.bool, }; DropdownItem.defaultProps = { onFilterClick: PropTypes.func, activeFilter: '', filterType: '', hasPadding: false, }; export default DropdownItem; Я хочу отказаться от DropdownItem.defaultProps в пользу значений по умолчанию в пропсах: const DropdownItem = ({ onFilterClick = () => {}, activeFilter = '', filterType = '', hasPadding = false, }) => ( ... Но я чувствую, что неправильно прописываю значения по умолчанию. Вопрос: как делать это правильно в моём случае?

Есть такой код: const DropdownItem = ({ onFilterClick = () => {}, activeFilter = '', filterType = '', hasPadding = false, }) => ( <div role="presentation" onClick={onFilterClick} className={`dropdown-item dropdown-item ${hasPadding ? 'pl-5' : ''} ${activeFilter === filterType ? 'active' : ''}`} > {filterType} {(activeFilter === filterType) && <i className="check">✓</i>} </div> ); DropdownItem.propTypes = { onFilterClick: PropTypes.func, activeFilter: PropTypes.string, filterType: PropTypes.string, hasPadding: PropTypes.bool, }; DropdownItem.defaultProps = { onFilterClick: PropTypes.func, activeFilter: '', filterType: '', hasPadding: false, }; export default DropdownItem; Я хочу отказаться от DropdownItem.defaultProps в пользу значений по умолчанию в пропсах: const DropdownItem = ({ onFilterClick = () => {}, activeFilter = '', filterType = '', hasPadding = false, }) => ( ... Но я чувствую, что неправильно прописываю значения по умолчанию. Вопрос: как делать это правильно в моём случае?

Есть такой код: const DropdownItem = ({ onFilterClick = () => {}, activeFilter = '', filterType = '', hasPadding = false, }) => ( <div role="presentation" onClick={onFilterClick} className={`dropdown-item dropdown-item ${hasPadding ? 'pl-5' : ''} ${activeFilter === filterType ? 'active' : ''}`} > {filterType} {(activeFilter === filterType) && <i className="check">✓</i>} </div> ); DropdownItem.propTypes = { onFilterClick: PropTypes.func, activeFilter: PropTypes.string, filterType: PropTypes.string, hasPadding: PropTypes.bool, }; DropdownItem.defaultProps = { onFilterClick: PropTypes.func, activeFilter: '', filterType: '', hasPadding: false, }; export default DropdownItem; Я хочу отказаться от DropdownItem.defaultProps в пользу значений по умолчанию в пропсах: const DropdownItem = ({ onFilterClick = () => {}, activeFilter = '', filterType = '', hasPadding = false, }) => ( ... Но я чувствую, что неправильно прописываю значения по умолчанию. Вопрос: как делать это правильно в моём случае?

А кто-нибудь linaria с create-react-app и https://github.com/sharegate/craco этим ставил?)

Как значения по умолчанию в переменных это уже js

Вот что происхдит в моём случае http://prntscr.com/nzhlqg

Как значения по умолчанию в переменных это уже js

это то о чем я говорю. defaultProps срабатывают до проверки

А что вы предлагаете, чтобы решить эту проблему?

А как такое сделать именно в моём случае, где эти значения по умолчанию прописывать, в пропсах, как делаю я? Или как-то инач?

То есть вернуть обратно?

Если я верну, тогда какой смысл от этих значений по умолчанию в пропсах?

Жаль( Но я понял, спасибо!)

Выкидывайте проптайпы, вкручивайте типизацию нормальную

так это же разные вещи. тайпскрипт не все ловит, потому что в статический

Выкидывайте проптайпы, вкручивайте типизацию нормальную

И что, какая разница где упадет? В момент компиляции или в рантайме. Первое даже предпочтительнее

когда-нибудь что-то подобное всё же введут в функционал https://github.com/reactjs/rfcs/pull/107#discussion_r259277199

есть вещи которые при компиляции словить невозможно

Очень абстрактный довод

use englist

use englist

Don't use align, use css

am englistest than you so нет

my english is bad

As bad as your html?

a litter

Ребята, а вот эта штука нужна только для Функциональных компоненттов: https://ru.react.js.org/docs/react-api.html#reactmemo ? А что делать с классами?

у классов есть shouldcomponentupdate

Либо вот это: https://reactjs.org/docs/react-api.html#reactpurecomponent ?