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

Add symbolic links for the Plank dock 'show desktop' button #2706

Merged
merged 8 commits into from
Jun 5, 2021

Conversation

gitadum
Copy link
Contributor

@gitadum gitadum commented Jun 5, 2021

Plank dock has a button to show the desktop, whose icon name is 'show-desktop'
Adds symlinks to make this button adopt Papirus theme. Here is a before/after:
plank-dock-papirus-before
plank-dock-papirus-after
Works at the very least on Ubuntu MATE 20.04.

@SmartFinn
Copy link
Member

SmartFinn commented Jun 5, 2021

Places icon isn't the right choice for this button because of monochrome 16px icon. Please use apps icons for that:

rm Papirus/{16x16,22x22,24x24,32x32,48x48,64x64}/places/show-desktop.svg
cd tools/work
./new-symlink.sh apps cs-desktop.svg show-desktop
./put-into-theme.sh

Thanks!

Also changed symlinks targets from /places/user-desktop to /apps/cs-desktop
@SmartFinn
Copy link
Member

SmartFinn commented Jun 5, 2021

You try to add text files with cs-desktop.svg in content instead of symlinks. Please use commands above to create the symlinks.

@gitadum
Copy link
Contributor Author

gitadum commented Jun 5, 2021

You try to add text files with cs-desktop.svg in content instead of symlinks. Please you commands above to create the symlinks.

Sorry about that. I was trying doing it directly from GitHub web interface because I did not install the contributor's /tool/work set... I will just download the scripts now

@gitadum
Copy link
Contributor Author

gitadum commented Jun 5, 2021

Cloned my fork repository and made some real symlinks now with the scripts.

@SmartFinn SmartFinn merged commit e2b1852 into PapirusDevelopmentTeam:master Jun 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants