ThemeAwesome / WP-Forge

WP-Forge - A Foundation for Sites WordPress Theme
https://themeawesome.com/wp-forge-wordpress-foundation-for-sites-theme/
85 stars 28 forks source link

Allow URLs in customizer logo and background. #9

Closed baringji closed 10 years ago

baringji commented 10 years ago

Hi tsquez,

The current theme customizer allows us to set logo and background images. Would it be possible if we can put directly the URLs of the images we wanted for the Logo and for the background?

I got the following scenarios

First: I already uploaded the image from different directory of my theme. I wanted to use it directly without uploading the image. The current theme wants me to select a file always. Which might force me to upload again the file.

Second: I have a dev site where I uploaded the images. The images were committed to git and pulled to the live site. The dev and live sites doesn't have the same record of the file uploads in the database. Thus forcing me to re-upload the same images.

I used your theme for some of my projects. And they're awesome. :) Clients like it. Hehehe...

tsquez commented 10 years ago

Hi baringji,

I don't believe that you can use URLs directly for the Logo and the background. I did notice that when you look at the actual Twenty Twelve themes customizer the header and background portion act differently. I believe these are the default actions of the customizer and I believe WP-Forge customizer acts differently because of the way I built the customizer. Which by the way has been nagging me for some time.

I have actually been planning on redoing the customizer back to default but haven't gotten around to it yet. I believe this will make it into the next major release of WP-Forge. So look for that as well as hopefully a few new things :smile:

baringji commented 10 years ago

No worries. It's just a personal preference. But would be a good feature in the future. I'll keep posted on that.

Thanks

tsquez commented 10 years ago

Awesome! Coming your way soon (hopefully next major release)