CSSBox / Bugs / #45 error in parsing line-height using font shorthand

Update:

String css = "p {font: 400 italic 40px/1 \"Times New Roman\";}";
String html = "<html><head><style>"
  + css
  + "</style></head><body><p>Hello</p></body></html> ";

InputStream is = new ByteArrayInputStream(html.getBytes());
DocumentSource docSrc = new StreamDocumentSource(is, new URL("http://localhost"), "text/html");
DOMSource domSrc = new DefaultDOMSource(docSrc);
Document doc = domSrc.parse();
Element pElem = doc.createElement("p");
StyleSheet styleSheet = CSSFactory.parse(css);
DirectAnalyzer directAnalyser = new DirectAnalyzer(styleSheet);
NodeData firstNodeData = directAnalyser.getElementStyle(pElem, null, "screen");
final NodeData secondNodeData = directAnalyser.getElementStyle(pElem, null, "screen");

System.out.println("First run: " + firstNodeData);
System.out.println("Second run: " + secondNodeData);

if the above snippet is run, the value of secondNodeData is null. Basically after the first call the value returned is always null.

And if we remove /1 (line-height) form the font declaration, everything works fine.

Anonymous

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

Chetan Kothari - 2014-09-26

This bug is fixed by PR

Last edit: Chetan Kothari 2014-09-26

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Chetan Kothari - 2014-09-26

Tested and verified.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Radek Burget - 2014-09-26

status: open --> accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Radek Burget - 2015-01-10

Closed in 1.19

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Radek Burget - 2015-01-10

status: accepted --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

error in parsing line-height using font shorthand

Pure Java HTML / CSS rendering engine

Group

Searches

Help

#45 error in parsing line-height using font shorthand

Discussion