Bug 13011 - menu-methods/fluxbox не добавляет Icon
Summary: menu-methods/fluxbox не добавляет Icon
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: fluxbox (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: viy
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-01 21:01 MSD by Artem Zolochevskiy
Modified: 2008-06-24 15:33 MSD (History)
2 users (show)

See Also:

Attachments
fluxbox.patch (1.55 KB, patch)
2007-10-09 19:32 MSD, Sergey V Turchin 		no flags Details | Diff
Обновлённая версия патча. (878 bytes, patch)
2007-10-10 14:41 MSD, Slava Semushin 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Artem Zolochevskiy 2007-10-01 21:01:33 MSD 
# rpm -qf /etc/menu-methods/fluxbox
fluxbox-1.0-alt1rc3

на примере mutt1.5
$ grep Icon /usr/share/applications/mutt1.5.desktop
Icon=mutt
$ rpm -ql mutt1.5 | grep icons
/usr/share/icons/hicolor/16x16/apps/mutt.xpm
/usr/share/icons/hicolor/32x32/apps/mutt.xpm
/usr/share/icons/hicolor/48x48/apps/mutt.xpm

при этом
$ grep mutt /etc/X11/fluxbox/menu
         [exec] (Mutt) {xvt  -e mutt}

т.е. "иконка" не прописалась.
кажется это справедливо для Icon=xxx без расширений в .desktop файлах.
Comment 1 Artem Zolochevskiy 2007-10-01 21:08:17 MSD 
ой. кажется это недоразумение задокументировано

http://www.freesource.info/wiki/AltLinux/Policy/IconPaths

но, всё равно... :-)
Comment 2 Slava Semushin 2007-10-09 16:51:34 MSD 
Бага, есть. Но мы, с Сергеем, пока не нашли хороший вариант для её решения.

Как найдём -- исправлю :)
Comment 3 Sergey V Turchin 2007-10-09 19:32:33 MSD 
Created attachment 2220 [details]
fluxbox.patch

я добавил функцию locate_icon menu-2.1.35-alt2
Comment 4 Slava Semushin 2007-10-10 14:41:47 MSD 
Created attachment 2221 [details]
Обновлённая версия патча.
Comment 5 Sergey V Turchin 2007-10-10 16:41:24 MSD 
(In reply to comment #4)
> Обновлённая версия патча.
Лишнее. Она не будет пустой никогда, зато функция поиска 2 раза выполняться 
будет.
Comment 6 Slava Semushin 2007-10-11 09:45:03 MSD 
(In reply to comment #5)
> (In reply to comment #4)
> > Обновлённая версия патча.
> Лишнее. Она не будет пустой никогда, зато функция поиска 2 раза выполняться 
> будет.

Ясно, обновил у себя menu-method, чтобы он игнорировал иконки altlinux.png и
default_section.png -- теперь меню смотрится более сносно, правда, ф-ция поиска
выполняется аж три раза...

Zerg, может ты уберёшь эти иконки по умолчанию? Ну нафиг они нужны, а? Особенно
когда их много и половина меню желтая. Вся прелесть иконок, чтобы они были
разными и соотествовали приложению, чтобы это приложение можно было бы сразу
найти в меню. А если нет иконки, так и не надо вставлять какую-то дефолтную IMHO.
Comment 7 Sergey V Turchin 2007-10-11 15:10:20 MSD 
Нет там больше altlinux
Comment 8 Sergey V Turchin 2007-10-11 15:12:35 MSD 
(In reply to comment #6)
> А если нет иконки, так и не надо вставлять какую-то дефолтную IMHO.
Надо. Это в более едином стиле. У кого нет иконки - баг. Либо у меня любо в 
конкретном пакете.
Любо есть все иконки, либо нет вообще, либо только у секций меню.
Comment 9 Slava Semushin 2007-10-20 20:03:06 MSD 
(In reply to comment #8)
> (In reply to comment #6)
> > А если нет иконки, так и не надо вставлять какую-то дефолтную IMHO.
> Надо. Это в более едином стиле.

Я сделал так, как было раньше и как считаю нужным.

Баг исправлен в 1.0.0-alt1, который сейчас пойдёт в Сизиф.