Opened 20 months ago

Closed 17 months ago

Last modified 17 months ago

#1103 closed дефект (fixed)

Текст не помещается в поле ввода

Reported by: alx Owned by: Denis_N
Priority: major Component: БД изделий АДС
Keywords: Cc:

Description

В r180/base в панели фильтров есть поле ввода с текстом "+ добавить фильтры по истор". Очевидно, предполагалось написать "+ добавить фильтры по истории", но такой текст не поместился в поле ввода:


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

Дополнительно отмечу, что вообще непонятно, для чего здесь применено поле ввода, если в него, насколько я понимаю, не предполагается ничего вводить - оно просто отображает статический текст...

Attachments (3)

ss1.jpg (77.9 KB ) - added by alx 20 months ago.
11.png (88.8 KB ) - added by san 20 months ago.
ss2.jpg (38.4 KB ) - added by alx 18 months ago.

Download all attachments as: .zip

Change History (17)

by alx, 20 months ago

Attachment: ss1.jpg added

by san, 20 months ago

Attachment: 11.png added

comment:1 by san, 20 months ago

В моём файрфоксе текст влезает.

comment:2 by Denis_N, 18 months ago

Resolution: fixed
Status: newclosed

In 248/base:

Исправлен дефект:

  1. Текст не помещается в поле ввода на главной. Изменил тип вместо "поля" на "кнопку"
  2. При вводе текста пользователю выпадающий список под полем ввода не убирал варианты не соответствующие тому, что вводит пользователь

closes #1103

in reply to:  2 comment:3 by alx, 18 months ago

Replying to Denis_N:

Исправлен дефект:

  1. Текст не помещается в поле ввода на главной. Изменил тип вместо "поля" на "кнопку"

Каким образом изменение типа контейнера (с текстового поля на кнопку) устраняет описанную в тикете проблему? Насколько я вижу, контейнер как был фиксированного размера, так и остался!

Да, я вижу, что эта фиксированная ширина увеличена - у input'а она была 16em, а у button - 16.1em. Ты всерьез считаешь, что вот это увеличение ширины на 0.1em (на 0.6%) решает проблему выхода текста за пределы кнопки?

Чтобы быть конструктивным предлагаю делать размер контейнера динамическим (auto) - чтобы он автоматически рассчитывался браузером так, чтобы полностью вместить свой контент.

Last edited 18 months ago by alx (previous) (diff)

comment:4 by Denis_N, 18 months ago

Я так и сделал, просто был невнимателен. Теперь у элемента описанного в тикете, класс submBtn у него есть минимальная длина 11em и длина auto. Я просто сначала создал класс addFilterBtn, а потом понял, что это не решает проблем и изменил submBtn

Исправлено в r251/base

Last edited 18 months ago by Denis_N (previous) (diff)

in reply to:  4 ; comment:5 by alx, 18 months ago

Replying to Denis_N:

Я так и сделал,

Ну, вообще-то не так: кнопка имеет свойство стиля width: 16.1em;, а не width:auto.

К счастью, высота этой кнопки установлена в auto и (в отличие от строки ввода) контент кнопки может отображаться в несколько строк, что решает описанную в тикете проблему...

in reply to:  5 ; comment:6 by Denis_N, 18 months ago

Replying to alx:

Replying to Denis_N:

Я так и сделал,

Ну, вообще-то не так: кнопка имеет свойство стиля width: 16.1em;, а не width:auto.

Нет, это так. Теперь у этой кнопки класс submBtn, и теперь ее длина width: auto;

.submBtn{ 
139	    min-width: 11em;
140	    width: auto;
141	    height: auto;
142	    padding: 0.5em;
143	    background-color: #14223eb0;
144	    color: white;
145	    border: 0px;
146	    justify-self: center;
147	    border-radius: 0.3em;
148	}


in reply to:  6 ; comment:7 by alx, 18 months ago

Replying to Denis_N:

Ну, вообще-то не так: кнопка имеет свойство стиля width: 16.1em;, а не width:auto.

Нет, это так. Теперь у этой кнопки класс submBtn, и теперь ее длина width: auto;

.submBtn{ 
139	    min-width: 11em;
140	    width: auto;
141	    height: auto;
142	    padding: 0.5em;
143	    background-color: #14223eb0;
144	    color: white;
145	    border: 0px;
146	    justify-self: center;
147	    border-radius: 0.3em;
148	}

Да, действительно, класс submBtn имеет свойство width:auto. Однако в CSS есть еще и такая запись:

main.mainPage .addFilterBtn {
  width: 16.1em;
}

Селектор этого стиля более специфичный, поэтому данный стиль имеет более высокий приоритет по сравнению со стилем класса submBtn, поэтому указанное здесь свойство width: 16.1em; перекрывает свойство width: auto;, указанное в стиле submBtn.

in reply to:  7 ; comment:8 by Denis_N, 18 months ago

Replying to alx:

Да, действительно, класс submBtn имеет свойство width:auto. Однако в CSS есть еще и такая запись:

main.mainPage .addFilterBtn {
  width: 16.1em;
}

Если мы говорим про r251/base, в которой я отписался, что исправил дефект, то в ней нет такой записи

Last edited 18 months ago by Denis_N (previous) (diff)

by alx, 18 months ago

Attachment: ss2.jpg added

in reply to:  8 ; comment:9 by alx, 18 months ago

Replying to Denis_N:

Если мы говорим про r251/base, в которой я отписался, что исправил дефект, то в ней нет такой записи

Я говорил о ревизии r275/base, которая на момент написания комментария находилась на сервере. Я просто открыл в своем браузере https://r2.adc-line.ru/base/main.php и посмотрел, какие стили браузером применены к кнопке "добавить фильтр по истории". И вижу там вот это:


comment:10 by Denis_N, 17 months ago

Resolution: fixed
Status: closedreopened

in reply to:  9 ; comment:11 by Denis_N, 17 months ago

Replying to alx:

Replying to Denis_N:

Если мы говорим про r251/base, в которой я отписался, что исправил дефект, то в ней нет такой записи

Я говорил о ревизии r275/base, которая на момент написания комментария находилась на сервере. Я просто открыл в своем браузере https://r2.adc-line.ru/base/main.php и посмотрел, какие стили браузером применены к кнопке "добавить фильтр по истории". И вижу там вот это:


И правда. Ты прав, Алексей. Не знаю как так вышло, но в r251/base я удалял этот класс. А теперь он снова есть

Last edited 17 months ago by Denis_N (previous) (diff)

in reply to:  11 comment:12 by alx, 17 months ago

Replying to Denis_N:

Не знаю как так вышло, но в r251/base я удалял этот класс. А теперь он снова есть

Восставший из ада? :)

comment:13 by Denis_N, 17 months ago

Resolution: fixed
Status: reopenedclosed

In 301/base:

Исправлено: Текст не помещается в поле ввода

Удалил стили для класса .addFilterBtn

close #1103

in reply to:  11 comment:14 by alx, 17 months ago

Replying to Denis_N:

Не знаю как так вышло, но в r251/base я удалял этот класс. А теперь он снова есть

Нетрудно выяснить, что это было опять добавлено в r252/base при устранении не работавшего поиска по комментарию. Видимо, без этого стиля поиск не работал. :)

Note: See TracTickets for help on using tickets.