修复 HTMLRenderer 无法正确处理嵌套列表的问题 #5078
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request fixes an issue where HTMLRenderer could not correctly handle nested lists. The changes introduce indentation tracking and modern switch expression syntax to improve code readability and maintainability.
Key Changes
- Added indentation level tracking with
indentationLevelfield to properly handle nested list rendering - Introduced
INDENTATIONconstant for consistent spacing (4 spaces) - Refactored switch statements to use modern Java switch expressions with arrow syntax
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| private void appendText(String text) { | ||
| if (indentationLevel > 0) { | ||
| text = text.replaceAll("\n", INDENTATION.repeat(indentationLevel) + "\n"); |
There was a problem hiding this comment.
The regex replacement pattern has the indentation in the wrong position. Currently, replaceAll("\n", INDENTATION.repeat(indentationLevel) + "\n") replaces newlines with indentation + \n, which places the indentation before the newline character. This means the indentation appears at the end of each line rather than at the beginning of the next line. The replacement should be "\n" + INDENTATION.repeat(indentationLevel) to properly indent the text that follows each newline.
| text = text.replaceAll("\n", INDENTATION.repeat(indentationLevel) + "\n"); | |
| text = text.replaceAll("\n", "\n" + INDENTATION.repeat(indentationLevel)); |
|
怎么关闭了？ |
2 participants
No description provided.