View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002087 | Far Manager | Copy/Move | public | 2012-04-25 10:40 | 2012-05-17 11:22 |
| Reporter | Maximus | Assigned To | 2useven10 | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | 3.0 | ||||
| Fixed in Version | 3.0 | ||||
| Summary | 0002087: Некорректная обработка Esc в диалоге ошибки копирования | ||||
| Description | При ошибке копирования «The process cannot access the file because it is being used by another process» нажатие Esc приводит к ПРОДОЛЖЕНИЮ копирования. С какой радости? BTW, что значит код возврата "-2" для Message? | ||||
| Tags | No tags attached. | ||||
| Build | 2637 | ||||
|
|
120425.diff (388 bytes)
Index: copy.cpp
===================================================================
--- copy.cpp (revision 7943)
+++ copy.cpp (working copy)
@@ -2775,12 +2775,12 @@
switch (MsgCode)
{
- case -1:
case 1:
return COPY_NEXT;
case 2:
SkipMode=1;
return COPY_NEXT;
+ case -1:
case -2:
case 3:
return COPY_CANCEL;
|
|
|
-2 это Ctrl-Break |
|
|
2 Maximus: Esc == [Skip]. т.е. отменяется копирование текущего файла. Ctrl-Break == [Cancel] - отменяет весь процесс копирования. Твой патч делает их равными и для [Skip] нет удобного хоткея. add: imho такое же поведение надо сделать и при отработке предыдущего диалога (File already exist) а то явная непоследовательность... |
|
|
1. Для Skip - удобная кнопка S 2. Когда юзер видит в диалоге кнопку "Cancel", при нажатии Esc он ожидает выполнения "Cancel". Предлагать что-либо другое - интуитивно непонятно, и более того - обломно. 3. спорить не буду, в bis исправил, а вы как хотите |
|
|
посмотрел код... это единственное место приблизительно из полутора десятков, где Esc != Ctrl-Break. так что проще изменить его. |
|
|
2637 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2012-04-25 10:40 | Maximus | New Issue | |
| 2012-04-25 10:40 | Maximus | File Added: 120425.diff | |
| 2012-04-25 11:16 | 2useven10 | Note Added: 0009049 | |
| 2012-04-25 11:45 | 2useven10 | Note Added: 0009050 | |
| 2012-04-25 11:52 | 2useven10 | Note Edited: 0009050 | |
| 2012-04-25 12:04 | 2useven10 | Note Edited: 0009050 | |
| 2012-04-25 12:20 | Maximus | Note Added: 0009051 | |
| 2012-04-25 12:38 | 2useven10 | Note Added: 0009052 | |
| 2012-04-25 19:40 | 2useven10 | Note Added: 0009057 | |
| 2012-04-25 19:40 | 2useven10 | Assigned To | => 2useven10 |
| 2012-04-25 19:40 | 2useven10 | Status | new => feedback |
| 2012-05-17 11:22 | 2useven10 | Build | => 2637 |
| 2012-05-17 11:22 | 2useven10 | Status | feedback => closed |
| 2012-05-17 11:22 | 2useven10 | Resolution | open => fixed |
| 2012-05-17 11:22 | 2useven10 | Fixed in Version | => 3.0 |
