Re: [PR] [DOXIA-734] Restore handling of XHTML 1.0 obsolete attributes [maven-doxia]

2024-03-19 Thread via GitHub
kwin merged PR #204: URL: https://github.com/apache/maven-doxia/pull/204 -- 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: issues-unsubscr...@maven.apache

Re: [PR] [DOXIA-734] Restore handling of XHTML 1.0 obsolete attributes [maven-doxia]

2024-03-15 Thread via GitHub
kwin commented on PR #204: URL: https://github.com/apache/maven-doxia/pull/204#issuecomment-1999355773 > Does it make sense to log a warning to tell the user that the attribute should be migrated? I don't think, because the user is complying with the spec while using XHTML5 only attr

Re: [PR] [DOXIA-734] Restore handling of XHTML 1.0 obsolete attributes [maven-doxia]

2024-03-14 Thread via GitHub
kwin commented on code in PR #204: URL: https://github.com/apache/maven-doxia/pull/204#discussion_r1525424113 ## doxia-core/src/main/java/org/apache/maven/doxia/parser/Xhtml1BaseParser.java: ## @@ -0,0 +1,159 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [DOXIA-734] Restore handling of XHTML 1.0 obsolete attributes [maven-doxia]

2024-03-14 Thread via GitHub
michael-o commented on code in PR #204: URL: https://github.com/apache/maven-doxia/pull/204#discussion_r1525401870 ## doxia-core/src/main/java/org/apache/maven/doxia/parser/Xhtml1BaseParser.java: ## @@ -0,0 +1,159 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] [DOXIA-734] Restore handling of XHTML 1.0 obsolete attributes [maven-doxia]

2024-03-14 Thread via GitHub
michael-o commented on PR #204: URL: https://github.com/apache/maven-doxia/pull/204#issuecomment-1998102060 I see, it will map on the fly from XHML1.0 to XHTML5. Not bad at all and something I can clearly live with. Question: Does it make sense to log a warning to tell the user that the