1. Oleg Sychev Avatar Oleg Sychev
  2. POAS Moodle plugins
  3. Moodle Plugins
  4. Issues

Проблемы в описании и графе к релизу

Issue #325 closed
Oleg Sychev repo owner created an issue

Originally reported on Google Code with ID 325

1)Регекс 
then then

Посмотрите на описание при выделении как показано скобками (это не скобки подмаски,
это выделение в этом отрезке!)
t(hen th)en

У меня перед началом выделения почему-то разорвалось слово и вставилась лишняя "then"
хотя в конце выделения (в другом слове) этого не произошло. Я думаю лучше чтобы этого
не было.

2) Синий цвет литеральных символов очень темный, на фоне темнооранжевого выделения
плохо отличим от черного

3) Синий также плохо заметен на знаках препинания - см описание для [!,;] 

Надо обсудить и исправить.

Reported by oasychev on 2015-01-23 14:02:51

Comments (43)

  1. Oleg Sychev reporter 
    Пахомов, вы с этими траблами что-то решать будете?

Также всем слегка оффтопный вопрос - 
4) у кого на блоке конструктора работает сворачивание/разворачивание секций. У меня
не работает. Прошу проверить - если ни у кого не пашет, надо фиксить...

    Reported by oasychev on 2015-02-08 22:17:26

  2. Former user Account Deleted 
    4) у меня не работает

    Reported by grvlter on 2015-02-10 21:29:41

  3. Valeriy Streltsov 
    4) у меня тоже не работает.

    Reported by vostreltsov on 2015-02-15 16:10:08

  4. Former user Account Deleted 
    4) В блоке нету библиотеки форм js поэтому не инициализизуются сворачивалки.
вот такой кусок кода писал чтобы окно хоть както открывалось: 
if (M.form && M.form.shortforms) {
    M.form.shortforms({"formid":"mformauthoring"}); // TODO - find native way to init
headers collapce functionatily
}

    Reported by TOPT.iiiii on 2015-02-15 18:10:01

  5. Oleg Sychev reporter 
    Советую обратиться к Клевцову, он у вас консультировался - теперь наоборот. У него все
сворачивается....

    Reported by oasychev on 2015-02-15 19:47:57

  6. Oleg Sychev reporter 
    5) при выделении в регексе, графе и т.д. (когда не обновляется регекс, а только меняется
выделение) не происходит перезапроса тестовых строк - поэтому не отображается корректно
выделение совпавших участков тестов. При обновлении тестовых строк выделение отображается,
так что ошибка явно в ajax-запросе. Прошу исправить - и побыстрее!

    Reported by oasychev on 2015-02-17 23:07:50

  7. Former user Account Deleted 
    приходит ошибка:

{
    "error": "error\/Error: match was found but no match information returned",
    "stacktrace": "* line 249 of \/question\/type\/preg\/preg_matcher.php: qtype_preg_exception
thrown\n* line 557 of \/question\/type\/preg\/preg_matcher.php: call to qtype_preg_matching_results->validate()\n*
line 119 of \/question\/type\/preg\/authoring_tools\/preg_regex_testing_tool.php: call
to qtype_preg_matcher->match()\n* line 96 of \/question\/type\/preg\/authoring_tools\/preg_regex_testing_tool.php:
call to qtype_preg_regex_testing_tool->data_for_accepted_regex()\n* line 37 of \/question\/type\/preg\/authoring_tools\/preg_regex_testing_tool_loader.php:
call to qtype_preg_regex_testing_tool->generate_json()\n* line 40 of \/question\/type\/preg\/authoring_tools\/preg_regex_testing_tool_loader.php:
call to qtype_preg_get_json_array()\n",
    "debuginfo": "\nError code: Error: match was found but no match information returned\n$a
contents: ",
    "reproductionlink": "http:\/\/edu.vstu.org\/"
}

    Reported by TOPT.iiiii on 2015-02-18 14:42:11

  8. Oleg Sychev reporter 
    5) исправлено - был матчер

Жду исправления/обсуждения остальных. Обсуждение вариантов - по 2) и 3), возможно 1)
- Валерий, Григорий - подключайтесь....

    Reported by oasychev on 2015-02-18 20:58:49

  9. Oleg Sychev reporter 
    6) регекс  (?###parens_opt<)(?###word)(?###>) среди инструментов авторинга корректно
отобразился лишь в дереве. Граф отказался рисоваться вообще, в описании " $$1 in parens
or not"

    Reported by oasychev on 2015-02-23 20:42:20

  10. Oleg Sychev reporter 
    Добавляю Иванова т.к. пункт 6 выявил, что граф не поддерживает шаблоны с параметрами.

В дереве предлагаю в тултипах шаблонов писать "a template ..." а не просто "template"

В графе кстати странные тултипы к надписям простых ассертов, почему-то то лезут id
какие-то в видимый пользователю текст...

    Reported by oasychev on 2015-02-23 20:56:56

  11. Former user Account Deleted 
    Поправлен пункт 6 в графе.

    Reported by ZluMYO on 2015-03-01 14:52:06

  12. Oleg Sychev reporter 
    Нужно доделать вариант с custom_parens - множеством параметров. И юнит-тесты на граф
с новыми видами узлов, чтобы не разладилось...

    Reported by oasychev on 2015-03-01 22:15:50

  13. Oleg Sychev reporter 
    Я думаю в случае шаблона с несколькими параметрами в графе надо использовать два уровня
кластеров. На верхнем уровне один кластер - на весь шаблон, в нем надпись со смыслом
шаблона. Внутри него несколько кластеров - по каждому параметру, более тусклой версией
того же цвета.
Если параметр один, то второй уровень кластеров не нужен конечно...

    Reported by oasychev on 2015-03-01 22:21:37

  14. Oleg Sychev reporter 
    4) fixed.

7) ВАЖНО: описание не показывает шаблоны с параметрами - $$1 и т.д.
Описания параметров подставляются в шаблоны как и любые операнды, попробуйте описать
шаблоны со скобками и вы это увидите - $1 в любом количестве круглых скобок например
- где вместо $1 описание параметра шаблона.
Отдельно протестировать не только parens, но и custom_parens шаблон - в нем три параметра,
убедиться что нормально работает при более чем одном параметре

8) ВАЖНО: окно ввода регекса на форме авторинга перестало растягиваться за правый нижний
угол, также из него исчезла иконка растягивания

    Reported by oasychev on 2015-03-10 16:32:31

  15. Oleg Sychev reporter 
    по 2) предлагаю лучше сконтрастировать цвета фона выделения и цвета символов, мне кажется
что логичнее осветлять фон т.к. другие виды фона (обычный, серый для стандартного участка)
у нас довольно светлые, чтобы не испортить отображение символов на них

по 3) предлагаю попытаться знаки препинания, выглядящие маленькими - обычно узкими
(точки, запятые, восклицательный знак, минус и т.д.) отображать моноширинным шрифтом
- он обычно содержит гораздо более заметные их варианты. Если есть другие предложения
 - пишите.

    Reported by oasychev on 2015-03-10 16:32:39

  16. Oleg Sychev reporter 
    7 - пофиксил сам
8 - работает Валерий.

9) Иванову - проблемы с отображением шаблонов с несколькими параметрами типа custom_parens_req
- во-первых, он сводит все параметры в единый узел что недопустимо. Если в шаблоне
больше одного параметра, то для каждого параметра нужно отдельный под-кластер, более
легкого цвета. Ну и идея подставлять "содержание" вместо каждого параметра делает строку
не больно читаемой, лучше отдельные строки или данные использовать - или уж взять из
описания что там внутри.

    Reported by oasychev on 2015-03-17 14:59:59

  18. Former user Account Deleted 
    "... лучше отдельные строки или данные использовать - или уж взять из описания что там
