jeromegn / DocumentUp

Pretty documentation generator for Github projects with proper Readme.
http://documentup.com
884 stars 90 forks source link

Change path to stylesheet to // for https access #75

Closed lorddev closed 10 years ago

lorddev commented 10 years ago

Enable stylesheet from SSL repository blobs by replacing http:// with //.

Current:

<link rel="stylesheet" type="text/css" href="http://documentup.com/stylesheets/themes/v1.css">

Change to:

<link rel="stylesheet" type="text/css" href="//documentup.com/stylesheets/themes/v1.css">
jeromegn commented 10 years ago

Feel free to create a pull request for this.

lorddev commented 10 years ago

Ok!

Sent from my iPhone

On Dec 9, 2013, at 11:46 AM, Jerome Gravel-Niquet notifications@github.com wrote:

Feel free to create a pull request for this.

— Reply to this email directly or view it on GitHub.

lorddev commented 10 years ago

Status update, please?

samhashemi commented 10 years ago

:+1: This is a big issues for folks using HTTPS Everywhere.

E.g. see this bug: NYTimes/backbone.stickit#240

jeromegn commented 10 years ago

Fixed. Sorry about the HUGE delay for such a simple issue.

I suggest forcing a re-render if you want to benefit from that. http://documentup.com/#hosted/manual-recompile

lorddev commented 10 years ago

Did you use my pull request or do it yourself?

jeromegn commented 10 years ago

I had been so out of touch with this project I didn't remember the PR and so I just did it myself. My mistake, sorry about that.

lorddev commented 10 years ago

:'-(— Sent from Mailbox

On Thu, May 29, 2014 at 9:41 AM, Jerome Gravel-Niquet notifications@github.com wrote:

I had been so out of touch with this project I didn't remember the PR and so I just did it myself. My mistake, sorry about that.

Reply to this email directly or view it on GitHub: https://github.com/jeromegn/DocumentUp/issues/75#issuecomment-44553711