View Issue Details

IDProjectCategoryView StatusLast Update
0001940WishesMacropublic2013-02-03 00:33
ReporterMaximus Assigned Tovskirdin  
PrioritynormalSeverityfeatureReproducibilityhave not tried
Status assignedResolutionopen 
Product Version3.0 
Summary0001940: Встроенный редактор макросов
DescriptionПосле записи макроса можно нажать CtrlShift. и подредактировать то что получилось.

Хорошо бы иметь возможность подредактировать макрос уже после записи. То есть при работе с временным макросом стало нужно его чуть изменить - записывать его по новой да еще и править после этого - не гуманно.

Чего предлагается. Жмем Ctrl. CtrlShift. и кнопку на которой уже висит макрос.
Фар выдает предупреждение, что макрос уже есть. Удалить: Yes/No.
Вот в этот диалог добавить кнопку Edit. Макрос не удаляем, а открываем диалог, в которой показать старый текст, позволяя редактировать.
TagsNo tags attached.
Build

Activities

vskirdin

2011-12-08 13:04

administrator   bugnote:0008098

Кнопка Change будет доступна если ты удаляешь запись.
Так? (пример на пощупать: http://skirda-file.googlecode.com/files/m1940.7z )

JohnDoe

2011-12-08 14:15

updater   bugnote:0008100

Обсуждение было когда-то: http://forum.farmanager.com/viewtopic.php?f=6&t=5887

vskirdin

2011-12-08 14:21

administrator   bugnote:0008101

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

Maximus

2011-12-08 14:44

reporter   bugnote:0008102

меня устраивает! коммить

Maximus

2011-12-08 14:50

reporter   bugnote:0008103

про обсуждение на форуме - это не про то.

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

DrKnS

2011-12-08 14:52

administrator   bugnote:0008104

> Итоговый результат обсуждения здесь покажи (с учетом того, что есть в тестовом билде)

Это тема отдельного тикета.

Комить.

vskirdin

2011-12-08 16:08

administrator   bugnote:0008106

build 2291

DrKnS

2011-12-08 16:36

administrator   bugnote:0008109

Только может кнопку не после cancel добавлять, а перед?
Cancel традиционно последний.

vskirdin

2011-12-08 17:47

administrator   bugnote:0008111

build 2292

Maximus

2011-12-09 12:12

reporter   bugnote:0008123

падаем при нажатии F11 в диалоге после Change

собстно лечить нада в двух местах
1. зацикливание Dialog::ProcessKey(KEY_F11) -> FrameManager->ProcessKey() -> Dialog::ProcessKey() и т.д. Это из-за DMODE_NOPLUGINS
2. вызов диалога Change наверное нужно убрать за пределы InProcessAssignMacroKey

Maximus

2011-12-09 12:14

reporter   bugnote:0008124

И еще. Хорошо бы перед открытием диалога параметров макроса тушить статус записи (ака красная R в левом-верхнем)
А то не рабатают макросы в диалоге. Обидно, F4, например, на DialogTools

vskirdin

2011-12-09 13:31

administrator   bugnote:0008131

патч давай

vskirdin

2011-12-31 08:42

administrator   bugnote:0008292

Maximus, патч то будет?

Maximus

2011-12-31 08:51

reporter   bugnote:0008293

Времени не было. Вот на праздниках и займусь

DrKnS

2011-12-31 09:23

administrator   bugnote:0008294

> А то не рабатают макросы в диалоге. Обидно, F4, например, на DialogTools

Приехали, это фича.

А если я хочу записать макрос как раз на это F4?

Maximus

2011-12-31 09:36

reporter   bugnote:0008295

DrKnS, что-то тебя не в ту степь...
1. Речь идет о диалоге, в котором редактируется _текст_ макроса. Чем этот диалог такой особенный, что в нем _не_ работают макросы? Насколько я знаю, в фаре имеет право быть только один такой диалог - а именно, диалог ввода макро-клавиши. Зачем другие ограничивать?
2. Как можно называть фичей падение? ( http://bugs.farmanager.com/view.php?id=1940#c8123 ) А ведь оно связано. Переделка диалога в обычный, а не то что есть сейчас - автоматом вылечит и stack overflow.

Maximus

2011-12-31 09:37

reporter   bugnote:0008296

> А если я хочу записать макрос как раз на это F4?

А что по-твоему может мешать?

DrKnS

2011-12-31 09:42

administrator   bugnote:0008297

> Насколько я знаю, в фаре имеет право быть только один такой диалог - а именно, диалог ввода макро-клавиши.

А, пардон, с ним и попутал.

Тогда вопросов не имею.

JohnDoe

2012-03-01 08:59

updater   bugnote:0008653

Предлагаю добавить такую же кнопку в предупреждение о переназначении.
http://forum.farmanager.com/viewtopic.php?p=71880#p71880

P.S.
И кстати [ Change ] стоит исправить на [ Edit ]

vskirdin

2012-03-02 05:45

administrator   bugnote:0008667

Что ты хочешь менять этой кнопкой?

JohnDoe

2012-03-02 07:37

updater   bugnote:0008669

"Менять" ничего. Название [ Change ] не очень корректно в обоих случаях.
Хочу "Править" существующий макрос.

Скриншоты, и "расширенный" вариант предложения тут:
http://forum.farmanager.com/viewtopic.php?p=91903#p91903

vskirdin

2012-03-02 08:38

administrator   bugnote:0008671

Для правки существующего нажми Ctrl. Ctrl. Клавиша, выбери Change и изменяй.
Я не пойму логики: ты только что мучился, рисовал свои последовательности, а потом вдруг решил отредактировать предыдущий макрос. В чём прикол?

JohnDoe

2012-03-02 09:39

updater   bugnote:0008674

Мучался, записывал макрос - выбираешь [ Redefine ].
Но бывает и иначе, например на полдороге вспоминаешь что уже записал его вчера, и лучше подредактировать, чем заново отлаживать - выбираешь [ Edit ].

И этот дополнительный [ Edit ] только часть предложения об унификации 2х диалогов,
Мне интереснее обсудить унификацию.
Полагаю, это выходит за рамки тикета, буду признателен за комментарии на форуме.

Issue History

Date Modified Username Field Change
2011-12-08 10:13 Maximus New Issue
2011-12-08 13:04 vskirdin Note Added: 0008098
2011-12-08 14:15 JohnDoe Note Added: 0008100
2011-12-08 14:21 vskirdin Note Added: 0008101
2011-12-08 14:44 Maximus Note Added: 0008102
2011-12-08 14:50 Maximus Note Added: 0008103
2011-12-08 14:52 DrKnS Note Added: 0008104
2011-12-08 16:08 vskirdin Note Added: 0008106
2011-12-08 16:08 vskirdin Assigned To => vskirdin
2011-12-08 16:08 vskirdin Status new => feedback
2011-12-08 16:36 DrKnS Note Added: 0008109
2011-12-08 17:47 vskirdin Note Added: 0008111
2011-12-09 12:12 Maximus Note Added: 0008123
2011-12-09 12:12 Maximus Status feedback => assigned
2011-12-09 12:14 Maximus Note Added: 0008124
2011-12-09 13:31 vskirdin Note Added: 0008131
2011-12-31 08:42 vskirdin Note Added: 0008292
2011-12-31 08:42 vskirdin Status assigned => feedback
2011-12-31 08:51 Maximus Note Added: 0008293
2011-12-31 08:51 Maximus Status feedback => assigned
2011-12-31 09:23 DrKnS Note Added: 0008294
2011-12-31 09:23 DrKnS Status assigned => feedback
2011-12-31 09:36 Maximus Note Added: 0008295
2011-12-31 09:36 Maximus Status feedback => assigned
2011-12-31 09:37 Maximus Note Added: 0008296
2011-12-31 09:42 DrKnS Note Added: 0008297
2012-03-01 08:59 JohnDoe Note Added: 0008653
2012-03-02 05:45 vskirdin Note Added: 0008667
2012-03-02 07:37 JohnDoe Note Added: 0008669
2012-03-02 08:38 vskirdin Note Added: 0008671
2012-03-02 09:39 JohnDoe Note Added: 0008674
2013-02-03 00:33 alexy Severity minor => feature