![[MC-04 logo]](/mc-04/chrome/site/logo.png){kind=logo}
#1100 closed улучшение (fixed)
Улучшение интерфейса "Несоответствия"
| Reported by: | alx | Owned by: | Denis_N |
|---|---|---|---|
| Priority: | major | Component: | БД изделий АДС |
| Keywords: | Cc: |
Description
В r180/base когда в интерфейсе "Несоответствия" нажимаешь кнопку "Сохранить данные", на странице пропадает история изделия, поле ввода комментария и кнопка "Сохранить данные", и вместо этого пользователю предлагается заново ввести серийный номер изделия. Это очень неудобно, когда необходимо добавить несколько записей в одно и то же изделие.
Предлагаю при нажатии кнопки "Сохранить данные" продолжать отображать серийный номер и историю ранее выбранного изделия, а также поле ввода комментария и кнопку "Сохранить данные".
Change History (3)
follow-up: 2 comment:1 by , 13 months ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:2 by , 13 months ago
Replying to Denis_N:
- при вводе символов при поиске записей, ожидается секунда, в течение которой, если символы ввода были изменены, то ajax-запрос не отправляется
- также при пользовании поиском поля ввода растягивались во время ожидания ответа от сервера
- и, согласно тикету alx, поля для записи несоответствий тперь не скрываются, а отображаются согласно тикету "#1100" closes #1100
Не уверен, что не писал этого раньше (если уже писал, прошу прощения за повтор): объединять несколько не связанных между собой изменений в один коммит - плохая идея.
Во-первых, если вдруг будет принято решение вернуть (откатить) одно из сделанных изменений (например выяснится, что оно ломает что-то важное), откат коммита приведет к тому, что вместе с "проблемным" изменением будут "за компанию" откачены другие, ни в чем не повинные изменения. Как результат, придется разруливать эту проблему вручную.
Во-вторых, такой коммит просто трудно читать, так как непонятно, какие изменения кода относятся к какому изменению функционала. Например как догадаться, в данном коммите изменение border: 1px solid black; на border: 1px solid #0000005c; решает какую из трех перечисленных проблем?
comment:3 by , 13 months ago
И еще замечание, раз уж я все равно начал занудствовать. :)
Перемешивать HTML и javascript - тоже IMHO плохая идея. Есть ведь уже специальный файл для javascript, почему бы не поместить весь код туда вместо того чтобы размещать его в <script/>? Я бы еще мог понять, если бы код генерировался динамически на сервере, но здесь, насколько я вижу, код статический...
In 192/base: