Skip to content

Commit 2f98eed

Browse files
committed
Remove outdated TODOs
The one in CssContentPropertyResolver seems to be already resolved, since we do float children The one in SpanTagCssApplier could have been done, but the code application seems to be quite dirty either in a css applier or in a node DEVSIX-6522
1 parent 9c95a69 commit 2f98eed

File tree

2 files changed

+0
-2
lines changed

2 files changed

+0
-2
lines changed

src/main/java/com/itextpdf/html2pdf/css/apply/impl/SpanTagCssApplier.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,6 @@ public void apply(ProcessorContext context, IStylesContainer stylesContainer, IT
9595
}
9696
}
9797

98-
// TODO as for now spans are flattened, let's at least make kids of floating spans floating too
9998
String floatVal = cssStyles.get(CssConstants.FLOAT);
10099
if (floatVal != null && !CssConstants.NONE.equals(floatVal)) {
101100
for (IPropertyContainer elem : spanTagWorker.getAllElements()) {

src/main/java/com/itextpdf/html2pdf/css/resolve/CssContentPropertyResolver.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,6 @@ static List<INode> resolveContent(Map<String, String> styles, INode contentConta
222222
} else if (token.getValue().startsWith("url(")) {
223223
Map<String, String> attributes = new HashMap<>();
224224
attributes.put(AttributeConstants.SRC, CssUtils.extractUrl(token.getValue()));
225-
//TODO: probably should add user agent styles on CssContentElementNode creation, not here.
226225
attributes.put(AttributeConstants.STYLE, CssConstants.DISPLAY + ":" + CssConstants.INLINE_BLOCK);
227226
result.add(new CssContentElementNode(contentContainer, TagConstants.IMG, attributes));
228227
} else if (CssGradientUtil.isCssLinearGradientValue(token.getValue())) {

0 commit comments

Comments
 (0)