-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12472/#review23030
-----------------------------------------------------------


Andrew I prefer the website to work on relative domain/urls and not hardcode on 
having images on the root /

For example running the website from here for debugging and development:
http://localhost/~csantana23/cordova-website/
or
http://csantanapr.github.io/cordova-blog/

Here are two alternatives:
1. use ../images/
like background: url(../images/sprite.png)  no-repeat

2. use jekyll {{site.base.url}}
copy public/css/master.css to www/css/master.css
use template
like background: url({{ site.baseurl }}/images/sprite.png)  no-repeat






- Carlos Santana


On July 11, 2013, 9:52 p.m., Andrew Grieve wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12472/
> -----------------------------------------------------------
> 
> (Updated July 11, 2013, 9:52 p.m.)
> 
> 
> Review request for cordova, Carlos Santana and Michael Brooks.
> 
> 
> Bugs: CB-3997
>     https://issues.apache.org/jira/browse/CB-3997
> 
> 
> Repository: cordova-site
> 
> 
> Description
> -------
> 
> Convert our site to use Jekyll instead of mustache.
> - Adds a blog section with a first post.
> - Adds a favicon
> - Tweaks Rakefile & friends
> 
> This is a combination of Carlos' and my changes.
> 
> Changes since the version on https://github.com/csantanapr/cordova-blog
> - Change the first blog post to be much shorter so that it's entirely 
> uncontroversial.
> - Added docs/, wiki-images, etc. to the keep_files setting
> - Set the basepath within _config.yml and removed it from scripts. This 
> prevents paths from being different depending on running rake serve vs rake 
> build.
> - Tweaked README.md to say that some files are in public/ while others are in 
> www/
> - Restored search box script that was accidentally removed
> - Fixed an unclosed div on the main page (that I messed up in my pull request 
> to Carlos)
> 
> 
> Diffs
> -----
> 
>   /Gemfile 1502264 
>   /README.md 1502264 
>   /Rakefile 1502264 
>   /_config.yml PRE-CREATION 
>   /bin/gen.rb 1501850 
>   /bin/serve.rb PRE-CREATION 
>   /public/artwork.html 1502289 
>   /public/blog/2013/07/11/cordova-has-a-blog.html PRE-CREATION 
>   /public/blog/index.html PRE-CREATION 
>   /public/css/main.css PRE-CREATION 
>   /public/css/master.css PRE-CREATION 
>   /public/css/syntax.css PRE-CREATION 
>   /public/index.html 1502289 
>   /public/master.css 1502264 
>   /public/rss.xml PRE-CREATION 
>   /www/_includes/footer.html PRE-CREATION 
>   /www/_includes/head.html PRE-CREATION 
>   /www/_includes/header.html PRE-CREATION 
>   /www/_includes/links.html PRE-CREATION 
>   /www/_layouts/cordova.html PRE-CREATION 
>   /www/_layouts/default.html PRE-CREATION 
>   /www/_layouts/post.html PRE-CREATION 
>   /www/_posts/2013-07-08-cordova-has-a-blog.md PRE-CREATION 
>   /www/artwork.html 1502294 
>   /www/blog/index.html PRE-CREATION 
>   /www/css/_master.less PRE-CREATION 
>   /www/css/main.css PRE-CREATION 
>   /www/css/syntax.css PRE-CREATION 
>   /www/docs/README.md 1502264 
>   /www/downloads/BlueZedEx.mp3 UNKNOWN 
>   /www/downloads/icons_3.zip UNKNOWN 
>   /www/downloads/logos_2.zip UNKNOWN 
>   /www/downloads/screens_3.zip UNKNOWN 
>   /www/images/cordova_128.png UNKNOWN 
>   /www/images/cordova_16.png UNKNOWN 
>   /www/images/cordova_24.png UNKNOWN 
>   /www/images/cordova_256.png UNKNOWN 
>   /www/images/cordova_32.png UNKNOWN 
>   /www/images/cordova_48.png UNKNOWN 
>   /www/images/cordova_512.png UNKNOWN 
>   /www/images/cordova_64.png UNKNOWN 
>   /www/images/cordova_bot.jpg UNKNOWN 
>   /www/images/cordova_bot.png UNKNOWN 
>   /www/images/grid.png UNKNOWN 
>   /www/images/logo_full.png UNKNOWN 
>   /www/images/shadow.png UNKNOWN 
>   /www/images/splash.png UNKNOWN 
>   /www/images/sprite.png UNKNOWN 
>   /www/index.html PRE-CREATION 
>   /www/js/less-1.1.5.min.js 1502264 
>   /www/master.less 1502264 
>   /www/rss.xml PRE-CREATION 
>   /www/template.html 1502264 
> 
> Diff: https://reviews.apache.org/r/12472/diff/
> 
> 
> Testing
> -------
> 
> Ensured that when you type "rake build", that "svn status" reports no changes.
> 
> I also tried to manually compare the site to the existing one and tested most 
> links.
> 
> 
> Thanks,
> 
> Andrew Grieve
> 
>

Reply via email to