Bug 21507 - неправильный порядок кнопок в файловых диалогах Mozilla
Summary: неправильный порядок кнопок в файловых диалогах Mozilla
Status: CLOSED WORKSFORME
Alias: None
Product: Sisyphus
Classification: Development
Component: seamonkey (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Andrey Cherepanov
QA Contact: qa-sisyphus
URL:
Keywords: usability
Depends on: 21511
Blocks: 19564
  Show dependency tree
 
Reported: 2009-09-09 21:12 MSD by Zerg
Modified: 2011-06-03 15:05 MSK (History)
3 users (show)

See Also:


Attachments
mozilla-gtkfilepicker-buttonorder.patch (2.45 KB, patch)
2009-09-09 21:12 MSD, Zerg
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zerg 2009-09-09 21:12:42 MSD
Created attachment 3862 [details]
mozilla-gtkfilepicker-buttonorder.patch

В FireFox неправильный порядок кнопок в диалогах открытия файлов
Патч взять из пакета mozilla-xulrunner191
http://download.opensuse.org/repositories/mozilla:/Factory/openSUSE_Factory/src/
Ссылка по теме
https://bugzilla.novell.com/show_bug.cgi?id=527418
Comment 1 Zerg 2009-09-09 21:13:29 MSD
(В ответ на комментарий №0)
> Патч взять
Патч взят
Comment 2 Alexey Gladkov 2009-09-09 21:19:43 MSD
Поясните, что вы имеете ввиду.
Я считаю, что с кнопками всё в порядке.
Comment 3 Zerg 2009-09-09 22:11:11 MSD
(В ответ на комментарий №2)
> Я считаю, что с кнопками всё в порядке.
Повезло. Он просто совпадает с правильным порядком в GNOME.
Я ж привел ссылку на подробности.
https://bugzilla.novell.com/show_bug.cgi?id=527418
Он игнорирует настройку, которую каждая DE в состоянии себе выставить нужную.
Comment 4 Zerg 2009-09-09 22:50:12 MSD
Если что, не бойтесь. В GNOME всё останется по прежнему ;-)

А то в KDE сейчас разнобой. Те диалоги, которые не игнорируют установленный порядок(печать, свойства страницы) одно, а при открытии файла все наоборот
Comment 5 Alexey Gladkov 2009-09-09 23:16:24 MSD
(В ответ на комментарий №4)
> Если что, не бойтесь. В GNOME всё останется по прежнему ;-)

Я не боюсь за них... только сочувствую немного. :)

Сам я не навижу гномовский диалог. Я сам работаю с:

ui.allow_platform_file_picker = false

и сбрасываю эту опцию, только когда проверяю сборку.

FYI: http://konquefox.free.fr
Comment 6 Zerg 2009-09-09 23:31:17 MSD
(В ответ на комментарий №5)
> FYI: http://konquefox.free.fr
Это не особо актуально, если я правильно понял беглым взглядом.
Че-то и различий не заметил после его установки.
В KDE по умолчанию подключается gtk2-qtcurve, и она сейчас гребет иконки oxygen после небольшой обработки иконок напильником. Не все, правда.
В SuSE тему Oxygen ставят для этого в Firefox.

Актуальнее файловые диалоги KDE-шные, чтоб, например, можно было сохранить файл через ssh на машину в интернете или на внешний диск, который примонтируется в момент его открытия в KDE-шном диалоге.
Comment 7 Zerg 2009-09-09 23:34:14 MSD
(В ответ на комментарий №5)
> ui.allow_platform_file_picker = false
Кстати, патч может влиять только на один из них.

А зачем по умолчанию false? В SuSE просмотрел и не нашел, чтоб они выставляли в false. Может, народу будет удобнее? Там фичей побольше и интеграции.
Comment 8 Zerg 2009-09-09 23:42:35 MSD
(В ответ на комментарий №6)
> Че-то и различий не заметил после его установки.
А-а-а, он клевые кнопки добавляет, которыми я постоянно в Konq пользуюсь: "Очистить адрес" и "Вверх". Кнопки "Go" не хватает только
Comment 9 Zerg 2009-09-09 23:46:04 MSD
Еще такое для KDE есть https://addons.mozilla.org/ca/firefox/addon/12196 , но тоже не так принципиально, как "родные" файловые диалоги
Comment 10 Alexey Gladkov 2009-09-09 23:50:32 MSD
(В ответ на комментарий №6)
> Актуальнее файловые диалоги KDE-шные

Вы сделали изменения согласно инструкции:

http://konquefox.free.fr/#trick_filepicker ?

(В ответ на комментарий №7)
> Кстати, патч может влиять только на один из них.

Не понял вас.

> А зачем по умолчанию false? В SuSE просмотрел и не нашел, чтоб
> они выставляли в false. 

Я для SuSE firefox не пакую. У меня для такого умолчания были причины.

> Может, народу будет удобнее? Там фичей побольше и интеграции.

Собственно причиной было создание пакета xulrunner-support-gnome. В нём вкючён нативный диалог. Я думаю, что с ростом интеграции firefox с gnome в этом пакете различий будет больше.
Comment 11 Zerg 2009-09-09 23:58:50 MSD
(В ответ на комментарий №10)
> > Кстати, патч может влиять только на один из них.
> Не понял вас.
Код достаточно дремучий, чтоб я не понял, патчиться оба диалога или только GNOME-овский. Судя по патчу, изменяется только одно место. Если оно используется для инициализации обоих диалогов, то все шоколадно.

