phoenixbsl icon indicating copy to clipboard operation
phoenixbsl copied to clipboard

Добавить ссылки на диагностики BSL Language Server

Open alei1180 opened this issue 2 years ago • 8 comments

Хотелось бы видеть в виде отдельной колонки напротив текста ошибок или предупреждений.

alei1180 avatar Jun 19 '23 18:06 alei1180

@alei1180, я когда то начинал это делать, но дело остановилось на выборе какая ссылка будет - dev или release в BSL LS

otymko avatar Jun 20 '23 02:06 otymko

@alei1180, я когда то начинал это делать, но дело остановилось на выборе какая ссылка будет - dev или release в BSL LS

@otymko если пользователя ничего не ограничивает в выборе релиза BSL я думаю есть смысл смотреть в сторону DEV.

alei1180 avatar Jun 20 '23 04:06 alei1180

А зачем решать куда смотреть? Бсл лс сам вернёт ссылку на документацию в href

nixel2007 avatar Oct 24 '24 12:10 nixel2007

Я добавил фичу в форке. Обкатаем на работе, если все ОК, сделаю МР. Image

sv-sand avatar Sep 21 '25 18:09 sv-sand

Я добавил фичу в форке. Обкатаем на работе, если все ОК, сделаю МР. Image

@sv-sand Супер!

Я как пользователь видел несколько другую визуальную реализацию:

  • Сделать отдельную колонку Диагностика и в нее помещать название ключей диагностик
Image
  • Переход по клику в ячейке диагностики осуществляет переход в браузер с конкретной диагностикой

  • По правой кнопке вызывает меню Скопировать ссылку на страницу диагностики

alei1180 avatar Sep 21 '25 20:09 alei1180

Дело в том, что переход по ссылке на страничку с описанием issue более информативен. На страничке есть инфо про имя диагностики, описание, пример экранирования и прочая информация. В первом варианте я добавлял колонку с диагностикой, потом добавил колонку со ссылкой на страничку диагностики (коллеги просили добавить), из-за них сжимаются остальные колонки. Надо расширять форму, чтобы было красиво, форма становится громоздкой и сложной для восприятия. Поэтому пришла идея сделать контекстное меню для дополнительных фичей. А потом стало понятно, что имя диагностики можно посмотреть на страничке и нет смысла дублировать его на форме.

sv-sand avatar Sep 22 '25 03:09 sv-sand

Думаю надо переназвать не "Открыть стандарт", а "Открыть диагностику"

sv-sand avatar Sep 22 '25 03:09 sv-sand

Открыть документацию?

nixel2007 avatar Sep 22 '25 07:09 nixel2007