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

bg.png is a symbolic link to your own folder. #3

Closed
rouzbeh opened this issue Nov 5, 2014 · 3 comments
Closed

bg.png is a symbolic link to your own folder. #3

rouzbeh opened this issue Nov 5, 2014 · 3 comments

Comments

@rouzbeh
Copy link

rouzbeh commented Nov 5, 2014

"bg.png" in the default theme folder points to /home/setkeh/.config.... . This prevents the file from being found if the username is different, and causes an error. There should probably be a file instead of the symbolic link.

@setkeh
Copy link
Owner

setkeh commented Nov 7, 2014

This is the way it should be as i dont include the wallpapers directory for various reasons its set that way to show how the wallpaper changer works a bit more visually then some flat lua.
to get rid of the error mkdir the wallpapers directory and add you wall paper images and rm the symbolic link and use the wallpaper changer to recreate the link to a file that exists in your awesome config directory.
Please also keep in mind this is supposed to be an example config it has allot of awesome features enabled by default it was never really designed to be a drop in and go with no modification configuration but a good example of how the various libs and api's work and how to create your own widgets whilst still being a working config with some user intervention.
Thanks for the Feedback :)

@rouzbeh
Copy link
Author

rouzbeh commented Nov 10, 2014

Hey

No worries. It's just that this single thing prevents the whole config from being a very nice drop-in configuration set. By having the default wallpaper as a file, people indeed only need to clone the repository and they are good to go. Right now, they get a cryptic message, and nothing works.

If you don't want to change this, you may want to document this in the readme.md.

Cheers

@setkeh
Copy link
Owner

setkeh commented Nov 13, 2014

Ill play with a couple of ways of showing this Off better :) and add it in maybe a nice opensource image and include the wallpapers dir may fix it without taking away from the underlying principle :) thanks for the input mate much appreciated 👍

@setkeh setkeh closed this as completed Nov 13, 2014
setkeh added a commit that referenced this issue Jun 19, 2015
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

No branches or pull requests

2 participants