Changeset 1545 for trunk/src/org/expeditee/items/Item.java

Timestamp:

11/19/20 13:41:28 (3 years ago)

Author:

bnemhaus

Message:

Expeditee now respects the users antialiasing setting. The previous system

1. Attempted to respect antialiasing by checking the setting and enabling it if the setting was true
2. In the process of painting the frame some special items (link marks etc) want to use antialiasing reguardless of setting. This was achieved by turning antialiasing on, doing the thing and then turning it off. This unfortunately meant that, in the scenario that the users antialiasing setting is on, it gets turned off after drawing one of these special items and doesn't get turn on again until all items have been drawn.

The new system still always turns antialiasing on for these special items, but instead of turning it off after, it returns it to the setting it was previously on. This is achieved with two new functions: setTransientAntialiasingOn() and setTransientAntialiasingOff().

On a related note, there was also an issue with some polygons being drawn with a thickness of 0.0f, which was causing them to antialias badly. They now have a thickness of 1.0f.

File:

• trunk/src/org/expeditee/items/Item.java (modified) (5 diffs)

trunk/src/org/expeditee/items/Item.java

@@ -105 +105 @@
         public static final Float DEFAULT_THICKNESS = 2f;
 
-        public static final Float MINIMUM_THICKNESS = 0f;
+        public static final Float MINIMUM_THICKNESS = 1f;
 
         public static final Float MINIMUM_PAINT_THICKNESS = 1f;
@@ -3170 +3170 @@
 
                         // Small circles look rubbish without AA. cts16
-                        g.setAntialiasing(true);
+                        g.setTransientAntialiasingOn();
                         
                         if (hasLink && getLinkMark()) {
@@ -3190 +3190 @@
                         }
                         
-                        g.setAntialiasing(false);
+                        g.setTransientAntialiasingOff();
                 }
         }
@@ -3208 +3208 @@
                 
                 GraphicsManager g = EcosystemManager.getGraphicsManager();
-                g.setAntialiasing(true);
+                g.setTransientAntialiasingOn();
                 Font font = new Font();
                 g.drawString("\uD83D\uDD11", new Point(x,y), font, fontColour);
-                g.setAntialiasing(false);
+                g.setTransientAntialiasingOff();
         }
         
@@ -3218 +3218 @@
                 if (encryptionLabel != null && encryptionLabel.length() > 0) {
                         GraphicsManager g = EcosystemManager.getGraphicsManager();
-                        g.setAntialiasing(true);
+                        g.setTransientAntialiasingOn();
                         Font font = new Font();
                         g.drawString("\uD83D\uDD12", new Point(x,y), font, Colour.BLACK);
-                        g.setAntialiasing(false);
+                        g.setTransientAntialiasingOff();
                 }
         }