Bug 26326 - CSS error: white space property ignored
Summary: CSS error: white space property ignored
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml renderer (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
: 55957 70484 73038 76675 82273 82583 82906 84280 94756 (view as bug list)
Depends on:
Blocks:
 
Reported: 2001-05-29 00:48 UTC by Jonathan Ingram
Modified: 2004-12-29 19:28 UTC (History)
14 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Ingram 2001-05-29 00:43:31 UTC
(*** This bug was imported into bugs.kde.org ***)

Package:           khtml
Version:           KDE 2.2.0 CVS/CVSup/Snapshot
Severity:          normal
Installed from:    Compiled sources
Compiler:          gcc version 2.96 20000731 (Linux-Mandrake 8.0 2.96-0.48mdk)   
OS:                Linux
OS/Compiler notes: Not Specified

http://www.w3.org/Style/CSS/Test/current/sec562.htm

No attempt is made to render the whitespace correctly.

(Submitted via bugs.kde.org)
Comment 1 Jim Dabell 2002-11-13 18:10:42 UTC
Still a problem in 3.1-rc3 
Comment 2 Lars Knoll 2003-01-06 20:09:19 UTC
We need to change the parser to not remove white space to fix this. 
Comment 3 Jim Dabell 2003-05-30 12:59:26 UTC
*** Bug 55957 has been marked as a duplicate of this bug. ***
Comment 4 Jim Dabell 2003-05-30 13:00:06 UTC
Still present in 3.1.2.
Comment 5 S Leiber 2003-06-08 18:33:31 UTC
This was not correctly rendert in KDE cvs-Version. 
Here is another side for testing that option for css2: 
http://www.meyerweb.com/eric/css/tests/css2/sec16-06.htm 
 
Sven 
 
Comment 6 Viktor Rosenfeld 2004-01-14 03:35:56 UTC
As of KDE 3.1.4:
- normal works
- nowrap works
- pre does not work

As to changing the parser: It works for the standard <pre>-tag, so the functionality is there?
Comment 7 Stephan Kulow 2004-01-14 17:56:33 UTC
*** Bug 70484 has been marked as a duplicate of this bug. ***
Comment 8 Stephan Kulow 2004-01-20 20:01:31 UTC
*** Bug 73038 has been marked as a duplicate of this bug. ***
Comment 9 Stephan Kulow 2004-03-03 21:13:35 UTC
*** Bug 76675 has been marked as a duplicate of this bug. ***
Comment 10 Haeber 2004-05-15 01:20:21 UTC
In KDE 3.2.2 there is still no support the "white-space:pre" css-attribut in normal elements. When will there be any channges?
I've seen a working functionality for this in Safari. Is it a problem to use this Safari-functionality in Konqueror?
Comment 11 Stephan Kulow 2004-05-15 10:53:52 UTC
obviously?
Comment 12 Stephan Kulow 2004-05-29 22:42:37 UTC
*** Bug 82273 has been marked as a duplicate of this bug. ***
Comment 13 Maksim Orlovich 2004-05-31 17:24:03 UTC
*** Bug 82583 has been marked as a duplicate of this bug. ***
Comment 14 Stephan Kulow 2004-06-22 19:30:24 UTC
*** Bug 82906 has been marked as a duplicate of this bug. ***
Comment 15 Maksim Orlovich 2004-06-30 21:31:04 UTC
*** Bug 84280 has been marked as a duplicate of this bug. ***
Comment 16 Stephan Kulow 2004-07-28 16:46:23 UTC
*** Bug 82906 has been marked as a duplicate of this bug. ***
Comment 17 Josh Berry 2004-08-19 19:40:56 UTC
This bug makes Konqueror very difficult to use when working with Gentoo's bugzilla.  I'll probably have to keep firefox around just for that site ...

Please fix this.  You'll make your users happy! ;)
Comment 18 Jay Luker 2004-08-19 20:46:26 UTC
Ahrg. 3.3 and still no white-space:pre support.
Comment 19 Brian "netdragon" Bober 2004-08-30 17:50:52 UTC
white-space: pre-wrap; is part of the CSS3 standard.
Comment 20 Jay Luker 2004-08-30 19:53:21 UTC
I'm mostly concerned with white-space: pre, but regardless, both "pre" and "pre-wrap" are contained in the CSS2 standard. See: http://www.w3.org/TR/CSS21/text.html#white-space-prop
Comment 21 Brian "netdragon" Bober 2004-08-31 00:04:55 UTC
At ths point, I believe web-develoepers will use the following and will ignore that Konqueror cannot handle it:

pre {
 white-space: -moz-pre-wrap;  /* Mozilla, supported since 1999 */
 white-space: -pre-wrap;      /* Opera 4 - 6 */
 white-space: -o-pre-wrap;    /* Opera 7 */
 white-space: pre-wrap;       /* CSS3 - Text module (Candidate Recommendation)
                                 http://www.w3.org/TR/css3-text/#white-space */
 word-wrap: break-word;       /* IE 5.5+ */
} 
Comment 22 Maksim Orlovich 2004-12-10 16:29:34 UTC
*** Bug 94756 has been marked as a duplicate of this bug. ***
Comment 23 Allan Sandfeld 2004-12-29 19:28:46 UTC
CVS commit by carewolf: 

Implement CSS 2.1 white-space: pre, pre-wrap and pre-line
FEATURE: 26326

Note: Compared to the patch send to kfm-devel and the WebCore implementation 
  this one handles linefeed cleaning in render_text.cpp. 
  This is much more efficient and simplifies code in bidi.cpp.


  M +14 -0     ChangeLog   1.361
  M +3 -1      css/cssparser.cpp   1.307
  M +6 -0      css/cssstyleselector.cpp   1.339
  M +120 -114  css/cssvalues.c   1.51
  M +61 -59    css/cssvalues.h   1.45
  M +9 -3      css/cssvalues.in   1.43
  M +20 -13    html/htmlparser.cpp   1.354
  M +32 -40    html/htmltokenizer.cpp   1.293
  M +25 -26    rendering/bidi.cpp   1.205
  M +1 -1      rendering/render_layer.h   1.24
  M +24 -6     rendering/render_style.h   1.114
  M +66 -7     rendering/render_text.cpp   1.260