GH-431 clean up viewer component dialogs (#433)

Author: pcorless

viewer/viewer-awt/src/main/java/org/icepdf/ri/common/views/DocumentViewControllerImpl.java

@@ -27,6 +27,7 @@
 import org.icepdf.ri.common.views.annotations.AbstractAnnotationComponent;
 import org.icepdf.ri.common.views.annotations.AnnotationState;
 import org.icepdf.ri.common.views.annotations.PopupAnnotationComponent;
+import org.icepdf.ri.common.views.annotations.signing.BasicSignatureAppearanceCallback;
 import org.icepdf.ri.common.views.destinations.DestinationComponent;
 import org.icepdf.ri.images.Images;
 
@@ -361,7 +362,12 @@ public AnnotationCallback getAnnotationCallback() {
      * @return assigned callback
      */
     public SignatureAppearanceCallback getSignatureAppearanceCallback() {
-        return signatureAppearanceCallback;
+        if (signatureAppearanceCallback != null) {
+            return signatureAppearanceCallback;
+        } else {
+            // component will use the default implementation
+            return new BasicSignatureAppearanceCallback();
+        }
     }
 
     /**

viewer/viewer-awt/src/main/java/org/icepdf/ri/common/views/annotations/summary/AnnotationSummaryFrame.java

@@ -46,7 +46,8 @@ public void refreshDocumentInstance() {
             if (document.getInfo() != null) {
                 title = document.getInfo().getTitle();
             }
-            Object[] messageArguments = {title != null ? title : controller.getViewerFrame().getTitle()};
+            Object[] messageArguments = {title != null ? title : controller.getViewerFrame() != null ?
+                    controller.getViewerFrame().getTitle() : ""};
             MessageFormat formatter = new MessageFormat(
                     messageBundle.getString("viewer.window.annotationSummary.title.open.default"));
             setTitle(formatter.format(messageArguments));

viewer/viewer-awt/src/main/java/org/icepdf/ri/common/widgets/RgbColorChooser.java

@@ -52,15 +52,6 @@ public class RgbColorChooser {
     public static Color showDialog(Component component, String title, Color initialColor)
             throws HeadlessException {
 
-        try {
-            String defaultLF = UIManager.getSystemLookAndFeelClassName();
-            if (defaultLF.contains("GTK")) {
-                UIManager.setLookAndFeel("javax.swing.plaf.metal.MetalLookAndFeel");
-            }
-        } catch (Exception e) {
-            logger.warning("Could not set metal look and feel.");
-        }
-
         JColorChooser colorChooser = new JColorChooser(initialColor != null ?
                 initialColor : Color.white);
         AbstractColorChooserPanel[] panels = colorChooser.getChooserPanels();
@@ -83,6 +74,7 @@ public static Color showDialog(Component component, String title, Color initialC
                 title,
                 true, colorChooser, colorTracker, cancelActionListener);
 
+        colorChooserDialog.setLocationByPlatform(true);
         colorChooserDialog.setVisible(true);
 
         try {