p4ulypops / jquery-clean

Automatically exported from code.google.com/p/jquery-clean
0 stars 0 forks source link

Blank spaces before word removed on <u> tags #14

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Using this configuration:
$.htmlClean.defaults.allowedTags = ['div', 'ol', 'ul', 'li', 'em', 'strong', 
'u', 'br'];

htmlClean removes blank spaces before the start of an <ul> tag 

What is the expected output? What do you see instead?
The <u> tag and content should be kept and the spaces before / after the <u> 
tag should not be removed.

What version of the product are you using? On what operating system?

1.2.3 revision 51

Please provide any additional information below.

line 447 should contain the "ul" tag defined as inline (this solves the issue)

Original issue reported on code.google.com by stein...@gmail.com on 30 Apr 2012 at 10:33

GoogleCodeExporter commented 9 years ago
last line should say:

line 447 should contain the "u" tag defined as inline (this solves the issue)

instead of "ul"

Original comment by stein...@gmail.com on 30 Apr 2012 at 10:34

GoogleCodeExporter commented 9 years ago
hi steinkel, can you give me an example of the input for a test?

Original comment by antixsof...@gmail.com on 30 Apr 2012 at 12:07

GoogleCodeExporter commented 9 years ago
yes:

input: "test1 <u>test2</u> test3"

config:
  $.htmlClean.defaults.replace.push([['p'], 'div']);
  $.htmlClean.defaults.allowedTags = ['div', 'ol', 'ul', 'li', 'em', 'strong', 'u', 'br'];
  $.htmlClean.defaults.removeTags = [];

Original comment by stein...@gmail.com on 30 Apr 2012 at 1:22

GoogleCodeExporter commented 9 years ago
yes:

input: "test1 <u>test2</u> test3"

config:
  $.htmlClean.defaults.replace.push([['p'], 'div']);
  $.htmlClean.defaults.allowedTags = ['div', 'ol', 'ul', 'li', 'em',
'strong', 'u', 'br'];
  $.htmlClean.defaults.removeTags = [];

2012/4/30 <jquery-clean@googlecode.com>

Original comment by stein...@gmail.com on 30 Apr 2012 at 4:23

GoogleCodeExporter commented 9 years ago
thanks steinkel - thats done now
there is a test in the repo if you need to confirm 
jquery.htmlClean.test-issue14.htm

Original comment by antixsof...@gmail.com on 30 Apr 2012 at 6:41

GoogleCodeExporter commented 9 years ago
cool, thank you.

Original comment by stein...@gmail.com on 30 Apr 2012 at 6:43