Review Request: JDK-8210361: Add images to docs for public API classes of controls

2019-01-15 Thread Nir Lisker
Hi, Please review the current state of the patch for: https://bugs.openjdk.java.net/browse/JDK-8210361 http://cr.openjdk.java.net/~nlisker/8210361/webrev.00/ Thanks, Nir

Re: JDK-8210361: Add images to docs for public API classes of controls

2018-09-12 Thread Nir Lisker
Hi Anirvan, A gallery will fit best in the tutorials. The page for UI controls [1] will be ideal in my opinion (put the gallery as the first link). About the screenshots: that page contains much more suitable screenshots in each control's page when it comes to a gallery. They are stylistic,

Re: JDK-8210361: Add images to docs for public API classes of controls

2018-09-12 Thread Anirvan Sarkar
Hi Nir, Thanks for this RFE. Now other UI toolkits like GTK and QT have a separate gallery of controls[1][2]. I was considering to add something similar in JavaFX but I never got around to start working on it, mostly due to all the screenshots which is needed to be taken beforehand ! Since you

Re: JDK-8210361: Add images to docs for public API classes of controls

2018-09-12 Thread Kevin Rushforth
Agreed. For simple controls like checkbox, button, etc., it makes sense to use an undecorated Stage. It's less busy and puts the emphasis on the control being illustrated. -- Kevin On 9/12/2018 7:57 AM, Pedro Duque Vieira wrote: Yes I agree with you. Controls on which their window

Re: JDK-8210361: Add images to docs for public API classes of controls

2018-09-12 Thread Pedro Duque Vieira
Yes I agree with you. Controls on which their window decorations are a part of them, should show up with them. If those decorations change depending on the platform their running, I would put a note saying that. The vast majority of controls don't belong to that category. Cheers, On Wed, Sep

Re: JDK-8210361: Add images to docs for public API classes of controls

2018-09-12 Thread Nir Lisker
That's sensible, though for Alert and the like I think they should stay as it's effectively part of the control. On Wed, Sep 12, 2018 at 4:19 PM Pedro Duque Vieira < pedro.duquevie...@gmail.com> wrote: > I would remove the window decorations. That's platform dependent, windows, > linux, mac have

Re: JDK-8210361: Add images to docs for public API classes of controls

2018-09-12 Thread Pedro Duque Vieira
I would remove the window decorations. That's platform dependent, windows, linux, mac have different aesthetics for this. They also don't add any value to the representation of the control. That's what I usually do. Cheers, -- Pedro Duque Vieira - https://www.pixelduke.com

Re: JDK-8210361: Add images to docs for public API classes of controls

2018-09-12 Thread Kevin Rushforth
They look fine to me. I think placing them next to the code sample, near the bottom, seems best. -- Kevin On 9/12/2018 1:42 AM, Nir Lisker wrote: Hi, Iv'e started working on JDK-8210361 [1] and I'd like my comment there addressed before continuing. [1]

JDK-8210361: Add images to docs for public API classes of controls

2018-09-12 Thread Nir Lisker
Hi, Iv'e started working on JDK-8210361 [1] and I'd like my comment there addressed before continuing. [1] https://bugs.openjdk.java.net/browse/JDK-8210361 Thanks, Nir