minigui styling 2.0 start

dom.d

@@ -6828,7 +6828,7 @@ int intFromHex(string hex) {
 		///.
 		static immutable string[] selectorTokens = [
 			// It is important that the 2 character possibilities go first here for accurate lexing
-		    "~=", "*=", "|=", "^=", "$=", "!=", // "::" should be there too for full standard
+		    "~=", "*=", "|=", "^=", "$=", "!=",
 		    "::", ">>",
 		    "<<", // my any-parent extension (reciprocal of whitespace)
 		    // " - ", // previous-sibling extension (whitespace required to disambiguate tag-names)

jni.d

@@ -1137,6 +1137,15 @@ struct ActivateJniEnv {
 	"no jvm dll" in the message. That error can also be thrown if
 	you have a 32 bit program but try to load a 64 bit JVM, or vice
 	versa.
+
+	Returns:
+		an opaque object you should hold on to but not actually use.
+		Its destructor does necessary cleanup tasks to unload the jvm
+		but otherwise its effect is global.
+
+		As long as that object is in scope, you can work with java classes
+		globally and it will use that jvm's environment and load classes and jars
+		through the java classpath.
 +/
 auto createJvm()() {
 	version(Windows)

minigui_xml.d

@@ -96,7 +96,7 @@ void loadMiniguiPublicClasses() {
 	import std.traits;
 	import std.conv;
 
-	foreach(memberName; __traits(allMembers, mixin("arsd.minigui"))) static if(__traits(compiles, __traits(getMember, mixin("arsd.minigui"), memberName))) {
+	foreach(memberName; __traits(allMembers, mixin("arsd.minigui"))) static if(!__traits(isDeprecated, __traits(getMember, mixin("arsd.minigui"), memberName))) {
 		alias Member = ident!(__traits(getMember, mixin("arsd.minigui"), memberName));
 		static if(is(Member == class) && !isAbstractClass!Member && is(Member : Widget) && __traits(getProtection, Member) != "private") {
 			widgetFactoryFunctions[memberName] = (string[string] args, Widget parent) {

simpledisplay.d

@@ -2665,6 +2665,14 @@ public:
 		return false;
 	}
 
+	/++
+		Event listeners added with [addEventListener] have their exceptions swallowed by the event loop. This delegate can handle them again before it proceeds.
+
+		History:
+			Added May 12, 2021
+	+/
+	void delegate(Exception e) nothrow eventUncaughtException;
+
 	/** Add listener for custom event. Can be used like this:
 	 *
 	 * ---------------------
@@ -2691,8 +2699,10 @@ public:
 				if (auto co = cast(ET)o) {
 					try {
 						dg(co);
-					} catch (Exception) {
+					} catch (Exception e) {
 						// sorry!
+						if(eventUncaughtException)
+							eventUncaughtException(e);
 					}
 					return true;
 				}
@@ -7298,6 +7308,8 @@ enum FontWeight : int {
 	heavy = 900
 }
 
+// FIXME: i need a font cache and it needs to handle disconnects.
+
 /++
 	Represents a font loaded off the operating system or the X server.
 
@@ -7980,7 +7992,7 @@ struct ScreenPainter {
 			impl.create(handle);
 			impl.referenceCount = 1;
 			window.activeScreenPainter = impl;
-		//	writeln("constructed");
+			//import std.stdio; writeln("constructed");
 		}
 
 		copyActiveOriginals();
@@ -8001,7 +8013,7 @@ struct ScreenPainter {
 		impl.referenceCount--;
 		//writeln("refcount -- ", impl.referenceCount);
 		if(impl.referenceCount == 0) {
-			//writeln("destructed");
+			//import std.stdio; writeln("destructed");
 			impl.dispose();
 			*window.activeScreenPainter = ScreenPainterImplementation.init;
 			//import std.stdio; writeln("paint finished");
@@ -8062,21 +8074,33 @@ struct ScreenPainter {
 	}
 
 
-	/// Sets the clipping region for drawing. If width == 0 && height == 0, disabled clipping.
-	void setClipRectangle(Point pt, int width, int height) {
-		if(impl is null) return;
+	/++
+		Sets the clipping region for drawing. If width == 0 && height == 0, disabled clipping.
+
+		Returns:
+			The old clip rectangle.
+
+		History:
+			Return value was `void` prior to May 10, 2021.
+
+	+/
+	arsd.color.Rectangle setClipRectangle(Point pt, int width, int height) {
+		if(impl is null) return currentClipRectangle;
 		if(pt == currentClipRectangle.upperLeft && width == currentClipRectangle.width && height == currentClipRectangle.height)
-			return; // no need to do anything
+			return currentClipRectangle; // no need to do anything
+		auto old = currentClipRectangle;
 		currentClipRectangle = arsd.color.Rectangle(pt, Size(width, height));
 		transform(pt);
 
 		impl.setClipRectangle(pt.x, pt.y, width, height);
+
+		return old;
 	}
 
 	/// ditto
-	void setClipRectangle(arsd.color.Rectangle rect) {
-		if(impl is null) return;
-		setClipRectangle(rect.upperLeft, rect.width, rect.height);
+	arsd.color.Rectangle setClipRectangle(arsd.color.Rectangle rect) {
+		if(impl is null) return currentClipRectangle;
+		return setClipRectangle(rect.upperLeft, rect.width, rect.height);
 	}
 
 	///
@@ -8331,6 +8355,12 @@ struct ScreenPainter {
 			lowerRightInclusive.x - upperLeft.x + 1, lowerRightInclusive.y - upperLeft.y + 1);
 	}
 
+	// overload added on May 12, 2021
+	/// ditto
+	void drawRectangle(Rectangle rect) {
+		drawRectangle(rect.upperLeft, rect.size);
+	}
+
 	/// Arguments are the points of the bounding rectangle
 	void drawEllipse(Point upperLeft, Point lowerRight) {
 		if(impl is null) return;
@@ -10961,12 +10991,24 @@ version(X11) {
 		arsd.color.Rectangle _clipRectangle;
 		void setClipRectangle(int x, int y, int width, int height) {
 			_clipRectangle = arsd.color.Rectangle(Point(x, y), Size(width, height));
-			if(width == 0 || height == 0)
+			if(width == 0 || height == 0) {
 				XSetClipMask(display, gc, None);
-			else {
+
+				version(with_xft) {
+					if(xftFont is null || xftDraw is null)
+						return;
+					XftDrawSetClip(xftDraw, null);
+				}
+			} else {
 				XRectangle[1] rects;
 				rects[0] = XRectangle(cast(short)(x), cast(short)(y), cast(short) width, cast(short) height);
 				XSetClipRectangles(XDisplayConnection.get, gc, 0, 0, rects.ptr, 1, 0);
+
+				version(with_xft) {
+					if(xftFont is null || xftDraw is null)
+						return;
+					XftDrawSetClipRectangles(xftDraw, 0, 0, rects.ptr, 1);
+				}
 			}
 		}
 
@@ -11711,13 +11753,15 @@ mixin DynamicLoad!(XRender, "Xrender", 1, false, true) XRenderLibrary;
 	struct BOX {
 		short x1, x2, y1, y2;
 	}
-	struct Region {
+	struct _XRegion {
 		c_long size;
 		c_long numRects;
 		BOX* rects;
 		BOX extents;
 	}
 
+	alias Region = _XRegion*;
+
 	// ok actually Xft
 
 	struct XftFontInfo;
@@ -17858,6 +17902,8 @@ static:
 			}
 		}
 
+		Color selectionXorColor = Color(255, 255, 127);
+
 		void highlightSelection(ScreenPainter painter) {
 			if(selectionStart is selectionEnd)
 				return; // no selection
@@ -17870,7 +17916,7 @@ static:
 
 			painter.rasterOp = RasterOp.xor;
 			painter.outlineColor = Color.transparent;
-			painter.fillColor = Color(255, 255, 127);
+			painter.fillColor = selectionXorColor;
 
 			auto at = selectionStart.inlineElement;
 			auto atOffset = selectionStart.offset;