Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8100602

Need to remove existing Alert class

    XMLWordPrintable

    Details

    • Type: Enhancement
    • Status: Resolved
    • Priority: P2
    • Resolution: Fixed
    • Affects Version/s: fx2.0
    • Fix Version/s: fx2.0
    • Component/s: javafx
    • Labels:

      Description

      We currently have an Alert class with some static convenience methods defined.

      This class is not currently functional at all on Prism. In addition to the changes needed to make it functional, the current "Toolkit-based" implementation is not flexible (cannot be skinned), is based on an old JavaFX-TV implementation, and has introduced a dependency in javafx-ui-quantum on javafx-ui-controls. It needs to be rewritten.

      Additionally, the public Alert API is missing a necessary parameter for the owner of the dialog.

      I have prototyped an updated version of the API (by adding the missing "owner" parameter) and a bare-bones sample implementation in toys/HelloWorld/alertdialog that can be the basis of a real implementation.

      We need to do one of the following:

      1) Remove Alert and don't provide the functionality at all in this release. This is the easiest, but we would not be providing something that was there (after a fashion) in JavaFX 1.3.

      2) Move Alert to javafx.scene.control and fix up the API by adding the missing Window owner parameter to all of the methods. Implement a placeholder implementation, initially with no UI design. The problem is it will be a late-breaking feature and doing the UI design is not scheduled work. I had also wondered whether we could make it "preview" control for 2.0, but Rich thinks this may not fly.

      Both options will include removing javafx.stage.Alert and the toolkit implementations (which, as noted above, are not functional anyway).

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              kcr Kevin Rushforth
              Reporter:
              kcr Kevin Rushforth
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Imported: