View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003303 | Wishes.Plugins | LuaMacro / Macro Browser | public | 2016-09-12 18:23 | 2016-11-14 08:48 |
Reporter | JohnDoe | Assigned To | shmuel | ||
Priority | normal | Severity | feature | Reproducibility | N/A |
Status | closed | Resolution | fixed | ||
Platform | x86 | OS | Windows | OS Version | XP SP3 |
Product Version | 3.0 | ||||
Summary | 0003303: far.Menu: клавиши для BreakKeys | ||||
Description | Предлагаю принимать в качестве BreakKeys принятые в фаре обозначения клавиш, такие как CtrlAltEnter. В InputRecord они легко переводятся через FarNameToInputRecord. Как мне видится, можно внедрить это предложение сохранив совместимость со станой нотацией. Ещё одно предложение: возможность указывать несколько клавиш, через пробел. При возврате из Menu в BreakKey записывается реально нажатая (Это используются мной на практике, через функцию-враппер) | ||||
Additional Information | В тикете 0003134 предлагаются альтернативные улучшения Имхо мои предложения последовательнее и значительно проще. | ||||
Tags | No tags attached. | ||||
|
Первая часть предложения реализована в билде 591 LuaFAR - можно проверять. Насчёт второй части: пока не уверен, что буду делать. |
|
Спасибо, буду первую часть внедрять. Кстати, не стоит ли предпочесть именно этот стандарт наименования клавиш, описав только его в мануале? (оставить и старый, для совместимости) И ещё, для элементов предусмотрено поле AccelKey, которое тоже было бы логично указывать строкой |
|
Насчёт документации - я на данный момент консервативен, а дальше - видно будет. Насчёт AccelKey - сделано в билде 592 LuaFAR. |
|
LuaFAR 594: BreakKeys можно задавать строкой. |
|
Удобная возможность! А несколько клавиш но в таблице планируется? |
|
Я думал об этом, но пришёл к выводу, что это ничего практически не добавит к тому, что сейчас есть, а документацию усложнит (боюсь, что описание far.Menu уже неудобоваримо). Оригинальный вариант с отдельными таблицами имеет преимущество включения в эти таблицы специфических полей, например "action", "filename" и т.п., что позволяет упростить обработку возвращаемого значения. Вариант, добавленный в билде 594, упрощает код в тех случаях, когда эти специальные поля не употребляются. А тот вариант, о котором вы сейчас спросили, является фактически разновидностью "варианта 594", усложнит документацию, не добавляя функциональности. |
|
Именно так. Но и тут возможность перечисления нескольких клавиш способна упростить жизнь, поскольку часто нужны пары типа "Up Num8", "PgUp Num9" |
|
Согласен, но мне кажется где-то нужно остановиться. Иначе для понимания статьи-описания far.Menu нужен будет уровень кандидата наук. В качестве примера "перенасыщенной" функции (по моей вине) можете глянуть описание gsub в библиотеке lrexlib. far.Menu ещё до этого не дошла, и мне бы не хотелось. |
|
Согласен что перенасыщенность вредит (хотя far.Menu до lrexlib.gsub ещё очень далеко) В защиту своей идеи скажу, что дополнение в документации потребовалось бы минимальное (в одну строку), и выглядело бы органично, ведь сейчас документация ссылается на макросы, а там возможность указывать несколько клавиш обычна. P.S. Настаивать не намерен, если это всё то можно закрывать тикет. |
|
Согласен, что ваше предложение имеет смысл, но мне, честно говоря, уже надоело возиться с этой функцией. Поэтому закрываю. |
Date Modified | Username | Field | Change |
---|---|---|---|
2016-09-12 18:23 | JohnDoe | New Issue | |
2016-10-22 16:56 | shmuel | Note Added: 0014947 | |
2016-10-25 14:48 | JohnDoe | Note Added: 0014954 | |
2016-10-25 16:02 | shmuel | Note Added: 0014955 | |
2016-11-13 16:16 | shmuel | Note Added: 0015006 | |
2016-11-13 16:16 | shmuel | Assigned To | => shmuel |
2016-11-13 16:16 | shmuel | Status | new => feedback |
2016-11-13 17:25 | JohnDoe | Note Added: 0015007 | |
2016-11-13 17:25 | JohnDoe | Status | feedback => assigned |
2016-11-13 17:37 | shmuel | Note Added: 0015008 | |
2016-11-13 17:43 | JohnDoe | Note Added: 0015009 | |
2016-11-13 17:55 | shmuel | Note Added: 0015010 | |
2016-11-14 08:09 | JohnDoe | Note Added: 0015015 | |
2016-11-14 08:11 | JohnDoe | Note Edited: 0015015 | |
2016-11-14 08:48 | shmuel | Note Added: 0015016 | |
2016-11-14 08:48 | shmuel | Status | assigned => closed |
2016-11-14 08:48 | shmuel | Resolution | open => fixed |