Укажите отработанное время

Отработанное время:
Продуктивное время:
Bug 6494 - Переделать диалог About для eterhack   Make a simular bug
Summary: Переделать диалог About для eterhack
Status: CLOSED FIXED
Alias: None
Product: WINE@Etersoft
Classification: Продукты (Products)
Component: Стандартные диалоги (show other bugs)
Version: 2.0
Hardware: PC All
: P3 normal
Target Milestone: ---
Deadline: 2011-09-20
Assignee: Роман Дадьков
QA Contact: Andrey Vusik
URL:
Whiteboard:
Keywords:
: 5697 (view as bug list)
Depends on:
Blocks: 7054 5101 8500
  Show dependency treegraph
 
In work:
Reported: 2010-11-18 16:44 MSK by Виталий Перов
Modified: 2012-07-24 21:22 MSK (History)
6 users (show)

See Also:
Заявки RT:
Связано с:
Дата напоминания:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Виталий Перов 2010-11-18 16:44:33 MSK
Для синхронизации с официальным wine пришлось временно откатить все наши изменения, связанные с диалогом About.

Патч, откатывающий наши изменения:

commit 038543baeb23cf695a9d19ac8327febf5b478a87
Author: Vitaly Perov <vitperov@etersoft.ru>
Date:   Thu Nov 18 16:20:26 2010 +0300

    shell32: revert all etersoft code in about dialog

Откатить пришлось потому, что в официальном wine теперь другой механизм добавления авторов.
Переделать пока нет времени, а обновляться до pure надо.

Новый механизм добавляется коммитом:
commit 40977bf1d2f0f11a24fd9330dffac264fced2306
Author: Alexandre Julliard <julliard@winehq.org>
Date:   Thu Nov 11 12:24:46 2010 +0100

    shell32: Store the raw AUTHORS file in resources instead of trying to
convert to a C array.

Требуется вернуть прежнюю систему.
Для проверки можно сравнить с eter-1.0.12.
Для этого можно в любой бутылке запустить winecfg.
Обратить внимание на вкладку "О программе" и на открывающиеся окно при нажатии на "Подробнее".
Comment 1 Иван Загарский 2010-12-04 17:12:37 MSK
Где расположены эти патчи?
Comment 2 Александр Морозов 2010-12-04 17:34:30 MSK
> Где расположены эти патчи?
В репозитории git.office:/projects/eterhack.git в ветке eterhack.
Comment 3 Иван Загарский 2010-12-18 19:49:53 MSK
Начал возврат прежней системы (восстановление изменений связанных с диалогом About)
Comment 4 Иван Загарский 2010-12-22 20:54:35 MSK
Произвожу добавление списка авторов с учетом нового механизма добавления
Comment 5 Иван Загарский 2010-12-24 19:40:43 MSK
Изменил механизма добавления, произвожу формирование списка авторов
Comment 6 Иван Загарский 2010-12-25 22:57:56 MSK
Переделал добавление авторов etersoft, внес все необходимые изменения. Патч прилагается.
Comment 7 Иван Загарский 2010-12-26 18:16:39 MSK
Произвожу оптимизацию текста программы.
Comment 8 Иван Загарский 2010-12-27 22:42:27 MSK
Вынес  прорисовку формы и добавление авторов в отдельный файл. Оптимизирую код.
Comment 9 Иван Загарский 2011-01-14 14:06:02 MSK
Начинаю структурирование кода, отвечающее за прорисовку окна.
Comment 10 Иван Загарский 2011-01-14 19:54:07 MSK
Завершил оптимизацию кода.Сделал необходимый коммит.Произвожу обновление ветки.
Comment 11 Иван Загарский 2011-01-15 17:17:10 MSK
Обновил ветку, проверил работоспособность внесенных изменений; по сделанному коммиту создал файл и выполнил массовую рассылку. Приступаю к решению проблемы по отрисовке кнопки  "About" на вкладке "О программе".
Comment 12 Иван Загарский 2011-01-19 17:08:11 MSK
Внес изменения по сделанным замечаниям. Произвожу вычисление размера файла.
Comment 13 Иван Загарский 2011-01-19 17:10:23 MSK
Где распологается модуль с прорисовкой окна и вкладок winecfg?
Comment 14 Александр Морозов 2011-01-19 17:15:40 MSK
> Где распологается модуль с прорисовкой окна и вкладок winecfg?
programs/winecfg
Comment 15 Иван Загарский 2011-01-24 17:19:01 MSK
Исправил сделанные замечания. Сделал коммит по сделанным изменениям и выполнил массовую рассылку.
Comment 16 Иван Загарский 2011-01-24 18:57:17 MSK
Выяснил проблему отрисовки кнопки "Подробнее..." на вкладке "О программе" программы winecfg. Области отрисовки выходила за рамки определения области диалогового окна, вследствие чего и возникла данная проблема. Решением данной проблемы может служить определение новых размеров области отрисовки кнопки "Подробнее...".
Comment 17 Иван Загарский 2011-01-24 19:33:19 MSK
Проверил работоспособность внесенных изменений, сделал рассылку.
Comment 18 Иван Загарский 2011-01-25 21:22:56 MSK
По последним замечаниям сделал изменения, сделал рассылку.
Задача решена.
Comment 19 Иван Загарский 2011-01-26 19:40:47 MSK
*** Bug 5697 has been marked as a duplicate of this bug. ***
Comment 20 Svetlana Zhukova 2011-04-20 16:50:42 MSK
Бутылка <wine@cellar bottle bugs/6494>

