|
zed
10-01-2020 12:54
руководитель
~0019602
Последние изменения: 10-01-2020 12:54
|
Я так понимаю, тега cut в html стандарте нету и это просто некий произвольный маркер? Тогда, по-моему лучше использовать такой маркер, который не сломает html, а именно - маркер-комментарий. Например: <!-- sas.cut -->
И наверное, не плохо бы добавить на форму с редактором описания метки кнопку, которая будет вставлять этот маркер.
|
|
|
|
>Например: <!-- sas.cut -->
> И наверное, не плохо бы добавить на форму с редактором описания метки кнопку, которая будет вставлять этот маркер.
Присоединяюсь. А в целом очень полезное и логичное будет изменение. |
|
|
dimasic
10-01-2020 13:25
разработчик
~0019604
Последние изменения: 10-01-2020 13:27
|
Не описанные в стандарте HTML теги ничего не ломают, просто игнорируются при отображении документа. Но да, предложенный тег <!-- sas.cut --> и вставляющая его кнопка - гораздо более изящное решение. Сделаем.
> А в целом очень полезное и логичное будет изменение.
Не то слово. Я всегда оформлял краткий комментарий под тегом <body>, затем добавлял признак наличия расширенного описания и после него оставшуюся часть в тегах p.../p - оно работает, но этот костыль неочевиден, неудобен и быстро забывается, поэтому периодически приходится подглядывать данную методику в описаниях существующих меток, что напрягает.
|
|
|
|
По данному тикету создан пулл-реквест. Проверьте, пожалуйста, все ли правильно.
Итого:
Во всплывающем окне отображается текст до тега <!-- sas.cut -->, остальная часть текста видна только при клике на метку (или группу меток, попадающую в зону клика).
В редакторе метки добавлена кнопка "ножницы", которая вставляет тег <!-- sas.cut --> |
|
|
|
Иконка "ножницы" у нас уже есть в ресурсах, правда в листе на главной форме (приложил файл cut.png). Может, для единообразия, её же использовать и тут? |
|
|
|
Давайте, пусть все будет однообразно.
Итак, мои действия:
1. Отдельный пулл-запрос к данному тикету: один коммит по отображению краткой версии описания и один коммит по добавлению кнопки с новой иконкой на панель форматирования.
2. Отдельный тикет и отдельный пулл-запрос насчет корявого отображения текста во всплывающей подсказке, о котором мы с вами говорили в комментариях к существующему запросу.
3. Старый пулл-реквест отклоняю (если я правильно понял принцип работы)?
Так? |
|
|
|
Всё так.
И ещё, у нас принято в названии коммита указывать номер тикета и через двоеточие краткое пояснение. Посмотрите в истории коммитов примеры. Желательно придерживаться такого шаблона.
Т.е. для этого тикета сообщение может быть примерно таким:
"3607: Добавлена возможность ...." |
|
|
|
dimasic
Если есть желание, можете дополнить wiki описанием новой функции: Описание меток. |
|