API Change Proposal - Re: MSAA and Scene anti aliasing

[Chien Yang]

Yes, very good point, we struggled with the DEFAULT value too. It 
doesn't really convey the in between value of FAST and NICEST.

- Chien

On 7/23/2013 1:41 PM, Felix Bembrick wrote:
> I am not sure I like the name of the value DEFAULT.  To me "default" 
> implies some kind of "typical" option for the particular platform 
> which could conceivably be very different on each platform.  For 
> example, on my Windows super-computer with ultra fast GPU the 
> "default" would logically be something like 16x whereas on my iPhone I 
> would expect the default to be using only 2x.  The comment for DEFAULT 
> implies that it's really more of a BALANCED or MODERATE setting that 
> would have the same characteristics on each platform.
>
>
> On 24 July 2013 06:29, Chien Yang <chien.yang at oracle.com 
> <mailto:chien.yang at oracle.com>> wrote:
>
>     Hi all,
>
>         We appreciate all the feedback you have contributed to this
>     topic. After listening to the feedback and an internal discussion,
>     we would like to propose a minor change to the API for supporting
>     scene anti-aliasing. We intentionally choose not to expose the
>     number of samples and techniques used in this release, but this
>     doesn't preclude future addition when the time is right for more
>     options. This change will be tracked by RT-31878
>     (https://javafx-jira.kenai.com/browse/RT-31878):
>
>     Anti-aliasing API Change Proposal:
>
>     Constructors remove:
>     public Scene(Parent root, double width, double height, boolean
>     depthBuffer, boolean antiAliasing)
>     public SubScene(Parent root, double width, double height, boolean
>     depthBuffer, boolean antiAliasing)
>
>     Constructor add:
>     public Scene(Parent root, double width, double height, boolean
>     depthBuffer, SceneAntiAliasing antiAliasing)
>     public SubScene(Parent root, double width, double height, boolean
>     depthBuffer, SceneAntiAliasing antiAliasing)
>
>     Note:The antiAliasing argument will be used if the underlying
>     graphics driver has anti-aliasing support.
>
>     Where SceneAntiAliasing is an enum with the following entries at
>     the moment:
>
>     public enum SceneAntiAliasing {
>         DISABLED, // disables anti-aliasing
>         DEFAULT, // enables anti-aliasing using a default system
>     setting available that balances speed and quality
>         FASTEST, // enables anti-aliasing using minimum system setting
>     available that results in better frame rate
>         NICEST // enables anti-aliasing using maximum system setting
>     available that results in best visual quality
>     }
>
>     Thanks,
>     - Chien
>
>