d4x и диалог открытия файла

[zl0i]

В d4x диалог открытия файла отличается от стандартного используемого в gtk(достаёт же?). Так уж получилось, что d4x больше не разрабатывается. И поэтому просьба к знающим gtk заменить этот староватый диалог на стандартный gtk`шный. Можете проигнорировать. Возможно сам разберусь, как сделать, но придется все-таки повозиться с gtk(Патчи с меня). Знающий, думаю, быстро найдет решение(теоретически там нужно заменить одну функцию).
Версия d4x 2.5.7.1-5
ядро 2.6.27-13-generic
Система Ubuntu 8.10

[zl0i]

Не все так просто оказалось, простой заменой одной функции не обойдется. Оказалось диалог вызывается старой функцией, а для новой нужны другие параметры, которые мне неизвестны.
Вот, что я выяснил:
Диалоги выбора файла создаются с помощью вызова  gtk_file_selection_new (const gchar *title); Он-то и устанавливает "старый" тип окна.
А вот новый тип - gtk_file_chooser_dialog_new ("open file",
                                      parent_window,
                                      gtk_file_chooser_action_open,
                                      gtk_stock_cancel, gtk_response_cancel,
                                      gtk_stock_open, gtk_response_accept,
                                      null);
Короче, для меня, не знающего gtk, можно зарыться, копать придется дальше.

[zl0i]

Все-таки разобрался. Как и обещал выкладываю deb-пакет и патч. Вот теперь во всех моих приложениях общий стиль . Тестим , хотя там мало изменений, но все ж могу где-нить наломать дров. Первый патч все-таки. Вот это я понимаю дух OpenSource.
З.Ы. Кто знает, как этот патч отправить разработчикам ubuntu? И могут ли его принять?

[K_E_S_T]

Надо зарегестрироваться на ланчпаде и записаться в группу тестеров/разработчиков.

ну а примт они его или нет - не известно

[digiwhite]

А зачем вы оставили в коде:

Код: [Выделить]

//gtk_widget_destroy(filesel->browser); //don`t need because it destroyed after file selection
ИМХО можно было удалить сразу тогда . А так, вы молодец, если все работает .

[zl0i]

В принципе можно было и удалить, так как уже удалил несколько строк. А так оставил на всякий случай . Просто из-за повторного вызова этой функции выводились warning, и диалог повторно не вызывался. Закоментил, попробовал - работает, вот и оставил.
На ланчпаде уже регился, может и стоит записаться в тестеры, но я не девелопер, заменить диалог на другой это не кучу багов ловить.
Очень хотелось бы чтоб, его приняли, потому как всем новичкам советуют именно d4x. Новички только привыкли к стандартному диалогу gtk, а тут на тебе - и d4x выкидывает "свой". Нет единого интерфейса.

[digiwhite]

Просто закоментированный код, засоряет код вообще ИМХО. Становится тяжелее писать.

[zl0i]

Да, но с другой стороны закомментированный код оставляет задатки на будущее, чтобы не забыть или наоборот откатить назад. Кстати там в коде очень много кода в комментариях и предупреждений при компиляции хватает, так что чистить и чистить, но не всем оно надо - "работает - не трож". Патч поправил. А в патчах такие строки действительно неуместны.
Что-то я никак не найду где на ланчпаде в тестеры попасть и патч им прислать...

[digiwhite]

Вот как раз чтобы откатиться назад, если что не так и существуют системы контроля версий.