внутри."
Хорошо. http://prntscr.com/6jwjc9
Всё же я не до конца понимаю, как это должно выглядеть на графе. Вы не могли бы на
примере
    (?###custom_parens_req<)<(?###,)(?###parens_opt<)(?###word)(?###>)(?###,)>(?###>)
привести простенький рисунок конечного результата (с tooltip'ами)? Всё таки объяснять,
как должно выглядеть изображение посредством изображений понятнее, чем словами.

    Reported by ZluMYO on 2015-03-22 13:42:25

  19. Oleg Sychev reporter 
    Я уже писал " Если в шаблоне больше одного параметра, то для каждого параметра нужно
отдельный под-кластер, более легкого цвета.". Нарисовать проще при встрече, вы около
политеха во вторник или среду пробегать не будете?

    Reported by oasychev on 2015-03-22 22:14:58

  20. Former user Account Deleted 
    Вытолкнул изменения с исправленным видом шаблона с параметрами. Жду уточнений цветовой
схемы.

    Reported by ZluMYO on 2015-03-23 19:04:59

  21. Oleg Sychev reporter 
    Ну смысл примерно этот, только меня линии (дуги) удивляют. По традиции они у нас не
упираются в кластер, а ведут внутрь к узлам. Т.е. стрелка от start должна вести к <
; от нее - к word, оттуда к > и от него к end.

По цвету - ну серые так серые, если другие цвета заняты - но я думаю что добавочные
кластеры для внутренних параметров должны быть более светлым оттенком того же цвета,
что и главный кластер шаблона. И я бы добавил в описание шаблона массив строк с именами
параметров - показанные в соответствующих шаблонах (т.е. в данном случае opening bracket,
text inside brackets, closing bracket).

    Reported by oasychev on 2015-03-24 21:17:07

  23. Former user Account Deleted 
    Насчёт стрелок. Может внутри их провести, а внешние так и оставить касающимися?

    Reported by ZluMYO on 2015-03-25 05:00:03

  24. Former user Account Deleted 
    Добавил строки с описанием параметров для сложных шаблонов.

    Reported by ZluMYO on 2015-03-25 18:21:38

  25. Oleg Sychev reporter 
    А внутренние стрелки откуда будут начинаться если местные касаются и останавливаются
на границе?

    Reported by oasychev on 2015-03-25 19:37:35

  26. Former user Account Deleted 
    Просто моё видение. Так то, как угодно...

    Reported by ZluMYO on 2015-03-25 19:42:47

  27. Oleg Sychev reporter 
    Не хочется разрывать линии стрелок - до сих пор граф был безразрывным. Если новая стрелка
пойдет от того же места на границе кластера, где кончалась предыдущая - тогда можно.
Но еще мне кажется эти стрелки не дадут "слипнуться" внутренним под-кластерам для аргументов
и сделают их более удобными.

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

    Reported by oasychev on 2015-03-25 19:45:10

  28. Former user Account Deleted 
    Вернул обычные стрелки.

    Reported by ZluMYO on 2015-03-25 20:24:14

  29. Oleg Sychev reporter 
    Приложите сюда примеры картинок - так удобнее же объяснять, чем каждый раз заливать
ваш код на edu.vstu.org и смотреть...

    Reported by oasychev on 2015-03-25 22:09:57

  30. Oleg Sychev reporter 
    Иванов - что случилось с вашим клоном? Для доступа требует пароль, из списка исчез.
Не могу вытянуть изменения. Кто-нибудь может? Последнее доступное - три дня назад,
вчерашнего изменения не видно...

    Reported by oasychev on 2015-03-26 21:24:50

  31. Former user Account Deleted 
    Вы смотрите этот? https://code.google.com/r/zlumyo-preg-28/
Я ничего с ним не делал такого.

    Reported by ZluMYO on 2015-03-27 06:12:18

  32. Oleg Sychev reporter 
    Нда, сами по себе стрелки не очень помогли. Может быть вставить по узлу-точке между
кластерами-аргументами, чтобы не лепились друг к другу?

    Reported by oasychev on 2015-03-27 20:16:15

  33. Oleg Sychev reporter 
    Владимир, в графе появились надписи над стрелками для параметров. Вам надо разместить
их в светло-серых кластерах для этих параметров.

    Reported by oasychev on 2015-03-28 22:37:42

  34. Oleg Sychev reporter 
    Да, и если в шаблоне только один параметр (как в parens_req) я бы второй, слабый кластер
не выводил - ибо смысла в нем нет. Там нужно условие когда параметров более одного.

    Reported by oasychev on 2015-03-28 22:56:13

  35. Former user Account Deleted 
    "Владимир, в графе появились надписи над стрелками для параметров. Вам надо разместить
их в светло-серых кластерах для этих параметров"
Что значит "появились"? Только приведите пример регулярного выражения.

    Reported by ZluMYO on 2015-03-29 07:11:13

  36. Former user Account Deleted 
    Добавил точки между параметрами и убрал кластер для одиночного параметра. На счёт тех
подписей: по всей видимости, вы перепутали с синтаксическим деревом. У которого, кстати
говоря, странные tooltip'ы на этих подписях http://prntscr.com/6mqhhm

    Reported by ZluMYO on 2015-03-29 07:41:40

  37. Oleg Sychev reporter 
    Я не перепутал. Я говорю что эти же надписи, которые появились в дереве, очень полезно
сделать надписями наверху для светло-серых кластеров с отдельными параметрами, они
будут пояснять их суть. Для отдельного параметра не нужно.

    Reported by oasychev on 2015-03-29 10:58:18

  38. Oleg Sychev reporter 
    Григорий - насчет тултипов посмотрите...

    Reported by oasychev on 2015-03-29 11:01:19

  39. Oleg Sychev reporter 
    *"Для отдельного параметра не нужно" - имелся ввиду шаблон с одним параметром, там из
названия и так понятно (Григорий - возможно и в дереве для шаблона с одним параметром
не нужно)

    Reported by oasychev on 2015-03-29 11:02:37

  40. Former user Account Deleted 
    Сделал подписи к кластерам как в синтаксическом дереве.

    Reported by ZluMYO on 2015-03-29 14:55:38

  41. Former user Account Deleted 
    Лишние тултипы у дерева убрал и надпись на стрелке в шаблоне с одним параметром тоже

    Reported by grvlter on 2015-03-29 19:09:31

  42. Oleg Sychev reporter 
    9 fixed

Валерий - обратите внимание, выделение не срабатывает при захвате например двух параметров
custom_parens - причем и в тексте регекса, и в графе так что проблема скорее всего
на стороне сервера.

    Reported by oasychev on 2015-03-31 22:59:24

  44. Log in to comment
Assignee
Type
bug
Priority
critical
Status
closed
Component
–
Votes
0
Watchers
0
Jira: the preferred issue tracker for Bitbucket. Join the team!