Re: [OpenJDK 2D-Dev] [12] RFR JDK-8213051:Invalid use of HTML5 in javax.print files
+1 On 30/11/2018 13:16, Alexey Ivanov wrote: Hi Prasanta, Looks good to me. Regards, Alexey On 30/11/2018 17:23, Prasanta Sadhukhan wrote: Hi Alexey, Ok. Modified webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.2/ Regards Prasanta On 30-Nov-18 10:12 PM, Alexey Ivanov wrote: Hi Prasanta, Adding style="text-align: center" attribute to element is cleaner and achieves the same result as it applies to all (as well which are center-aligned by default anyway). If the content of the table is modified in the future, there will be no need to add style attributes to new cells which will have non-empty content. Does it look reasonable? Regards, Alexey On 30/11/2018 15:25, Prasanta Sadhukhan wrote: Hi Alexey, I have modified to keep the "X" centred. Please find the modified webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.1/ Regards Prasanta On 30-Nov-18 4:50 PM, Alexey Ivanov wrote: Hi Prasanta, Would the presentation benefit from to keep X centred and thus to preserve the visual appearance? Do you mind updating the copyright? Otherwise, the change looks good to me. Regards, Alexey On 30/11/2018 07:41, Prasanta Sadhukhan wrote: Hi All, Please review a doc-fix to comply with HTML5 standard by removing the obsolete attribute "align" Bug: https://bugs.openjdk.java.net/browse/JDK-8213051 webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.0/ Regards Prasanta -- Best regards, Sergey.
Re: [OpenJDK 2D-Dev] [12] RFR JDK-8213051:Invalid use of HTML5 in javax.print files
Hi Prasanta, Looks good to me. Regards, Alexey On 30/11/2018 17:23, Prasanta Sadhukhan wrote: Hi Alexey, Ok. Modified webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.2/ Regards Prasanta On 30-Nov-18 10:12 PM, Alexey Ivanov wrote: Hi Prasanta, Adding style="text-align: center" attribute to element is cleaner and achieves the same result as it applies to all (as well which are center-aligned by default anyway). If the content of the table is modified in the future, there will be no need to add style attributes to new cells which will have non-empty content. Does it look reasonable? Regards, Alexey On 30/11/2018 15:25, Prasanta Sadhukhan wrote: Hi Alexey, I have modified to keep the "X" centred. Please find the modified webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.1/ Regards Prasanta On 30-Nov-18 4:50 PM, Alexey Ivanov wrote: Hi Prasanta, Would the presentation benefit from to keep X centred and thus to preserve the visual appearance? Do you mind updating the copyright? Otherwise, the change looks good to me. Regards, Alexey On 30/11/2018 07:41, Prasanta Sadhukhan wrote: Hi All, Please review a doc-fix to comply with HTML5 standard by removing the obsolete attribute "align" Bug: https://bugs.openjdk.java.net/browse/JDK-8213051 webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.0/ Regards Prasanta
Re: [OpenJDK 2D-Dev] [12] RFR JDK-8213051:Invalid use of HTML5 in javax.print files
Hi Alexey, Ok. Modified webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.2/ Regards Prasanta On 30-Nov-18 10:12 PM, Alexey Ivanov wrote: Hi Prasanta, Adding style="text-align: center" attribute to element is cleaner and achieves the same result as it applies to all (as well which are center-aligned by default anyway). If the content of the table is modified in the future, there will be no need to add style attributes to new cells which will have non-empty content. Does it look reasonable? Regards, Alexey On 30/11/2018 15:25, Prasanta Sadhukhan wrote: Hi Alexey, I have modified to keep the "X" centred. Please find the modified webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.1/ Regards Prasanta On 30-Nov-18 4:50 PM, Alexey Ivanov wrote: Hi Prasanta, Would the presentation benefit from to keep X centred and thus to preserve the visual appearance? Do you mind updating the copyright? Otherwise, the change looks good to me. Regards, Alexey On 30/11/2018 07:41, Prasanta Sadhukhan wrote: Hi All, Please review a doc-fix to comply with HTML5 standard by removing the obsolete attribute "align" Bug: https://bugs.openjdk.java.net/browse/JDK-8213051 webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.0/ Regards Prasanta
Re: [OpenJDK 2D-Dev] [12] RFR JDK-8213051:Invalid use of HTML5 in javax.print files
Hi Prasanta, Adding style="text-align: center" attribute to element is cleaner and achieves the same result as it applies to all (as well which are center-aligned by default anyway). If the content of the table is modified in the future, there will be no need to add style attributes to new cells which will have non-empty content. Does it look reasonable? Regards, Alexey On 30/11/2018 15:25, Prasanta Sadhukhan wrote: Hi Alexey, I have modified to keep the "X" centred. Please find the modified webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.1/ Regards Prasanta On 30-Nov-18 4:50 PM, Alexey Ivanov wrote: Hi Prasanta, Would the presentation benefit from to keep X centred and thus to preserve the visual appearance? Do you mind updating the copyright? Otherwise, the change looks good to me. Regards, Alexey On 30/11/2018 07:41, Prasanta Sadhukhan wrote: Hi All, Please review a doc-fix to comply with HTML5 standard by removing the obsolete attribute "align" Bug: https://bugs.openjdk.java.net/browse/JDK-8213051 webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.0/ Regards Prasanta
Re: [OpenJDK 2D-Dev] [12] RFR JDK-8213051:Invalid use of HTML5 in javax.print files
Hi Alexey, I have modified to keep the "X" centred. Please find the modified webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.1/ Regards Prasanta On 30-Nov-18 4:50 PM, Alexey Ivanov wrote: Hi Prasanta, Would the presentation benefit from to keep X centred and thus to preserve the visual appearance? Do you mind updating the copyright? Otherwise, the change looks good to me. Regards, Alexey On 30/11/2018 07:41, Prasanta Sadhukhan wrote: Hi All, Please review a doc-fix to comply with HTML5 standard by removing the obsolete attribute "align" Bug: https://bugs.openjdk.java.net/browse/JDK-8213051 webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.0/ Regards Prasanta
Re: [OpenJDK 2D-Dev] [12] RFR JDK-8213051:Invalid use of HTML5 in javax.print files
Hi Prasanta, Would the presentation benefit from to keep X centred and thus to preserve the visual appearance? Do you mind updating the copyright? Otherwise, the change looks good to me. Regards, Alexey On 30/11/2018 07:41, Prasanta Sadhukhan wrote: Hi All, Please review a doc-fix to comply with HTML5 standard by removing the obsolete attribute "align" Bug: https://bugs.openjdk.java.net/browse/JDK-8213051 webrev: http://cr.openjdk.java.net/~psadhukhan/8213051/webrev.0/ Regards Prasanta