patch: implement white-space: pre-wrap for vsource
Just a thought I had. The web is filled with horrible generated html with lines a thousand characters long. With this (and parse_embedded_css), those lines become more tolerable. I would commit it in two pieces, of course.
On Tue, Mar 30, 2010 at 06:00:26AM +0000, corvid wrote:
Just a thought I had. The web is filled with horrible generated html with lines a thousand characters long. With this (and parse_embedded_css), those lines become more tolerable.
Very nice!
I would commit it in two pieces, of course.
Please go ahead - at least for the CSS / dw part. I don't know whether it's ok to use it for source view. Cheers, Johannes
On Tue, Mar 30, 2010 at 09:57:08PM +0200, Johannes Hofmann wrote:
On Tue, Mar 30, 2010 at 06:00:26AM +0000, corvid wrote:
Just a thought I had. The web is filled with horrible generated html with lines a thousand characters long. With this (and parse_embedded_css), those lines become more tolerable.
Very nice!
I would commit it in two pieces, of course.
Please go ahead - at least for the CSS / dw part. I don't know whether it's ok to use it for source view.
In any case we could add an id like this: <body id="dillo_sv"> to the source view page, so people can adjust it in style.css, e.g: body#dillo_sv pre {white-space: pre-wrap}
Johannes wrote:
On Tue, Mar 30, 2010 at 06:00:26AM +0000, corvid wrote:
Just a thought I had. The web is filled with horrible generated html with lines a thousand characters long. With this (and parse_embedded_css), those lines become more tolerable.
Very nice!
I would commit it in two pieces, of course.
Please go ahead - at least for the CSS / dw part. I don't know whether it's ok to use it for source view.
I've read that it's in CSS 2.1 and 3, but not 1 or 2. 2 is a "recommendation", but 2.1 is a "candidate recommendation". Are we aiming more in the direction of 2 or 2.1 (or their intersection)?
On Tue, Mar 30, 2010 at 08:12:34PM +0000, corvid wrote:
Johannes wrote:
On Tue, Mar 30, 2010 at 06:00:26AM +0000, corvid wrote:
Just a thought I had. The web is filled with horrible generated html with lines a thousand characters long. With this (and parse_embedded_css), those lines become more tolerable.
Very nice!
I would commit it in two pieces, of course.
Please go ahead - at least for the CSS / dw part. I don't know whether it's ok to use it for source view.
I've read that it's in CSS 2.1 and 3, but not 1 or 2. 2 is a "recommendation", but 2.1 is a "candidate recommendation".
Are we aiming more in the direction of 2 or 2.1 (or their intersection)?
I think we should go with 2.1. 2.1 fixes some shortcomings of 2, and 3 is still experimental - as far as I know.
On Tue, Mar 30, 2010 at 09:57:08PM +0200, Johannes Hofmann wrote:
On Tue, Mar 30, 2010 at 06:00:26AM +0000, corvid wrote:
Just a thought I had. The web is filled with horrible generated html with lines a thousand characters long. With this (and parse_embedded_css), those lines become more tolerable.
Very nice!
+1 I really dislike when emails with paragraphs in one line get under PRE. If this fixes that, I'm all for it! :)
I would commit it in two pieces, of course.
Please go ahead - at least for the CSS / dw part. I don't know whether it's ok to use it for source view.
AFAIS, there's no problem for source view. -- Cheers Jorge.-
Jorge wrote:
On Tue, Mar 30, 2010 at 09:57:08PM +0200, Johannes Hofmann wrote:
On Tue, Mar 30, 2010 at 06:00:26AM +0000, corvid wrote:
Just a thought I had. The web is filled with horrible generated html with lines a thousand characters long. With this (and parse_embedded_css), those lines become more tolerable.
Very nice!
+1
I really dislike when emails with paragraphs in one line get under PRE. If this fixes that, I'm all for it! :)
I would commit it in two pieces, of course.
Please go ahead - at least for the CSS / dw part. I don't know whether it's ok to use it for source view.
AFAIS, there's no problem for source view.
Committed the CSS/dw part. For vsource, I have no opinion on whether to use my original solution or to add an id like Johannes mentioned.
On Wed, Mar 31, 2010 at 12:02:16AM +0000, corvid wrote:
Jorge wrote:
On Tue, Mar 30, 2010 at 09:57:08PM +0200, Johannes Hofmann wrote:
On Tue, Mar 30, 2010 at 06:00:26AM +0000, corvid wrote:
Just a thought I had. The web is filled with horrible generated html with lines a thousand characters long. With this (and parse_embedded_css), those lines become more tolerable.
Very nice!
+1
I really dislike when emails with paragraphs in one line get under PRE. If this fixes that, I'm all for it! :)
I would commit it in two pieces, of course.
Please go ahead - at least for the CSS / dw part. I don't know whether it's ok to use it for source view.
AFAIS, there's no problem for source view.
Committed the CSS/dw part.
For vsource, I have no opinion on whether to use my original solution or to add an id like Johannes mentioned.
As Jorge is also ok with it, I'd say make it the default for source view. Additionally I will add the id, for customization in style.css.
participants (3)
-
corvid＠lavabit.com -
jcid＠dillo.org -
Johannes.Hofmann＠gmx.de