SASGIS

Веб-картография и навигация


View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001751SAS.ПланетаРефакторингpublic29-12-2012 09:4430-12-2021 08:59
Reportervdemidov 
Assigned Tovdemidov 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusassignedResolutionopen 
PlatformOSOS Version
Product Version121010 
Target Version25xxxxFixed in Version 
Summary0001751: Вынести операции из пунктов главного меню главной формы в TActionList
DescriptionВынести операции из пунктов главного меню главной формы в TActionList, желательно лежащий в датамодуле. Датамодуль создвать в конструкторе главной формы и передавать ему все что надо через параметры конструктора.

Не уверен насколько просто это получиться сделать, но уже давно пора.
TagsNo tags attached.
Attached Files

- Relationships
parent of 0003143assignedvdemidov Переделать генерацию меню со списком карт 
child of 0001271confirmed Возможность вынести на панель инструментов кнопки "поставить метку" и "сохранить трек в базе меток" 
child of 0002580confirmed Настройка горячих клавиш для пунктов контекстного меню 
Not all the children of this issue are yet resolved or closed.

-  Notes
(0015236)
zed (manager)
12-02-2015 19:34

Т.е. ты предлагаешь половину кода из frmMain перенести в датамодуль? Хотя, там наверное даже не половина, а процентов 80 - различные обработчики OnClick.
(0015237)
vdemidov (manager)
12-02-2015 19:41

В общем именно это и предлагаю. Сейчас файл главной формы просто таки несоразмерно огромный. Почти 8000 строк.
(0015238)
vdemidov (manager)
12-02-2015 19:42

И это будет где-то половина кода. Там еще много инициализации.
(0015240)
zed (manager)
12-02-2015 19:48

Да, достаточно грандиозная задача. Можешь начать и перенести пару пунктов для примера? А потом я подключусь и буду по-немного копи-пастить.
(0015241)
zed (manager)
12-02-2015 20:09

Там ещё оно как-то должно быть завязано на TShortcutManager, чтобы он учитывал и ActionList. По крайней мере в переходный период, пока не перенесутся все пункты меню. А в последствии ShortcutManager должен будет ориентироваться только на этот лист и не зависеть от менюшки.
(0015242)
vdemidov (manager)
12-02-2015 20:17

Легко сказать начать. Я пока к этому не готов. :) Но если ты займешься, то буду только рад. Я сейчас глубоко в матрицы тайлов закопался.
(0015243)
vdemidov (manager)
12-02-2015 20:18

И что делать с TShortcutManager я еще тоже не решил. Точнее делать что-то нужно, и возможно даже до начала миграции в ActionList. Но как это лучше сделать я тоже не знаю. В общем вопросов дофига, а ответов у меня нет и пока не предвидится.
(0015244)
zed (manager)
12-02-2015 20:19

Обнадёжил :)
(0015245)
vdemidov (manager)
12-02-2015 20:31

Вполне может быть, что я вообще все усложняю, и достаточно просто копировать пункты меню в ActionList и просто потом назначать пуктам меню нужные действия и не париться. У меня просто очень мало опыта работы с такими вещами как ActionList и менюшки. Я больше по бекенду, а не по ГУИ специалист :)
(0017645)
vdemidov (manager)
06-09-2016 20:18

Начал наполнять ActionList. Вроде бы проблем быть не должно, но будем смотреть.
(0017647)
vdemidov (manager)
08-09-2016 12:06

А что специфичного у нас из TBX используется? Может реально обойтись стандартными TActionManager, TActionMainMenuBar и TActionToolBar?

Zed, Garl подскажите, а то я что-то не уверен в этом вопросе?
(0017648)
zed (manager)
09-09-2016 05:40

Без понятия.
(0017650)
vdemidov (manager)
09-09-2016 07:47

Zed, ты писал:
> Можешь начать и перенести пару пунктов для примера? А потом я подключусь и буду по-немного копи-пастить.
Начал. Можно продолжать. Предлагаю давать экшенам по возможности понятные имена с префиксом act как у тех что я уже сделал. Пока никакого дата модуля - пока будет проще просто TActionList. Думаю потом перенести все в более навороченный TActionManager будет уже не сложно. Код по возможности причесывать. Заменять проверку и установку Checked пунктов меню и тулбаров на соответствующее обращение к экшену. Генерируемые програмно пункты и подменю пока не трогаем.

