typeplate/typeplate.github.io

LESS Codeblock Output Missing 'white-space: -moz-pre-wrap'

Closed this issue · 2 comments

Calling Mr. @lukebarton to the stand. So here is the trouble with the compiling of the code blocks for the LESS port at the moment. I should also mention there were a ton of compiling errors in your original version(1.0.0) that I went ahead and fixed. In the future make sure to run it through something like Codepen just to be doubly sure and we will make sure to do the same on our end. Thanks! Aces. Now on with the show.

LESS

// $C o d e b l o c k s
// --------------------------------------------------------------------------

.white-space (@wrap-space) when (@wrap-space = 'pre-wrap') {
    white-space: "-moz-@{wrap-space}";  // Firefox 1.0-2.0
    white-space: ~"@{wrap-space}";      // current browsers
}
.white-space (@wrap-space) when not (@wrap-space = 'pre-wrap') {
    white-space: ~"@{wrap-space}";
}

pre code {
    .normal-wrap;
    .white-space(pre-wrap);
}

pre {
    .white-space(pre);
}

code {
    .white-space(pre);
    font-family: monospace;
}

Compiled LESS

 pre code {
  word-wrap: normal;
  white-space: pre-wrap;
}
pre {
  white-space: pre;
}
code {
  white-space: pre;
  font-family: monospace;
}

The Required Typeplate Output

pre code {
    white-space: -moz-pre-wrap;
    white-space: pre-wrap;
}

pre {
    white-space: pre;
}

code {
    white-space: pre;
    font-family: monospace;
}

I'm surprised because I spent a good 20 minutes comparing the output in a
diff tool. There were a couple of selectors which had moved though that
were eye checks, I guess this was one. I plead innocent to there being a
'ton of compiling errors' but I'll plead guilty to there being a handful of
inaccuracies post-first round conversion :)

Unfortunately, I won't be able to look at it until tomorrow or maybe Friday
unless I can steal some time on the train this morning or at lunchtime in
the next few days. In fact, I'll try by best to do that.
On 5 Mar 2013 03:08, "Dennis Gaebel" notifications@github.com wrote:

Calling Mr. @lukebarton https://github.com/lukebarton to the stand. So
here is the trouble with the compiling of the code blocks for the LESS port
at the moment. I should also mention there were a ton of compiling errors
in your original version that I went ahead and fixed. In the future make
sure to run it through something like Codepen just to be doubly sure and we
will make sure to do the same on our end. Thanks! Aces. Now on with the
show.

LESS

// $C o d e b l o c k s
// --------------------------------------------------------------------------

.white-space (@wrap-space) when (@wrap-space = 'pre-wrap') {
white-space: "-moz-@{wrap-space}"; // Firefox 1.0-2.0
white-space: ~"@{wrap-space}"; // current browsers
}
.white-space (@wrap-space) when not (@wrap-space = 'pre-wrap') {
white-space: ~"@{wrap-space}";
}

pre code {
.normal-wrap;
.white-space(pre-wrap);
}

pre {
.white-space(pre);
}

code {
.white-space(pre);
font-family: monospace;
}

Compiled LESS

pre code {
word-wrap: normal;
white-space: pre-wrap;
}
pre {
white-space: pre;
}
code {
white-space: pre;
font-family: monospace;
}

The Required Typeplate Output

pre code {
white-space: -moz-pre-wrap;
white-space: pre-wrap;
}

pre {
white-space: pre;
}

code {
white-space: pre;
font-family: monospace;
}


Reply to this email directly or view it on GitHubhttps://github.com//issues/74
.

@lukebarton
There were not a "ton" so maybe a bit dramatic so I apologize, but there were still a few to keep it from compiling. No worries though as I fixed them. We just have this little bit left from above to make it kosher. 👍