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

Xfce symlinks #5624

Merged
merged 8 commits into from
Apr 14, 2021
Merged

Xfce symlinks #5624

merged 8 commits into from
Apr 14, 2021

Conversation

palob
Copy link
Member

@palob palob commented Nov 10, 2020

Fixes #5623

@palob palob added the merge label Nov 10, 2020
@palob
Copy link
Member Author

palob commented Dec 20, 2020

There are a few more org.xfce.panel.[something] icon names but I'm not sure yet whether or not those are supposed to be app icons.

org.xfce.panel.actions
org.xfce.panel.applicationsmenu
org.xfce.panel.clock
org.xfce.panel.directorymenu
org.xfce.panel.launcher
org.xfce.panel.pager
org.xfce.panel.separator
org.xfce.panel.showdesktop
org.xfce.panel.statustray
org.xfce.panel.tasklist
org.xfce.panel.windowmenu

@meganukebmp
Copy link
Contributor

Very much needed commit. Here's some of the icons that appear in the icon picker under the org.xfce namespace.
image

@palob
Copy link
Member Author

palob commented Feb 3, 2021

Could you please tell me where the org.xfce.panel.* icons are shown? Are those app icons or panel icons?

@meganukebmp
Copy link
Contributor

Could you please tell me where the org.xfce.panel.* icons are shown? Are those app icons or panel icons?

Those seem to be panel/applet icons, yes.

@palob
Copy link
Member Author

palob commented Feb 3, 2021

Thank you, this means they should not be added here but in Numix base.

@fish47
Copy link

fish47 commented Feb 15, 2021

vinceliuice/Qogir-icon-theme#46

some icon theme repos have adapted to xfce4 icons renaming.

https://wiki.xfce.org/releng/4.16/icons_rdns_naming_for_icons

maybe the full list of renaming?

@palob
Copy link
Member Author

palob commented Feb 15, 2021

Thank you for the hint to the Xfce wiki, will check this.

@meganukebmp
Copy link
Contributor

Could you please tell me where the org.xfce.panel.* icons are shown? Are those app icons or panel icons?

I'm very sorry, seems I have misinformed you. The icons appear in the panel applet list (tho some also seem to appear in the panel itself). Examples attached

image

Here's the "show desktop" icon also being repeated in the panel itself.
image

@palob
Copy link
Member Author

palob commented Feb 21, 2021

I think if the icons are shown inside the panel they should not be designed as app (Circle/Square) icons. Hopefully this won't lead to a mix of styles (Numix base icons and Circle/Square icons) in the panel applet list.

@meganukebmp
Copy link
Contributor

I think if the icons are shown inside the panel they should not be designed as app (Circle/Square) icons. Hopefully this won't lead to a mix of styles (Numix base icons and Circle/Square icons) in the panel applet list.

Ultimately this is something to discuss with the applet and xfce panel developers. Not much we can do here other than make a decision. Some applets share an icon with their configuration utility (which has a styled icon) and display that icon in the panel also. Some of the icons in the applet list screenshot above are also used inside the application in some "header" type element, so styling it would make sense.

In my opinion having those be styled is fine, as developers are discouraged to be using app icons in the panel unless it's a launcher or something else. We cant solve for every bad implementation.

@fish47 linked an official list of renamed icons:

some icon theme repos have adapted to xfce4 icons renaming.

https://wiki.xfce.org/releng/4.16/icons_rdns_naming_for_icons

I think this should be the first (and possibly immediate) course of action for the time being as it's guaranteed to return us back to the same functionality as prior to the rename. Any other fixes and changes should be their own unrelated changes.

@palob
Copy link
Member Author

palob commented Feb 26, 2021

I'd say let's just merge this and compare with https://wiki.xfce.org/releng/4.16/icons_rdns_naming_for_icons afterwards.
There's probably going to be duplicate allocations, need to boot into a Xfce session with this merged first.

@Foggalong Foggalong merged commit f4a23b3 into master Apr 14, 2021
@Foggalong Foggalong deleted the xfce branch April 14, 2021 04:11
@palob
Copy link
Member Author

palob commented Feb 6, 2022

Papirus added some more symlinks PapirusDevelopmentTeam/papirus-icon-theme@99dd1cb?short_path=f25d47c#diff-c230992aebc23f8ba79d75ac5dd0b5e8b6d9b4bf2f62b2fcfb361fef51437a40

I'm not sure if those are app icons.
I'm taking the liberty of pinging you, @meganukebmp @marcosins @fish47 @bgstack15.

If you are still Numix and Xfce users can you please help me out as to where those icons are shown?

@bgstack15
Copy link
Contributor

Gigolo is some sort of app in Xfce. I must not be on the newest version of Gigolo or don't have that app installed.

CPUGraph is the panel plugin icon and same for the weather. I haven't found specifically how to view the Xfce4 fsguard plugin icon, but it's some plugin that checks mount points disk space.

@palob
Copy link
Member Author

palob commented Feb 6, 2022

Thanks. Panel plugin means it's not something you start like an app via launcher and the icon is only shown in the panel?
fsguard for instance has no .desktop file it appears.

@bgstack15
Copy link
Contributor

A panel plugin is for displaying an icon/menu/item in the panel. A plugin usually does some small function of Xfce but can be its own separate binary. There's definitely no specific need for a separate .desktop file that would use the icon. The icon could be used only internally in xfce for the panel.

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

Successfully merging this pull request may close these issues.

New xfce4-screensaver icon name.
5 participants