Для сокращения конфликтов предлагаю коммитить и пушить максимально часто - в идеале после каждого пункта меню. Я буду идти по главному меню слева направо, но будет это не быстро.
(0017651)
zed (manager)
09-09-2016 08:36

Да, писал. Только это было полтора года назад и у предложения уже истёк срок действия. Так что можешь рефакторить без опаски, конфликтов не будет.
(0017652)
vdemidov (manager)
09-09-2016 09:47

Печально.
(0017656)
vdemidov (manager)
27-09-2016 08:58

Забыл удалить NanimateClick
(0017698)
Garl (manager)
26-11-2016 08:36

а теперь на закрытие панельки результатов поиска нельзя прикрепить хоткей? или я проглядел?
(0017709)
vdemidov (manager)
01-12-2016 08:28

Да, судя по всему, потерялась такая возможность в процессе борьбы с лишними пунктами в списке хоткеев. Там добавилось условие, что у пункта меню должен быть обработчик или экшен, а у этих пунктов видимости панелей ни того ни другого нет.
(0017710)
vdemidov (manager)
01-12-2016 08:30
edited on: 01-12-2016 08:31

Но как мне надоел уже этот ГУЙ. Судя по всему нужно отвлечься. Займусь пока наработкой заготовок для реализации конвейеров подготовки видимого изображеня описанных здесь: http://www.sasgis.org/wikisasiya/doku.php/sasdev:конвейер_подготовки_видимого_изображения

(0017747)
vdemidov (manager)
17-12-2016 17:21

Вернул хоткеи для панелек. И для результатов поиска в том числе.

- Users who viewed this issue
User List Anonymous (3095x), vdemidov (37x), Aveveritas (1x), Garl (6x), stepanxxx (1x), netsky (2x), ygorigor (1x), gma (1x), alexandr-dolgov (2x), bk99 (6x), RedRat (1x), zed (8x)
Total Views 3161
Last View 02-07-2022 09:12

- Issue History
Date Modified Username Field Change
29-12-2012 09:44 vdemidov New Issue
29-12-2012 09:45 vdemidov Status new => confirmed
29-12-2012 09:45 vdemidov Product Version => 121010
29-12-2012 09:45 vdemidov Target Version 121010 => 27xxxx
29-12-2012 09:47 vdemidov Relationship added child of 0001271
23-01-2015 11:04 vdemidov Relationship added child of 0002580
12-02-2015 19:34 zed Note Added: 0015236
12-02-2015 19:41 vdemidov Note Added: 0015237
12-02-2015 19:42 vdemidov Note Added: 0015238
12-02-2015 19:48 zed Note Added: 0015240
12-02-2015 20:09 zed Note Added: 0015241
12-02-2015 20:17 vdemidov Note Added: 0015242
12-02-2015 20:18 vdemidov Note Added: 0015243
12-02-2015 20:19 zed Note Added: 0015244
12-02-2015 20:31 vdemidov Note Added: 0015245
06-09-2016 20:18 vdemidov Note Added: 0017645
08-09-2016 12:06 vdemidov Note Added: 0017647
09-09-2016 05:40 zed Note Added: 0017648
09-09-2016 07:47 vdemidov Note Added: 0017650
09-09-2016 08:36 zed Note Added: 0017651
09-09-2016 09:47 vdemidov Note Added: 0017652
27-09-2016 08:58 vdemidov Note Added: 0017656
27-09-2016 12:44 vdemidov Target Version 27xxxx => 181221
27-09-2016 12:50 vdemidov Assigned To => vdemidov
27-09-2016 12:50 vdemidov Status confirmed => assigned
17-10-2016 07:56 vdemidov Relationship added parent of 0003143
26-11-2016 08:36 Garl Note Added: 0017698
01-12-2016 08:28 vdemidov Note Added: 0017709
01-12-2016 08:30 vdemidov Note Added: 0017710
01-12-2016 08:31 vdemidov Note Edited: 0017710 View Revisions
17-12-2016 17:21 vdemidov Note Added: 0017747
01-12-2017 07:25 vdemidov Target Version 181221 => 190707
23-07-2019 14:16 vdemidov Target Version 190707 => 191221
23-07-2019 17:04 vdemidov Target Version 191221 => 211230
30-12-2021 08:59 zed Target Version 211230 => 25xxxx



Copyright © 2007 - 2022 SAS.Planet Team