-
Notifications
You must be signed in to change notification settings - Fork 13
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] Support for Xfce 4.16 rdns icons. #3
Comments
I will fix that as soon as I'm over with the other bug 😉 |
I tested it yesterday. Some of them are present. This is the list of the new icons created with the script, symlinking to the existent old Xfce icons (still Material/WhiteSur styled).
Will make a PR before the weekend. |
Sweet, thanks !! |
This is the current look of xfce4-settings-manager with current Nordzy icon theme. Added some context to the screenshot because some tools are not really from Xfce.
Context: Icons within a red square are not part of Xfce. They are 3rd party tools I just integrated into xfce4-settings-manager because I felt final users may like to find them within a single tool.
This is the new name schema for Xfce 4.16:
I have a 'shortcut method' to create the rdns icons. Saves some time and work. First I used the official Xfce list for rdns icons, then created a two column text file with this table data and saved it as ~/new-rdns-icons-xfce4-4.16.txt. Then used the following script, which basically looks for the old xfce4 icons within the src sub-folders, then tries to create a symlink for new rdns icon at the same location. The script only creates the symlink with relative path if the old xfce4 icon (png/svg) does exist and it does not overwrite any previously existent symlink (unless -fsr is used instead of -lr):
Simple, but effective. The catch: if there is no previous old Xfce icon, there will not be a new Xfce rdns icon.
There is also this python script https://gist.github.com/bluesabre/582d886a02c793285f4e3081df9b2a3c. Have not tested it, but basically does the same as my bash script, but with harder to read code.
I can make a PR if you want, but I think you may like to play a little with both alternatives to check and test the results.
The text was updated successfully, but these errors were encountered: