View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002497 | Far Manager | Main menu | public | 2013-09-24 08:41 | 2013-09-24 20:21 |
| Reporter | Denis | Assigned To | DrKnS | ||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 3.0 | ||||
| Fixed in Version | 3.0 | ||||
| Summary | 0002497: Если не на последнем элементе меню нажать и удерживать стрелку вниз, то курсор убегает за пределы меню | ||||
| Description | Если не на последнем элементе меню нажать и удерживать стрелку вниз, то курсор убегает за пределы меню. Со стрелкой вверх аналогично. | ||||
| Tags | No tags attached. | ||||
| Build | 3680 | ||||
|
|
Fix: diff --git i/unicode_far/vmenu.cpp w/unicode_far/vmenu.cpp index a94240b..1cf6950 100644 --- i/unicode_far/vmenu.cpp +++ w/unicode_far/vmenu.cpp @@ -279,7 +279,10 @@ int VMenu::SetSelectPos(int Pos, int Direct, bool stop_on_edge) } if (stop_on_edge && CheckFlags(VMENU_WRAPMODE) && ((Direct>0 && Pos<SelectPos) || (Direct<0 && Pos>SelectPos))) + { + UpdateItemFlags(SelectPos, Items[SelectPos].Flags|LIF_SELECTED); return SelectPos; + } UpdateItemFlags(Pos, Items[Pos].Flags|LIF_SELECTED); |
|
|
А не лучше откатить сброс флагов от DrKnS ? Зачем это сделано? Вообще что за странная практика - менять логику при 'рефакторинге'... Если изменения были нужны - надо писать для чего. |
|
|
Не лучше, потому что это сломает CtrlUp/CtrlDown в фильтрах. Потому что теперь там просто свап, а раньше был тихий ужас на тему "скопируем себе два элемента меню, потом удалим из меню два элемента, потом вставим обратно, но уже в другой последовательности, при этом меню пересчитается 100500 раз и в конце концов selected будет там, где надо". Если после свапа не сбрасывать selected, начнутся странные вещи. А Если "логика" ломается от таких изменений - это проблемы логики. |
|
|
3680 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2013-09-24 08:41 | Denis | New Issue | |
| 2013-09-24 09:32 | Denis | Note Added: 0011447 | |
| 2013-09-24 18:56 | 2useven10 | Note Added: 0011451 | |
| 2013-09-24 18:56 | 2useven10 | Note Edited: 0011451 | |
| 2013-09-24 19:57 | DrKnS | Note Added: 0011452 | |
| 2013-09-24 20:11 | DrKnS | Note Added: 0011453 | |
| 2013-09-24 20:11 | DrKnS | Assigned To | => DrKnS |
| 2013-09-24 20:11 | DrKnS | Status | new => feedback |
| 2013-09-24 20:16 | Denis | Status | feedback => resolved |
| 2013-09-24 20:21 | DrKnS | Build | => 3680 |
| 2013-09-24 20:21 | DrKnS | Status | resolved => closed |
| 2013-09-24 20:21 | DrKnS | Resolution | open => fixed |
| 2013-09-24 20:21 | DrKnS | Fixed in Version | => 3.0 |
