#1 review only,请不要提交】review-250521

باز‌کردن
dengjia قصد ادغام 2 تغییر را از yyc3/review-250521 به yyc3/review-250520 دارد
  1. unit test不要使用main
  2. 路径操作使用Paths和Path库,兼容Linux/Mac/Win
  3. css的内容写到css文件中,java代码里只需要按需挂载style,class,tag就可以,马上就会面临参数化的问题,在代码里hardcode没办法维护。
  4. removeEmptyParagraphs 如之前讨论的思路,尽量不用删除元素来控制layout,考虑用控制空行size的办法。
  5. processFirstImageAsBackground理论上不需要全部设置来特殊对待第一张图片。如果图片不按设计显示,那就调整不准确的显示属性,其他的图片也是一样的处理方式。
  6. @SuppressWarnings("checkstyle:NestedIfDepth") 拆分过于复杂的方法,不要suppressWarning。
  7. 考虑建立HtmlConverter,PdfConverter类,参数化hardcode的值,把过程性代码对象化。
  8. TOC的超链在pdf转换的过程中能正确生成吗?如果能那我就不去调查怎么在flyingsaucer处理html的过程中写定制插件了。因为如果要定制渲染过程中才能计算得到的数据,这里似乎是最合理的地方。
1. unit test不要使用main 2. 路径操作使用Paths和Path库,兼容Linux/Mac/Win 3. css的内容写到css文件中,java代码里只需要按需挂载style,class,tag就可以,马上就会面临参数化的问题,在代码里hardcode没办法维护。 4. removeEmptyParagraphs 如之前讨论的思路,尽量不用删除元素来控制layout,考虑用控制空行size的办法。 5. processFirstImageAsBackground理论上不需要全部设置来特殊对待第一张图片。如果图片不按设计显示,那就调整不准确的显示属性,其他的图片也是一样的处理方式。 6. @SuppressWarnings("checkstyle:NestedIfDepth") 拆分过于复杂的方法,不要suppressWarning。 7. 考虑建立HtmlConverter,PdfConverter类,参数化hardcode的值,把过程性代码对象化。 8. TOC的超链在pdf转换的过程中能正确生成吗?如果能那我就不去调查怎么在flyingsaucer处理html的过程中写定制插件了。因为如果要定制渲染过程中才能计算得到的数据,这里似乎是最合理的地方。
این pull request می تواند به صورت خودکار ادغام شود.
برای پیوستن به گفتگو، وارد شودید.
بدون برچسب
بدون نقطه عطف
بدون مسئول رسیدگی
1 مشارکت کننده
درحال بارگذاری...
لغو
ذخيره
هنوز محتوایی ایجاد نشده.