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

Child themes and distributor-logos #762

Closed
SmartFinn opened this issue Sep 23, 2017 · 9 comments
Closed

Child themes and distributor-logos #762

SmartFinn opened this issue Sep 23, 2017 · 9 comments

Comments

@SmartFinn
Copy link
Member

At the moment 16x16/places/distributor-logo-* and 16x16/places/start-here-* are missing in child themes.

There are three ways to fix this:

  1. Copy missing icons to those themes
  2. Move all distributor-logo-*, start-here-* and related icons to apps
  3. Create a separate directory for those icons and add it to index.theme under Places context.
@SmartFinn SmartFinn added the bug label Sep 23, 2017
@varlesh
Copy link
Contributor

varlesh commented Sep 23, 2017

Эти логотипы обычно в старт-меню юзаются их надо в монохром перекрасить для 16px.

@SmartFinn
Copy link
Member Author

Это не решит проблему с distributor-logo-* они должны быть цветные. Кроме того мы используем их как иконки для програм, в частности fedora-release-notes.svg.

Есть объективная причина держать их в places?

@varlesh
Copy link
Contributor

varlesh commented Sep 23, 2017

Ну давай в apps переместим тогда. В гноме один фиг юзается symbolic а крыса и матэ наверн 22px или там размера панели зависит. whisker-menu не цепляет значок.

@varlesh
Copy link
Contributor

varlesh commented Sep 23, 2017

Изначально я размещал их в places, так как хотел что бы старт-меню значок был монохромным. Но оно не работает один хрен.

@SmartFinn
Copy link
Member Author

Все те иконки можно разделить на две группы:

  • distributor-logo-*
  • desktop-environment-*

Предлагаю переименовать по этому принципу и перестроть симлинки в apps и для start-here-*

@varlesh
Copy link
Contributor

varlesh commented Sep 23, 2017

Согласен, так логичней

@SmartFinn
Copy link
Member Author

Ок, тогда я займусь пененосом и дедудликацией иконок в apps.

@SmartFinn
Copy link
Member Author

@varlesh можешь папки добавить если хочешь, я делаю в отдельном бранче т.ч. мы не конфликтуем.

@varlesh
Copy link
Contributor

varlesh commented Sep 23, 2017

ok

SmartFinn added a commit that referenced this issue Sep 23, 2017
SmartFinn added a commit that referenced this issue Sep 23, 2017
* renamed some apps icons to distro/DE logos and replace them to symlinks
SmartFinn added a commit that referenced this issue Sep 23, 2017
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

2 participants