matthiasblaesing commented on code in PR #5641:
URL: https://github.com/apache/netbeans/pull/5641#discussion_r1133753615
##########
ide/css.editor/src/org/netbeans/modules/css/editor/module/spi/CssCompletionItem.java:
##########
@@ -609,6 +620,42 @@ public ElementKind getKind() {
public int getSortPrioOverride() {
return super.getSortPrioOverride() - (related ? 1 : 0);
}
+
+ /**
+ * Escape the input for usage in CSS files.
+ *
+ * @param input
+ * @return
+ */
+ private static String escape(String input) {
Review Comment:
I would agree if we would talk about a web application, but in this case the
usage is not security, but compatibility.
I saw one thing in the description of the CSS escaper of owasp, that looks
like a potential problem for my approach. I overlooked that the hexadecimal
encoding and decoding needs a bounding whitespace or can have it. I'll revisit
this and the CSS grammar.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
