Bug: В окне "Copy task JSON model" переключатель "Show JSON" имеет атрибут "disabled" about codebattle HOT 10 CLOSED

@ReDBrother, спасибо за пояснения! Понимаю, что данная панель реализована как типовой компонент, который используется в разных частях системы.

Мне пришла в голову идея, которую я хотел бы предложить. Если компонент реализован с использованием React, возможно, стоит рассмотреть возможность его кастомизации через пропсы. Это позволило бы передавать аргумент boolean, который бы управлял отображением слайдера в зависимости от контекста. Таким образом, если в конкретной части интерфейса слайдер не требуется, его можно было бы скрыть, что помогло бы разгрузить интерфейс и сделать его более интуитивно понятным для пользователя.

Понимаю, что разработчики могут иметь свои соображения на этот счет, но подумал, что моя идея могла бы оказаться полезной. Спасибо за внимание!

from codebattle.

Да, это хорошая идея и реализовано в этом компоненте. Только не булеан, а нормальный стейт, потому что режимов может быть больше двух)

from codebattle.

В UI/UX одного компонента стараются держать интерфейс единым. Если часть функционала по тем или иным причинам зафиксирована в одной точке, то это должно быть явно показано и даже через disabled контролы

Элементы интерфейса стараются не скрывать без необходимости (Например если мало места для доступного управления). Если компонент еще и может переключаться между режимами, то из-за появления новых элементов остальной контент может скакать, что вызывает гораздо больше раздражения, чем disabled переключатель.

from codebattle.

Это не имеет смысла
Мы всегда копируем текст, а не визуальное представление

from codebattle.

@ReDBrother спасибо за ответ!

Тогда возникает вопрос зачем там нужен этот элемент интерфейса ?

from codebattle.

Он присутствует и в других частях системы, которые показывают в разных форматах текущую таску

from codebattle.

Это общая панель, которая в разных режимах работает и в разном контексте

from codebattle.

@ReDBrother, ооо прикольно! Спасибо за пояснения.

У меня возник вопрос: почему в данном контексте вы решили задизэйблить этот элемент, а не убрать его вовсе?

from codebattle.

По факту, можно так и так. Но это точно не баг - ну никак) Этот контрол ничего не ломает)

from codebattle.

@ReDBrother, спасибо большое за такое подробное разъяснение!
Это действительно помогает лучше разобраться в том, как и почему могут быть устроены интерфейсы.

from codebattle.