Konstantin,

On 11/10/14 4:54 AM, Konstantin Kolinko wrote:
> 2014-11-10 6:51 GMT+03:00 Christopher Schultz <ch...@christopherschultz.net>:
>> Konstantin,
>>
>> On 11/9/14 10:12 AM, kkoli...@apache.org wrote:
>>> Author: kkolinko
>>> Date: Sun Nov  9 15:12:29 2014
>>> New Revision: 1637684
>>>
>>> URL: http://svn.apache.org/r1637684
>>> Log:
>>> Minor corrections:
>>> - Remove '\' at end-of-line when wrapping long lists of command arguments. 
>>> Such character makes no sense on Windows ('^' is used there).
>>> I think that readers should be wise enough to unwrap the lines, and 
>>> unwrapping is easier when you do not have to delete stray '\' characters.
>>
>> It would probably be better to write the command on a single long line
>> and have the browser auto-wrap the code. That way, copy/paste will work
>> properly onto the command-line.
>>
>> I believe Konstantin Preißer recently changed some of the CSS classes to
>> allow word-wrapping in <code</code> elements... not sure about <source>.
> 
> r1371336  ?

I'm not sure that's it, but the changes Konstantin Preißer was making
included fixing things like (not) wrapping in source slippets.

> That solution involved a number of auxiliary tags (indent, outdent). Honestly.
> a) I do not understand that markup. How it works? It is not pretty.

The XSL converts all that to <pre> plus whitespace to produce the
indents. I agree, it's not a great solution. Better would be just
regular <pre> with word-wrapping enabled right there in the .xml file,
copied without modification into the resulting .html file.

> b) The only place where it survived [1], its rendering in the current
> Firefox 33 is odd, with lot of empty lines.
> 
> That sample text does not exist in Tomcat 8 anymore thanks to the new
> resources implementation.
> 
> [1] http://tomcat.apache.org/tomcat-7.0-doc/config/context.html#Virtual_webapp

I see that below "The configuration is:". It should be improved.

-chris

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to