WINE@Etersoft 1.0 SQL 1.0.12-eter8.21/19

$ winecfg

вкладкa "О программе" , окно при нажатии на "Подробнее" Все в порядке.
Comment 21 Денис Баранов 2011-04-22 19:32:48 MSK
(В ответ на comment #20)
> WINE@Etersoft 1.0 SQL 1.0.12-eter8.21/19

А должно быть в eterhack и eterschool все в порядке быть.
Comment 22 Svetlana Zhukova 2011-04-25 11:14:00 MSK
бут.<wine@eterschool bottle bugs/6494
WINE@Etersoft 1.0 School 1.7.1-eter1/1

бут. wine@eterhack bottle bugs/6494
WINE@Etersoft 1.0 SQL 1.7.1-eter1/1

все красиво и четко
Comment 23 Константин Кондратюк 2011-09-02 20:06:41 MSK
Проверить перед релизом, когда появится что-нибудь вроде rc-сборки.
Comment 24 Svetlana Zhukova 2011-09-06 13:53:23 MSK
Все в порядке.
Comment 25 Vitaly Lipatov 2011-09-17 18:51:11 MSK
(В ответ на comment #22)
> бут.<wine@eterschool bottle bugs/6494
> WINE@Etersoft 1.0 School 1.7.1-eter1/1
> 
> бут. wine@eterhack bottle bugs/6494
> WINE@Etersoft 1.0 SQL 1.7.1-eter1/1
> 
> все красиво и четко
Должно быть не всё красиво и чётко, а так же, как в 1.0.12

Пожалуйста, трудитесь описывать критерии проверки: что проверяли, что получилось.

А то «я проверил, всё хорошо». Чего проверял, чего хорошо, что значит «хорошо»...

Почему я должен ходить за вами и перепроверять?

Смотрел wine-etersoft-1.3.27-alt1.15
всё красиво и чётко, и не то, что нужно.
Comment 26 Роман Дадьков 2011-09-17 19:16:37 MSK
приступил.
нашел 4 патча, которые отправлялись grosso по этой баге. коммиты в них в настоящей версии eterhack не присутствуют.
Comment 27 Роман Дадьков 2011-09-17 22:44:26 MSK
в библиотеке shell32 отсутствует файл shres.rc. 
пока не удалось собрать все части из патчей в единую систему, отсутствуют прототипы некоторых функций.
Comment 28 Роман Дадьков 2011-09-20 11:53:36 MSK
правлю файл прорисовки окна и добавления списка авторов
Comment 29 Роман Дадьков 2011-09-20 21:56:38 MSK
сделал вкладку. внес изменения в файлы ресурсов. сделал перевод в po файлах.
отправил патч на wine-patches@
Comment 30 Роман Дадьков 2011-09-28 15:48:50 MSK
переделал функцию etersoft_picture_load(), но она, в первоначальном виде, каким-то образом влияла на функцию etersoft_text_info(HWND hWnd, LPCWSTR szApp), пока не удалось понять как именно.
решаю проблему.
Comment 31 Роман Дадьков 2011-10-03 22:26:17 MSK
видимо проблема в том, что поток в который загружается картинка и поток, в который загружаются имена вкладок, как-то неважно взаимодействуют друг с другом. пока не удалось решить проблему.
Comment 32 Роман Дадьков 2011-10-04 22:43:54 MSK
исправил создание вкладок. переделал функцию загрузки картинки.
отправил патч на wine-patches@
Comment 33 Роман Дадьков 2011-10-05 13:28:08 MSK
внес изменения.
отправил патч на wine-patches@.
Comment 34 Роман Дадьков 2011-10-07 15:57:51 MSK
переделываю загрузку.
Comment 35 Роман Дадьков 2011-10-08 20:31:04 MSK
переделал загрузку при помощи функций 
wine_get_dos_file_name и LoadImageW
отправил патч на wine-patches@.
Comment 36 Роман Дадьков 2011-10-10 14:17:04 MSK
работаю над багой
Comment 37 Роман Дадьков 2011-10-10 15:54:38 MSK
переделываю патч
Comment 38 Роман Дадьков 2011-10-10 21:17:57 MSK
ошибка возникала из-за неверной записи в ru.po. проблема устранена.
отправил патч на wine-patches@.
Comment 39 Роман Дадьков 2011-10-18 18:30:44 MSK
внес изменения в etersoft_about.c и etersoft_about.h.
отправил патч на wine-patches@.
Comment 40 Александр Морозов 2011-10-18 19:18:39 MSK
Приложил патч к eterhack
Comment 41 Andrey Vusik 2011-12-13 19:52:57 MSK
Кривовато как-то всё..
1) Картинка Wine не по размеру окна.
2) Непонятно, зачем нужны стрелки-переключатели разработчиков? Если посмотреть разработчиков Etersoft, то разработчики Wine пропадают. Вернуть их можно только этими самыми стрелками, наверное для этого они и были прикручены=)
3) Etersoft License при открытии прорисовывается дважды. И скопировать что-то оттуда проблематично - тормозит...
4) Следует как-то обозначить возможность нажать на ссылку проверки подлинности.
Comment 42 Роман Дадьков 2012-03-22 20:59:41 MSK
занимаюсь доработкой. странно, но у меня логотип отображается в размер окна.
Comment 43 Роман Дадьков 2012-03-26 14:05:06 MSK
на самом деле, стрелки не были прикручены искусственно. это как и то, что первая вкладка пропадает при нажатии на вторую, результат того, что окошко слишком маленькое. поправил.
работаю над ссылкой.
Comment 44 Роман Дадьков 2012-04-02 16:18:14 MSK
переделал панельс вкладками. добавил изменение вида курсора над ссылкой.

дело в том, что в отличие от простых окон, для которых
if (GetDlgCtrlID((HWND)wParam) == IDC_ABOUT_STATIC_LINK) {
            SetCursor(LoadCursorW(0, (LPWSTR)IDC_HAND));
            return TRUE;
        }
        break;
для диалоговых окон перед return добавляется
            SetWindowLongW(hWnd, 0, TRUE);

отправил патч на wine-patches@
Comment 45 Роман Дадьков 2012-04-03 12:49:23 MSK
сделал для eter-2.0.0
отправил патч на wine-patches@