> Я думаю, что с ростом интеграции firefox с gnome
> в этом пакете различий будет больше.
А-а-а, тогда ок.
Comment 12 Alexey Gladkov 2009-09-10 00:08:35 MSD
(В ответ на комментарий №11)
> (В ответ на комментарий №10)
> > > Кстати, патч может влиять только на один из них.
> > Не понял вас.
> Код достаточно дремучий, чтоб я не понял, патчиться оба диалога или только
> GNOME-овский. Судя по патчу, изменяется только одно место. Если оно
> используется для инициализации обоих диалогов, то все шоколадно.

Патчится файл /usr/lib/xulrunner/components/nsFilePicker.js. Это обёртка, которая дёргает платформозависимый модуль. На первый взгляд этот патч не должен повредить никому (нужно конечно проверить).

> А-а-а, тогда ок.

На самом деле, я бы с радостью создал бы аналог этого пакета для KDE. В него можно было бы добавить рецепт из:

http://ubuntuforums.org/showthread.php?t=205050

и возможно ещё параметры, специфичные для KDE.
Comment 13 Zerg 2009-09-10 00:30:14 MSD
(В ответ на комментарий №12)
> Патчится файл /usr/lib/xulrunner/components/nsFilePicker.js.
патчиться widget/src/gtk2/nsFilePicker.cpp

Может, его нужно и на xulrunner и на firefox накладывать?
Comment 14 Alexey Gladkov 2009-09-10 00:33:24 MSD
(В ответ на комментарий №13)
> патчиться widget/src/gtk2/nsFilePicker.cpp

Вы сейчас про какой патч говорите ?
Comment 15 Zerg 2009-09-10 00:40:14 MSD
(В ответ на комментарий №14)
> Вы сейчас про какой патч говорите ?
К этой баге прикреплен только 1 патч на данный момент
Comment 16 Zerg 2009-09-10 00:40:40 MSD
(В ответ на комментарий №12)
> На первый взгляд этот патч не должен повредить никому
Он только включает поддержку опции gtk-alternative-button-order , которая по умолчанию в стиле GNOME. Можно и GTK2 пропатчить (#21408), но не актуально, т.к. KDE ее выставляет при своей работе, если целиком запущено.
Comment 17 Alexey Gladkov 2009-09-10 00:43:39 MSD
(В ответ на комментарий №15)
> К этой баге прикреплен только 1 патч на данный момент

Ааааа... а я про изменение для http://konquefox.free.fr/#trick_filepicker .

В том, что приложено к баге патчится только GTK-шная версия. Совершенно верно.
Comment 18 Zerg 2009-09-10 00:57:42 MSD
Дык и в http://konquefox.free.fr/#trick_filepicker тоже только GTK-шная патчиться :-)
Enter "about:config" in the address bar, look for the "ui.allow_platform_file_picker" key and change its value to "false"
Comment 19 Zerg 2009-09-10 00:59:23 MSD
(В ответ на комментарий №18)
> Дык и в http://konquefox.free.fr/#trick_filepicker тоже только GTK-шная
> патчиться :-)
Причем, не работает
Comment 20 Alexey Gladkov 2009-09-10 01:26:02 MSD
(В ответ на комментарий №18)
> Дык и в http://konquefox.free.fr/#trick_filepicker тоже только GTK-шная
> патчиться :-)

Как я выяснил, существует две реализации filepicker:

Нативная (XUL):

http://git.altlinux.org/people/legion/packages/mozilla.org.git?p=mozilla.org.git;a=tree;f=toolkit/components/filepicker;h=b93a87a5fab6e0a0e92af63a8245a513483268c1;hb=c00e9a8110ef3130b6e5ddfbb8eb25134afac9fc

GTK-шная:

http://git.altlinux.org/people/legion/packages/mozilla.org.git?p=mozilla.org.git;a=blob;f=widget/src/gtk2/nsFilePicker.cpp;h=1ef0dc8ac81650eec575f3eddc9ad1fad6d2334b;hb=c00e9a8110ef3130b6e5ddfbb8eb25134afac9fc

Переключаются они тут:

http://git.altlinux.org/people/legion/packages/mozilla.org.git?p=mozilla.org.git;a=blob;f=widget/src/gtk2/nsWidgetFactory.cpp;h=86df17bed608e9bff1acd0b285a7202031d9f589;hb=c00e9a8110ef3130b6e5ddfbb8eb25134afac9fc#l176

Cобственно никто не мешает сделать ешё одну реализацию. Этот компонент будет вызываться везде, где нужно найти файл.

Глядя на нативную реализацию, думаю, можно попробовать подсовывать kdialog вместо реальной работы.
Comment 22 Zerg 2009-09-10 01:43:48 MSD
Давай, я еще одну багу повешу, чтоб не путать с этой
Comment 23 Zerg 2009-09-11 12:39:47 MSD
Этот патч включен в патч, прикрепленный к #21511 , поэтому актуально только если программа не пользуется xulrunner
Comment 24 Sergey V Turchin 2009-09-22 16:08:37 MSD
Для seamonkey пригодиться
Comment 25 Radik Usupov 2011-06-03 07:48:32 MSK
Перечитал тред и ничего не понял.
Прошу проверить с новой версией seamonkey :)
Comment 26 Sergey V Turchin 2011-06-03 15:04:51 MSK
Сейчас уже пофиг. Oxygen-gtk фиксит это, где сможет.
Comment 27 Sergey V Turchin 2011-06-03 15:04:53 MSK
Сейчас уже пофиг. Oxygen-gtk фиксит это, где сможет.