Menu

#20 CSSStyleDeclarationImpl#getCssText() return braces

closed
None
5
2012-07-18
2008-01-18
No

The CVS sourcecode today is:

public String getCssText() {
StringBuffer sb = new StringBuffer();
sb.append("{");
//if newlines requested in text
//sb.append("\n");
for (int i = 0; i < this.properties.size(); ++i) {
Property p = (Property) this.properties.elementAt(i);
if (p != null) {
sb.append(p.toString());
}
if (i < this.properties.size() - 1) {
sb.append("; ");
}
//if newlines requested in text
//sb.append("\n");
}
sb.append("}");
return sb.toString();
}

The spec says:

/*
* The parsable textual representation of the declaration block
* (excluding the surrounding curly braces). Setting this attribute will
* result in the parsing of the new value and resetting of all the
* properties in the declaration block including the removal or addition
* of properties.
/
public String getCssText();

So the braces are to much.

Discussion

  • Johannes Koch

    Johannes Koch - 2008-03-13

    Logged In: YES
    user_id=663593
    Originator: NO

    I fixed this issue. See CVS.

     
  • Daniel Gredler

    Daniel Gredler - 2008-03-25

    Logged In: YES
    user_id=1109422
    Originator: NO

    Thanks for the warning; fixed in CVS and will be included in the upcoming release.

     
  • Nobody/Anonymous

    o4u2xm 7135357 3sfyt3 [URL=http://www.kuwaitism.com/wire/lieben/] lieben [/URL] lieben http://www.kuwaitism.com/wire/lieben/

     

Anonymous
Anonymous

Add attachments
Cancel