error parsing property without value

Pure Java HTML / CSS rendering engine

Status: Beta

Brought to you by: radkovo

#47 error parsing property without value

Milestone: Any_version_(please_provide_version_in_description)

Status: closed

Owner: nobody

Labels: parsing (2)

Priority: 1

Updated: 2015-01-10

Created: 2014-09-10

Creator: Chetan Kothari

Private: No

the entire rule is discarded if there is some property that is not recognized in the css standards.
eg.
consider just null; without any value then the entire rule block is discarded by the parser.

Discussion

mercury - 2014-09-17

(I can't find a way to watch or star the issues on SF. Hence adding a comment and hoping to get notifications.)

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

I am currently working on this. The error recovery is quite complex and fixing this issue breaks other tests. I will do a complete revision.

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-29

I have made several revisions; the issue is hopefully fixed by this commit. I hope I haven't broken some other part (but the tests are passing well).

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-29

Thanks Radek, the problem that we were facing has been solved. Tested it with the test case that was failing. I had forgotten to add a test case in the issue. It is as follows.

<html> <head> <style> p { null; color: red; } </style> </head> <body> <p>red</p> <p style="null;color:green;">green</p> </body> </html>
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-29

The test case that I added in the previous comment, was the one which was failing, and after the fix it works as expected.

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-30

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

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.

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.

Anonymous