Please review:

JBS: https://bugs.openjdk.java.net/browse/JDK-8241518
Webrev: http://cr.openjdk.java.net/~hannesw/8241518/webrev.00/
API docs: http://cr.openjdk.java.net/~hannesw/8241518/api.00/

This changes the „parameters“ span in member signatures to contain both opening 
and closing parens, whereas previously it only contained the closing parens. 

To preserve visual alignment of parameters, a space character is added after 
each line break added to the parameter list. Thus, the new code renders method 
signatures as shown below with the box representing the „parameters“ span:
           __________
methodName|(int p1,  |
          | int p2,  |
          | int p3)  |
          |__________|

Previously it was rendered the following way:
            _________
methodName(|int p1,  |
           |int p2,  |
           |int p3)  |
           |_________|

IMO this also improves layout when method name and parameters don’t fit the 
browser width, as the whole parameter span including both parens is now broken 
to a new line, preserving its layout with the single space indentation.

Thanks,
Hannes

Reply via email to