Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Icon request: GNOME's "Default Applications" #857

Closed
heldderarbeit opened this issue Oct 30, 2017 · 29 comments
Closed

Icon request: GNOME's "Default Applications" #857

heldderarbeit opened this issue Oct 30, 2017 · 29 comments

Comments

@heldderarbeit
Copy link

After updating to GNOME 3.26 with its new settings app, some new applications were added. One of them (Default Applications) stands out and doesn't seem to have the right icon assigned.
screenshot_from_2017-10-29_19-51-09

@ghost
Copy link

ghost commented Oct 31, 2017

As I know this is not a new application, and it's one of gnome settings. the wired thing is why it is shown by default in your applications?
image
the current icon for this setting is good in my opinion:
screenshot from 2017-10-31 08-06-15

@varlesh
Copy link
Contributor

varlesh commented Oct 31, 2017

it's action icon

@heldderarbeit
Copy link
Author

Hi, developer Jeremy Bicha from GNOME said "My opinion is that this is indeed a bug in your theme for using a symbolic (monochrome) icon to represent the 'starred' icon instead of just for 'starred-symbolic'"

Source: https://bugzilla.gnome.org/show_bug.cgi?id=789727#c1

@SmartFinn
Copy link
Member

That's right, starred isn't an actions icon, it's status icon. We should fix it on our side.

@SmartFinn SmartFinn reopened this Nov 1, 2017
@SmartFinn SmartFinn added bug and removed upstream bug labels Nov 1, 2017
@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

@heldderarbeit icon name gnome-default-apps-panel ?

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

anybody please post desktop-file, we add symlink to preferences-desktop-default-applications

@heldderarbeit
Copy link
Author

Hi, here is it:

gnome-default-apps-panel.txt

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

That's right, starred isn't an actions icon, it's status icon. We should fix it on our side.

Anyway they use status icon for app - It's upstream bug!

@varlesh varlesh added upstream bug and removed bug labels Nov 1, 2017
@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

Я не буду это фиксить!

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

starred может юзаться во всяких плеерах или прогах, где нужно рейтинг поставить и т.д. И не правильно использовать значок статуса для иконки приложения.

@SmartFinn
Copy link
Member

@varlesh в любом случае в Adwaita, Numix и Paper это цветная иконка в status. Правильно было б соответствовать другим, а создать свои стандарты, тем более что мы еще не мажорити.

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

@SmartFinn А причем здесь Paper? Paper вобще-то уже давно начал разукрашивать монохромные значки, так что мы мало какое отношение к нему теперь имеем.
Я лично не хочу что бы эта желтая звезда маячила в плеерах и выбивалась из стиля монохрома. Собственно, им ничто не мешает поправить просто desktop-файл и вписать нормальное имя, там делов на 5 секунд

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

И все равно, это их косяк. Они используют иконку статуса для приложения. Это нарушение стандарта. Я привел им примеры:
"gnome-settings-default-applications"
or
"preferences-desktop-default-applications"

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

Если идти таким путем, то половина значков в actions должна быть перенесена в status и разукрашена. Тем самым, убив весь замысел и стиль монохрома.

@SmartFinn
Copy link
Member

SmartFinn commented Nov 1, 2017

@varlesh в этом в случае обе стороны не правы, ни первая ни вторая не придерживается соглашения. Так что мне сложно стать на какую-то сторону, я только знаю что у нас отсутствует иконка started для размеров 32px, 48px и т.д., что есть у других тем, и что желтая полоска рейтинга не самая большая проблема.

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

32px и выше ненужны (если говорить откровенно). Эти значки врядли будут юзаться больше чем 24px в прогах, как actions (по сути).
Я повторяю еще раз, мы убьем всю идею монохрома. Я не хочу этого делать... По мне так проще вручную исправить имя на preferences-desktop-default-applications и всё

@SmartFinn
Copy link
Member

@varlesh с их сторони тоже не просто поменять одну строчку, кроме этого нужно протолкнуть новую иконку в adwaita-icon-theme и только потом поменять у себя. И не важно что по твоему мнению 32px и выше не нужны, они есть и на них расчитывают разработчики. Мы можем искуственно затянуть решение этой проблемы, может что поменяется с их стороны, но если нет — нам придется подстраиватся.

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

@SmartFinn Ну допустим, я сейчас сделаю симлинк для 32 и выше на apps. А если ситуация повторится в каком-нибудь новом их релизе?
Пойми правильно, у нас идея монохрома. Да иконка не является actions, но у нас такой стиль и что бы его использовать, то приходится делать по своему. НО, если бы они изначально использовали нормально имя приложения, то подобных косяков не было бы

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

Кароче добавлю симлинк для 32 и выше как исключение.

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

status нет 64px, потому добавил в apps... один хрен она исключение и одна

@SmartFinn
Copy link
Member

As result of the discussion, we added workaround for sizes 32px and greater, but we still think that using a status icon as an app icon is a horrible idea.

@jbicha
Copy link

jbicha commented Nov 1, 2017

@SmartFinn what icon name do you suggest instead?

I don't think we can fix this for GNOME 3.26 because it's frozen but 3.28 development is open.

By the way, the icon was added here

@SmartFinn
Copy link
Member

@jbicha as @varlesh wrote "gnome-settings-default-applications" or "preferences-desktop-default-applications" is good choice, these icons already exists in most popular icon themes:

$ ffind --nosymlinks /usr/share/icons/ -- 'gnome-settings-default-applications'
/usr/share/icons/Humanity/apps/48/gnome-settings-default-applications.svg
/usr/share/icons/Numix-Circle/48/apps/gnome-settings-default-applications.svg
/usr/share/icons/Paper/48x48/categories/gnome-settings-default-applications.png
/usr/share/icons/Papirus/48x48/apps/gnome-settings-default-applications.svg
$ ffind --nosymlinks /usr/share/icons/ -- 'preferences-desktop-default-applications'
/usr/share/icons/Humanity/apps/48/preferences-desktop-default-applications.svg
/usr/share/icons/Numix-Circle/48/apps/preferences-desktop-default-applications.svg
/usr/share/icons/Paper/48x48/categories/preferences-desktop-default-applications.png
/usr/share/icons/Papirus/48x48/apps/preferences-desktop-default-applications.svg

@jbicha
Copy link

jbicha commented Nov 1, 2017

Your "most popular icon themes" list excludes GNOME's default icon theme Adwaita.

@SmartFinn
Copy link
Member

@jbicha sure, Adwaita doesn't contain this icon, so it should be added instead of using an icon from the wrong context.

@jbicha
Copy link

jbicha commented Nov 1, 2017

Could someone file a bug against adwaita-icon-theme then?

@varlesh
Copy link
Contributor

varlesh commented Nov 1, 2017

https://bugzilla.gnome.org/show_bug.cgi?id=789775

@SmartFinn
Copy link
Member

@varlesh preferences-desktop-default-applications-symbolic is also needed, see #857 (comment)

@SmartFinn
Copy link
Member

The issue is outdated. The newer version of GNOME doesn't show Default Applications in the Dash because gnome-default-apps-panel.desktop has enabled NoDisplay=true option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants