ymaheshwari1 edited a comment on pull request #89: URL: https://github.com/apache/roller/pull/89#issuecomment-850462212
> > Hi @ymaheshwari1 @mbien, > > For the current case, I think we can consider removing the HTML as there are security concerns enabling the HTML mode. > > WDYT? > > yeah sure, there are not that many anyway. I was checking if we could maybe enable some selectively for strings which are known to be static. But yeah lets just remove them. > > are those properties also tooltips or are they used somwhere else? > https://github.com/apache/roller/blob/743f1d246deab7d90221cc864093f00c4f672ecd/app/src/main/resources/ApplicationResources.properties#L1474-L1476 > > > since they contain links. Hi, These are displayed when the `authMethod != 'LDAP'`. Also, they are not tooltips. https://github.com/apache/roller/blob/master/app/src/main/webapp/WEB-INF/jsps/core/Register.jsp#L73 But, as they are used in `s:text`, it only considers string in it. https://struts.apache.org/tag-developers/text-tag.html -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
