Skip to content

Conversation

@zhzz
Copy link
Member

@zhzz zhzz commented Apr 11, 2022

fixes IF-441

Фикс проблемы из чата. Фрагменты даты перестали выделяться при первом фокусе, что блокировало ввод с клавиатуры.

Сломалось в #2614 (3.8.4). Там дата стала выводиться всегда, вне зависимости от фокуса и переданного value, но визуально скрывалась через прозрачность, когда ненужна. Это было сделано для стабилизации ширины контрола. Однако, клики по полю стали попадать на фрагменты этой скрытой даты, что приводило к инверсии this.isMouseDown и попаданию null в state.selected, вместо this.iDateMediator.getLeftmostType().

private handleFocus = (e: React.FocusEvent<HTMLElement>) => {
this.setState((prevState) => ({
focused: true,
selected: this.isMouseDown && !prevState.focused ? prevState.selected : this.iDateMediator.getLeftmostType(),
}));

Полечил через использование visibility вместо opacity для скрытия даты, который не пропускает клики.

@zhzz zhzz requested a review from JackUait April 11, 2022 12:13
@JackUait
Copy link
Contributor

Добавил в твой комментарий ссылку на ишью на ютреке. С настроенной интеграцией ютрека с гитхабом ссылки будут подтягиваться в ютрек автоматически. Пока прокинул ссылку в задачу на ютреке ручками.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants