Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Не срабатывают ссылки в Содержание #277

Closed
serg33rus opened this issue Jan 15, 2021 · 30 comments
Closed

Не срабатывают ссылки в Содержание #277

serg33rus opened this issue Jan 15, 2021 · 30 comments

Comments

@serg33rus
Copy link

Заметил один мелкий, но неприятный глючок.
В содержании в поле Форматированный текст можно вставить ссылку. Но вот перейти по ней нельзя.
Чтобы срабатывал переход надо чтобы это поле было в режиме "Только чтение".
В расширении База Знаний тест открывается так же форматированным документом, но по умолчанию в режиме "Только чтение". Чтобы что-то подправить надо нажать на кнопочку "Редактирование". Лишнее движение, зато можно в текст навтыкать ссылок и по ним переходить.
Даже не знаю как лучше. Но то, что ссылки не работают - несколько огорчает.

@BlizD
Copy link
Owner

BlizD commented Jan 16, 2021

Возможно стоить добавить кнопку "Только просмотр / редактирование" с подсказкой "Устанавливает/снимает режим только просмотр. Режим только просмотр позволяет переходить по ссылкам указанным в содержании". Кнопка показывается только на закладке с форматированным содержанием.

Думаю еще нужно сохранять установленный режим для каждой задачи отдельно. То есть нужен реквизит " ТолькоПросмотрФорматированногоСодержания"(Булево)

@serg33rus
Copy link
Author

serg33rus commented Jan 16, 2021

Я бы пожалуй попробовал сделать. На каком .CF? Если будет на 1.0.5.010, нормально будет? Или лучше из dev последнюю?

@BlizD
Copy link
Owner

BlizD commented Jan 16, 2021

Нужно на дев последней #123

@serg33rus
Copy link
Author

Принято.

@milanse
Copy link

milanse commented Jan 16, 2021

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

@BlizD
Copy link
Owner

BlizD commented Jan 16, 2021

@milanse это пока не нужно делать. Лучше выделить в отдельную задачу и там обсудим

@serg33rus
Copy link
Author

serg33rus commented Jan 16, 2021

Воткнул кнопочку "Редактирование". Реквизит делать не стал. Ну не нужен он. Содержание, как правило, заполняется при создании задачи. Бывает, что его нужно подредактировать, но это не значит, что и потом открывая эту задачу ты будешь ее редактировать. в общем сделано примерно как в Вопросах.
И еще. Добавил три строчки по #236. Теперь при создании Вопроса поле Задача заполняется автоматом.
Проверил, вроде работает все.
https://drive.google.com/file/d/19WyAspf2avAqerqvLwLoUwl7uByeAwQG/view?usp=sharing

@BlizD
Copy link
Owner

BlizD commented Jan 16, 2021

Сейчас при открытии задачи можно сразу изменять содержание задачи? или необходимо нажимать на кнопку?

Если надо нажимать на кнопку, то нужно изменить поведение, что при открытии задачи редактирование сразу доступно.

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

@serg33rus
Copy link
Author

При создании Задачи редактирование включено.
При просмотре уже созданной, выключено и включается по кнопке.
Я понимаю о чем ты. Но возлагать на пользователя переключение режима отображения ... как-то не очень.
Не было ссылок и пользователь поставил "Редактировать". Потом воткнул и ему надо поставить "Просмотр"? А если потом ее удалил? В обратку? Как-то неаккуратно. Реквизит сделать нетрудно. Вот только он сделает поведение не совсем понятным. Сейчас сделано тупо, но всегда одинаково. Созданная задача открывается в режиме просмотра. Надо исправить Содержимое, нажми на кнопочку. У меня в задачах и Базе знаний ссылки часто используются. И на http и на объекты конфигурации. Очень удобно.

@BlizD
Copy link
Owner

BlizD commented Jan 16, 2021

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

@serg33rus
Copy link
Author

Хммм. Ну так-то да. Но сейчас у ВСЕХ ссылки не срабатывают, а нынче будут. В общем-то тоже измениться и поведение и функционал. Просто понять бы насколько часто требуется исправлять Содержимое задачи. Если нечасто, то одна песня, Если постоянно, то другая. Я Содержимое не так часто правлю. У кого-то может быть и иначе.
Скажи как сделать. Воткнуть реквизит? И чему он должен быть равен по умолчанию? Если Истина, то для пользователя ничего не измениться, но и ссылки работать не будут, пока не нажмет на кнопочку. И ведь не нажмут и так и будут без ссылок жить.

@serg33rus
Copy link
Author

Может в телеге в группе мини-опрос провести? И уж тогда танцевать?

@79vlad
Copy link

79vlad commented Jan 16, 2021

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

@serg33rus
Copy link
Author

serg33rus commented Jan 16, 2021

Я покурил и сформулировал вариаты.

  1. При создании режим "Редактирование". При открытии существующей режим "Просмотр", ссылки работают, для редактирования нажать кнопочку "Редактировать". Визуально видно по серой или активной панели Редактирование.
  2. По умолчанию включен режим "Редактирование". Для того чтобы перейти по ссылке надо по кнопочке перейти в режим "просмотр". Визуальные эффекты как в п.1.
  3. Сделать глобальную константу режима просмотра. Каждый сам себе ее устанавливает. Для всех задач.
  4. При записи задачи запоминать последний режим. Отредактировал, установил режим, записал. Универсально, но не очень прозрачно. Ну и как правило, либо нормой является редактирование Содержимого, либо оно формируется при создании, а все остальное либо подзадачами, либо Комментариями, либо Вопросами. Потому как редактирование Содержимого в оповещения не падают.

@serg33rus
Copy link
Author

Хммм. Смотрю закрыто уже. А я надрываюсь :-). А какое хоть решение принято?

@79vlad
Copy link

79vlad commented Jan 16, 2021

Я покурил и сформулировал вариаты.

  1. При создании режим "Редактирование". При открытии существующей режим "Просмотр", ссылки работают, для редактирования нажать кнопочку "Редактировать". Визуально видно по серой или активной панели Редактирование.
  2. По умолчанию включен режим "Редактирование". Для того чтобы перейти по ссылке надо по кнопочке перейти в режим "просмотр". Визуальные эффекты как в п.1.
  3. Сделать глобальную константу режима просмотра. Каждый сам себе ее устанавливает. Для всех задач.
  4. При записи задачи запоминать последний режим. Отредактировал, установил режим, записал. Универсально, но не очень прозрачно. Ну и как правило, либо нормой является редактирование Содержимого, либо оно формируется при создании, а все остальное либо подзадачами, либо Комментариями, либо Вопросами. Потому как редактирование Содержимого в оповещения не падают.

Кстати, переход в режим редактирования - это доп триггер, на который можно навесить уведомления "Изменилось описание задачи"

PS видать случайно @serg33rus закрыл текущую ветку , надо бы повторно открыть

@serg33rus
Copy link
Author

serg33rus commented Jan 16, 2021

Кстати, переход в режим редактирования - это доп триггер, на который можно навесить уведомления "Изменилось описание задачи"

Ну сейчас изменение Содержания в уведомления не падает. По кнопке "Редактировать" нельзя делать уведомления. Нажал, ничего не изменил, обратно отжал. И о чем тут уведомлять?

PS видать случайно @serg33rus закрыл текущую ветку , надо бы повторно открыть

Это не я. Это Антон вероятно принял какое-то решение и закрыл.

@79vlad
Copy link

79vlad commented Jan 16, 2021

Ну сейчас изменение Содержания в уведомления не падает. По кнопке "Редактировать" нельзя делать уведомления. Нажал, ничего не изменил, обратно отжал. И о чем тут уведомлять?

Да, сейчас такого уведомления нет,
Но имея такую команду, можем придумать что угодно, например, можем используя измерение режима (например, добавить реквизит формы "БылоРедактированиеСодержания") перед записью задачи, спросить о необходимости отправить уведомление.
Главное, что есть событие изменения режима и можем по нему ориентироваться и если надо - то или спрашивать (чтобы не сравнивать было/стало) о необходимости уведомления (это лучше) или сразу отправлять уведомления (это хуже).

PS у меня было пару раз, когда работаешь над задачей - открываешь задачу "освежить память" - а там уже куча правок в описании

@BlizD
Copy link
Owner

BlizD commented Jan 16, 2021

Я не закрывал. Это ты serg33rus:smile:

image

@BlizD BlizD reopened this Jan 16, 2021
@BlizD
Copy link
Owner

BlizD commented Jan 16, 2021

@serg33rus

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

Мне нравится вот этот вариант, только константу создать в справочнике узКонстанты.
Константу предлагаю назвать "УстанавливатьТолькоПросмотрДляСодержанияЗадач" (Булево) синоним "Устанавливать только просмотр для содержания у существующих задач".
Если указано УстанавливатьТолькоПросмотрДляСодержданияЗадач = ДА, тогда для существующих задач ставить только просмотр для содержания.

Обеспечить автозаполенение при смене релиза, по умолчанию выставить УстанавливатьТолькоПросмотрДляСодержанияЗадач = ДА

@BlizD BlizD added this to the 1.0.5.ххх milestone Jan 16, 2021
@BlizD
Copy link
Owner

BlizD commented Jan 16, 2021

Ну сейчас изменение Содержания в уведомления не падает. По кнопке "Редактировать" нельзя делать уведомления. Нажал, ничего не изменил, обратно отжал. И о чем тут уведомлять?

Согласен, считаю, что уведомление не стоит делать по нажатию на кнопку "Редактировать".

@serg33rus
Copy link
Author

Обеспечить автозаполенение при смене релиза, по умолчанию выставить УстанавливатьТолькоПросмотрДляСодержанияЗадач = ДА

Следующий релиз считать 1.0.5.011?
Я могу сделать тупо. При первом входе в Задачу проверять и устанавливать если еще не установлено.
Тогда релиз будет пофиг.
Как скажешь, так и сделаю.

@serg33rus
Copy link
Author

Сделал при обновлении релиза заполнение константы. Воткнул в обновление до 11.0.5.11.
https://drive.google.com/file/d/1ui_c02BUkw0NaYo4qONH1ca9skfCUmc3/view?usp=sharing

@BlizD
Copy link
Owner

BlizD commented Jan 17, 2021

Спасибо.

BlizD added a commit that referenced this issue Jan 17, 2021
…я задачи

* Новая константа "Устанавливать только просмотр для содержания у существующих задач" = Да/Нет
@BlizD
Copy link
Owner

BlizD commented Jan 17, 2021

@serg33rus залил изменения, немного скорректировал код.
Управление формой (видимость, доступность) необходимо выполнять в процедуре "УстановитьВидимостьДоступность".

@serg33rus
Copy link
Author

Супер. Сейчас себе накачу. А то прямо неудобно работать.
Спасибо.

@serg33rus
Copy link
Author

Как мне кажется строка 465 в модуле формы элемента Задачи лишняя.
Поскольку ниже идет код который устанавливает значение Элементы.Содержание.ТолькоПросмотр
Хотя на ход поршней не влияет :-)

@BlizD
Copy link
Owner

BlizD commented Jan 23, 2021

@serg33rus

Как мне кажется строка 465 в модуле формы элемента Задачи лишняя.
Поскольку ниже идет код который устанавливает значение Элементы.Содержание.ТолькоПросмотр
Хотя на ход поршней не влияет :-)

Привет. Это просто такой подход, что сначала все запрещаем, а потом ниже что то разрешаем или показываем.

@serg33rus
Copy link
Author

Привет. Это просто такой подход, что сначала все запрещаем, а потом ниже что то разрешаем или показываем.

Понял.
Тогда вроде все, закрываю?

@BlizD
Copy link
Owner

BlizD commented Jan 23, 2021

Да можешь закрыть

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

No branches or pull requests

4 participants