Added documentation and changed preset themes

Author: spfrommer

src/com/demo/QueueManagerDemo.java

@@ -26,9 +26,10 @@ public static void main(String[] args) throws Exception {
 				break;
 			case 1:
 				note = factory.buildTextNotification("TextNotification", "Subtitle");
+				break;
 			case 2:
-				note = factory.buildAcceptNotification("AcceptNotification", "Click accept or decline");
-
+				note = factory.buildAcceptNotification("AcceptNotification", "Do you accept?");
+				break;
 			}
 			// note.setCloseOnClick(true);
 			// make it show in the queue for five seconds

src/com/notification/NotificationFactory.java

@@ -154,11 +154,12 @@ public AcceptNotification buildAcceptNotification(String title, String subtitle,
 	 * @return
 	 */
 	public IconNotification buildIconNotification(String title, String subtitle, ImageIcon icon) {
-		IconNotification iconNote = new IconNotification(icon);
+		IconNotification iconNote = new IconNotification();
 		iconNote.setWindowTheme(m_pack.windowTheme);
 		iconNote.setTextTheme(m_pack.textTheme);
 		iconNote.setTitle(title);
 		iconNote.setSubtitle(subtitle);
+		iconNote.setIcon(icon);
 
 		return iconNote;
 	}

src/com/notification/types/IconNotification.java

@@ -26,9 +26,8 @@ public class IconNotification extends BorderLayoutNotification {
 
 	public static final int ICON_PADDING = 10;
 
-	public IconNotification(ImageIcon icon) {
+	public IconNotification() {
 		m_iconLabel = new JLabel();
-		m_iconLabel.setIcon(icon);
 		m_titleLabel = new JLabel();
 		m_subtitleLabel = new JLabel();
 
@@ -50,40 +49,56 @@ public void setIcon(ImageIcon icon) {
 		m_iconLabel.setIcon(icon);
 	}
 
-	protected TextTheme getTextTheme() {
-		return m_theme;
-	}
-
 	/**
-	 * @param theme
-	 *            the two Fonts that should be used.
+	 * @return the title String
 	 */
-	public void setTextTheme(TextTheme theme) {
-		m_theme = theme;
-		m_titleLabel.setFont(theme.title);
-		m_subtitleLabel.setFont(theme.subtitle);
-		m_titleLabel.setForeground(theme.titleColor);
-		m_subtitleLabel.setForeground(theme.subtitleColor);
-	}
-
 	public String getTitle() {
 		return m_title;
 	}
 
+	/**
+	 * Sets the title String.
+	 * 
+	 * @param title
+	 */
 	public void setTitle(String title) {
 		m_titleLabel.setText(title);
 		m_title = title;
 	}
 
+	/**
+	 * @return the subtitle String
+	 */
 	public String getSubtitle() {
 		return m_subtitle;
 	}
 
+	/**
+	 * Sets the subtitle String.
+	 * 
+	 * @param subtitle
+	 */
 	public void setSubtitle(String subtitle) {
 		m_subtitleLabel.setText(subtitle);
 		m_subtitle = subtitle;
 	}
 
+	protected TextTheme getTextTheme() {
+		return m_theme;
+	}
+
+	/**
+	 * @param theme
+	 *            the two Fonts that should be used.
+	 */
+	public void setTextTheme(TextTheme theme) {
+		m_theme = theme;
+		m_titleLabel.setFont(theme.title);
+		m_subtitleLabel.setFont(theme.subtitle);
+		m_titleLabel.setForeground(theme.titleColor);
+		m_subtitleLabel.setForeground(theme.subtitleColor);
+	}
+
 	@Override
 	public void setWindowTheme(WindowTheme theme) {
 		super.setWindowTheme(theme);

src/com/theme/ThemePackagePresets.java

@@ -18,8 +18,8 @@ public static ThemePackage cleanLight() {
 		window.height = 100;
 
 		TextTheme text = new TextTheme();
-		text.title = new Font("Arial", Font.BOLD, 25);
-		text.subtitle = new Font("Arial", Font.PLAIN, 20);
+		text.title = new Font("Arial", Font.BOLD, 22);
+		text.subtitle = new Font("Arial", Font.PLAIN, 16);
 		text.titleColor = new Color(10, 10, 10);
 		text.subtitleColor = new Color(10, 10, 10);
 
@@ -40,8 +40,8 @@ public static ThemePackage cleanDark() {
 		window.height = 100;
 
 		TextTheme text = new TextTheme();
-		text.title = new Font("Arial", Font.BOLD, 25);
-		text.subtitle = new Font("Arial", Font.PLAIN, 20);
+		text.title = new Font("Arial", Font.BOLD, 22);
+		text.subtitle = new Font("Arial", Font.PLAIN, 16);
 		text.titleColor = new Color(200, 200, 200);
 		text.subtitleColor = new Color(200, 200, 200);