CSSBox / Bugs / #50 adjacent selector matches text node

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

Yes, I get notifications about all the tickets. Thanks for the reports. I already started to solve some of them but it may take some time before I find the source and posssible solution.

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.
- mercury - 2014-09-17
  
  Radek, thanks a lot for creating and maintaining this project. The previous comment was so that I can get notifications :) I hope you don't mind the noise.
  
  On a related note, do you think GitHub can be a better option for hosting the project issues? I notice that you have already hosted the code on GitHub, but haven't enabled issue tracking. Github has support for watching issues noiselessly.
  
  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-17

Test Case:
CSS

p {background: red;}
div+p {background: green;}

HTML

<body>
<div>This text should be green.</div>
<p id="id1">This text should be green. (sibling)</p>
</body>

Expected style for p element

background: green

Actual style applied to p element

background: red

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.
- mercury - 2014-09-17
  
  Btw, the BoxBrowser works fine with that test case, probably because the HTML parser within it doesn't append text nodes between div and p elements.
  
  However, the parser in our project adds the additional text-nodes and hence the test fails.
  
  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-19
    
    Sorry forgot to mention that.
    @mercury: thanks :)
    
    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-10-01

Are you using a DirectAnalyzer or Analyzer (with maps)? Or are both of them affected?

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-10-01

I am using DirectAnalyzer. Did not try it with Analyzer (with maps)

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-10-01

Ok, DirectAnalyzer should be fixed now. Analyzer seems not to be affected.

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-10-01

Will soon test all the fixes and update at once.

Appreciate the quick response and fixes. Thanks a lot.

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-10-02

Verified.

Thanks for the fix.

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-10-02

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.

adjacent selector matches text node

Pure Java HTML / CSS rendering engine

Group

Searches

Help

#50 adjacent selector matches text node

Discussion