X-Git-Url: http://git.megacz.com/?a=blobdiff_plain;f=src%2Forg%2Fxwt%2FBox.java.pp;h=4da565a2a54f997b98595df1aee3b80dff83bf61;hb=cf3587e2fd5966b7ebfd721d9413674224d1ad2a;hp=50ef3091e81375290c3457683c1a659ecae0145f;hpb=0a4cb2647316bda42e94ee37607f93f731552b37;p=org.ibex.core.git diff --git a/src/org/xwt/Box.java.pp b/src/org/xwt/Box.java.pp index 50ef309..4da565a 100644 --- a/src/org/xwt/Box.java.pp +++ b/src/org/xwt/Box.java.pp @@ -6,11 +6,10 @@ package org.xwt; // RULE: coordinates on non-static methods are ALWAYS relative to the // upper-left hand corner of this +// FIXME: font color, italicization, bolding, and underlining // FIXME: align -// FIXME use bitfields -// FIXME fixedaspect -// FIXME: reflow before allowing js to read from width/height -// FIXME: due to font inheritance, we must dirty and mark all null-font descendents of a node if its font changes +// FIXME: fixed aspect +// FEATURE: reflow before allowing js to read from width/height // FEATURE: fastpath for rows=1/cols=1 // FEATURE: reflow starting with a certain child // FEATURE: separate mark_for_reflow and mark_for_resize @@ -20,6 +19,7 @@ import java.net.*; import java.util.*; import org.xwt.js.*; import org.xwt.util.*; +import org.xwt.translators.*; /** *

@@ -55,10 +55,11 @@ import org.xwt.util.*; * SizeChanges trigger an Surface.abort; if rendering were done in the same * pass, rendering work done prior to the Surface.abort would be wasted. * - * Repacking is seperate from resizing since a box's size depends on - * both the box's parent's size (so the traversal must be preorder) - * contentwidths of siblings both before and after the box, which in - * turn depend on all descendents of the siblings. FIXME + * The two passes are repack and resize. Together they are known as + * reflow. Repacking assigns boxes to the appropriate grid + * coordinates within the parents and computes + * contentwidth/contentheight. Resize computes actual size and + * position. * * A note on coordinates: the Box class represents regions * internally as x,y,w,h tuples, even though the DoubleBuffer class @@ -81,12 +82,35 @@ public final class Box extends JS.Scope { //#define MARK_FOR_REFLOW_b for(Box b2 = b; b2 != null && !b2.needs_reflow; b2 = b2.parent) b2.needs_reflow = true; //#define MARK_FOR_REFLOW_b_parent for(Box b2 = b.parent; b2 != null && !b2.needs_reflow; b2 = b2.parent) b2.needs_reflow = true; - private boolean mouseinside = false; Box redirect = this; Surface surface = null; // null on all non-root boxen + + // FEATURE: combine this with the JSObject Hash Hash traps = null; + // Flags /////////////////////////////////////////////////////////////////////////////// + static int MOUSEINSIDE_FLAG = 0x00000001; + static int INVISIBLE_FLAG = 0x00000002; + static int ABSOLUTE_FLAG = 0x00000004; + static int HSHRINK_FLAG = 0x00000010; + static int VSHRINK_FLAG = 0x00000020; + static int TILE_FLAG = 0x00000040; + + /** + * Set when the font changes, cleared during repack. If set + * during repack, all font==null children are marked for reflow + * and given the font_changed_flag. We use this flag to avoid + * having to iterate over all descendents of a box when its font + * changes. + */ + static int FONT_CHANGED_FLAG = 0x00000100; + + static int ALIGN_FLAG = 0x00000000; + static int FIXEDASPECT_FLAG = 0x00000000; + int flags = 0; + + // Geometry //////////////////////////////////////////////////////////////////////////// // xwt can be compiled with 16-bit lengths to save memory on small devices @@ -106,7 +130,7 @@ public final class Box extends JS.Scope { private LENGTH textwidth = 0; private LENGTH textheight = 0; - // FIXME: use shorts + // FEATURE: use shorts private int rows = 1; private int cols = 0; private int rowspan = 1; @@ -117,9 +141,9 @@ public final class Box extends JS.Scope { LENGTH y = 0; public LENGTH width = 0; public LENGTH height = 0; - private int row = 0; // FIXME short - private int col = 0; // FIXME short - private LENGTH contentwidth = 0; // == max(minwidth, textwidth, sum(child.contentwidth) + pad) + private int row = 0; // FEATURE use a short + private int col = 0; // FEATURE use a short + private LENGTH contentwidth = 0; // == max(minwidth, textwidth+pad, sum(child.contentwidth) + pad) private LENGTH contentheight = 0; @@ -135,19 +159,13 @@ public final class Box extends JS.Scope { private String cursor = null; // the cursor for this box - //FIXME make private - public boolean invisible = false; // true iff the Box is invisible - private boolean absolute = false; // If true, the box will be positioned absolutely - private boolean vshrink = false; // If true, the box will shrink to the smallest vertical size possible - private boolean hshrink = false; // If true, the box will shrink to the smallest horizontal size possible - private boolean tile = false; // FIXME: drop this? // Instance Methods ///////////////////////////////////////////////////////////////////// - // FIXME: rethink /** Adds the intersection of (x,y,w,h) and the node's current actual geometry to the Surface's dirty list */ public final void dirty() { dirty(0, 0, width, height); } public final void dirty(int x, int y, int w, int h) { + /* for(Box cur = this; cur != null; cur = cur.parent) { w = min(x + w, cur.width) - max(x, 0); h = min(y + h, cur.height) - max(y, 0); @@ -158,6 +176,10 @@ public final class Box extends JS.Scope { x += cur.x; y += cur.y; } + */ + Box cur; + for(cur = this; cur.parent != null; cur = cur.parent); + if (cur.surface != null) cur.surface.dirty(0, 0, cur.width, cur.height); } /** @@ -170,9 +192,9 @@ public final class Box extends JS.Scope { void Move(int oldmousex, int oldmousey, int mousex, int mousey) { Move(oldmousex, oldmousey, mousex, mousey, false); } void Move(int oldmousex, int oldmousey, int mousex, int mousey, boolean forceleave) { - boolean wasinside = mouseinside; - boolean isinside = !invisible && inside(mousex, mousey) && !forceleave; - mouseinside = isinside; + boolean wasinside = (flags & MOUSEINSIDE_FLAG) != 0; + boolean isinside = !((flags & INVISIBLE_FLAG) != 0) && inside(mousex, mousey) && !forceleave; + if (isinside) flags |= MOUSEINSIDE_FLAG; else flags &= ~MOUSEINSIDE_FLAG; if (!wasinside && !isinside) return; @@ -204,14 +226,23 @@ public final class Box extends JS.Scope { /** Checks if the Box's size has changed, dirties it if necessary, and makes sure childrens' sizes are up to date */ void repack() { if (!needs_reflow) return; - if (numChildren() == 0) { contentwidth = minwidth; contentheight = minheight; return; } + if (numChildren() == 0) { + contentwidth = max(textwidth + 2 * hpad, minwidth); + contentheight = max(textheight + 2 * vpad, minheight); + return; + } // --- Phase 0 ---------------------------------------------------------------------- // recurse for(Box child = getChild(0); child != null; child = child.nextSibling()) { + if (((flags & FONT_CHANGED_FLAG) != 0) && child.font == null) { + child.flags |= FONT_CHANGED_FLAG; + child.needs_reflow = true; + } child.repack(); if (Surface.abort) { MARK_FOR_REFLOW_this; return; } } + flags &= ~FONT_CHANGED_FLAG; // --- Phase 1 ---------------------------------------------------------------------- // assign children to their row/column positions (assuming constrained columns) @@ -220,7 +251,7 @@ public final class Box extends JS.Scope { if (rows == 0) { int[] numRowsInCol = new int[cols]; // the number of cells occupied in each column Box child = getChild(0); - for(; child != null && (child.absolute || child.invisible); child = child.nextSibling()); + for(; child != null && (((child.flags & ABSOLUTE_FLAG) != 0) || ((child.flags & INVISIBLE_FLAG) != 0)); child = child.nextSibling()); OUTER: for(int row=0; child != null; row++) { for(int col=0; child != null && col < cols;) { INNER: while(true) { // scan across the row, looking for an unoccupied gap at least as wide as the child @@ -235,7 +266,7 @@ public final class Box extends JS.Scope { child.row = row; col += min(cols, child.colspan); child = child.nextSibling(); - for(; child != null && (child.absolute || child.invisible); child = child.nextSibling()); + for(; child != null && (((child.flags & ABSOLUTE_FLAG) != 0) || ((child.flags & INVISIBLE_FLAG) != 0)); child = child.nextSibling()); } } } @@ -251,7 +282,7 @@ public final class Box extends JS.Scope { numCols = max(numCols, child.col + child.colspan); LENGTH[] colWidth = new LENGTH[numCols]; for(Box child = getChild(0); child != null; child = child.nextSibling()) - if (!(child.absolute || child.invisible)) + if (!(((child.flags & ABSOLUTE_FLAG) != 0) || ((child.flags & INVISIBLE_FLAG) != 0))) colWidth[child.col] = max(colWidth[child.col], child.contentwidth / child.colspan); for(int col=0; col 0) while(slack > 0) { @@ -347,18 +378,18 @@ public final class Box extends JS.Scope { // --- Phase 4 ---------------------------------------------------------------------- // assign children's new sizes and positions and recurse for(Box child = getChild(0); child != null; child = child.nextSibling()) { - if (child.invisible) continue; + if ((child.flags & INVISIBLE_FLAG) != 0) continue; int child_x = 0, child_y = 0, child_width = 0, child_height = 0; - if (child.absolute) { + if ((child.flags & ABSOLUTE_FLAG) != 0) { child_x = child.x; child_y = child.y; - child_width = child.hshrink ? child.contentwidth : min(child.maxwidth, width - child.x - hpad); - child_height = child.vshrink ? child.contentheight : min(child.maxheight, height - child.y - vpad); + child_width = ((child.flags & HSHRINK_FLAG) != 0) ? child.contentwidth : min(child.maxwidth, width - child.x - hpad); + child_height = ((child.flags & VSHRINK_FLAG) != 0) ? child.contentheight : min(child.maxheight, height - child.y - vpad); } else { int diff; - //#repeat x/y y/x width/height col/row cols/rows colspan/rowspan colWidth/rowHeight maxwidth/maxheight minwidth/minheight contentwidth/contentheight colMaxWidth/rowMaxHeight hshrink/vshrink marginWidth/marginHeight hpad/vpad child_x/child_y child_width/child_height + //#repeat x/y y/x width/height col/row cols/rows colspan/rowspan colWidth/rowHeight maxwidth/maxheight minwidth/minheight contentwidth/contentheight colMaxWidth/rowMaxHeight HSHRINK_FLAG/VSHRINK_FLAG marginWidth/marginHeight hpad/vpad child_x/child_y child_width/child_height child_width = 0; for(int i=child.col; i i) { - for(int j = x + hpad; j < x + hpad + textdim(0); j += 2) - buf.fillRect(j, y + vpad + (xwf == null ? Platform.getMaxAscent(font()) : xwf.getMaxAscent()) + 2, - j + 1, y + vpad + (xwf == null ? Platform.getMaxAscent(font()) : xwf.getMaxAscent()) + 2 + 1, - textcolor); - - } else if (font().lastIndexOf('u') > i) { - buf.fillRect(x + hpad, - y + vpad + (xwf == null ? Platform.getMaxAscent(font()) : xwf.getMaxAscent()) + 2, - x + hpad + textdim(0), - y + vpad + (xwf == null ? Platform.getMaxAscent(font()) : xwf.getMaxAscent()) + 2 + 1, - textcolor); - } - */ + void renderText(int x, int y, int clipx, int clipy, int clipw, int cliph, DoubleBuffer buf) { + /* + // hack because (believe it or not) libgcj doesn't support UTF16. + byte[] b = new byte[text.length() * 2 + 2]; + for(int i=0; i> 8); + b[i * 2 + 1] = (byte)(((short)text.charAt(i)) & 0xff); + } + b[text.length()] = 0; + b[text.length() + 1] = 0; + */ + /* + try { + ImageDecoder id = org.xwt.translators.Font.render(new FileInputStream("COMIC.TTF"), 24, text, false); + Picture p = Platform.createPicture(id); + // FIXME: clipping (don't use setClip) + buf.drawPicture(p, + x + hpad, y + vpad, + x + hpad + p.getWidth(), y + vpad + p.getHeight(), + 0, 0, + p.getWidth(), p.getHeight()); + } catch (Exception e) { + Log.log(this, e); + } + */ } @@ -486,7 +514,17 @@ public final class Box extends JS.Scope { } else if ("apply".equals(method)) { if (checkOnly) return Boolean.TRUE; - if (args.elementAt(0) instanceof String) { + if (args.elementAt(0) instanceof Res) { + Res res = (Res)args.elementAt(0); + res = res.addExtension(".xwt"); + Template t = Template.buildTemplate(res, "fromResource"); + if (ThreadMessage.suspendThread()) try { + JS.Callable callback = args.length() < 2 ? null : (Callable)args.elementAt(1); + t.apply(this, null, null, callback, 0, t.numUnits()); + } finally { + ThreadMessage.resumeThread(); + } + } else if (args.elementAt(0) instanceof String) { String templatename = (String)args.elementAt(0); Template t = Template.getTemplate(templatename, null); if (t == null) { @@ -607,6 +645,7 @@ public final class Box extends JS.Scope { } } + public Object get_(Object name) { return super.get(name); } public Object get(Object name) { return get(name, false); } public Object get(Object name_, boolean ignoretraps) { if (name_ instanceof Number) return get(((Number)name_).intValue()); @@ -648,6 +687,7 @@ public final class Box extends JS.Scope { * @param ignoretraps if set, no traps will be triggered (set when 'cascade' reaches the bottom of the trap stack) * @param rp if this put is being performed via a root proxy, rp is the root proxy. */ + public void put_(Object name, Object value) { super.put(name, value); } public void put(Object name, Object value) { put(name, value, false, null); } public void put(Object name, Object value, boolean ignoretraps) { put(name, value, ignoretraps, null); } public void put(Object name_, Object value, boolean ignoretraps, RootProxy rp) { @@ -731,7 +771,7 @@ public final class Box extends JS.Scope { if (oldparent == null) return; MARK_FOR_REFLOW_this; dirty(); - mouseinside = false; + flags &= ~MOUSEINSIDE_FLAG; if (parent.children != null) { parent.children.removeElementAt(indexInParent); @@ -860,7 +900,7 @@ public final class Box extends JS.Scope { static final int min(int a, int b, int c) { if (a<=b && a<=c) return a; else if (b<=c && b<=a) return b; else return c; } static final int max(int a, int b, int c) { if (a>=b && a>=c) return a; else if (b>=c && b>=a) return b; else return c; } static final int bound(int a, int b, int c) { if (c < b) return c; if (a > b) return a; return b; } - final boolean inside(int x, int y) { return (!invisible && x >= 0 && y >= 0 && x < width && y < height); } + final boolean inside(int x, int y) { return (!((flags & INVISIBLE_FLAG) != 0) && x >= 0 && y >= 0 && x < width && y < height); } /** figures out what box in this subtree of the Box owns the pixel at x,y relitave to the Surface */ public static Box whoIs(Box cur, int x, int y) { @@ -874,7 +914,7 @@ public final class Box extends JS.Scope { // is UNSYNCHRONIZED for performance reasons. BE CAREFUL // HERE. - if (cur.invisible) return null; + if ((cur.flags & INVISIBLE_FLAG) != 0) return null; if (!cur.inside(x - globalx, y - globaly)) return cur.parent == null ? cur : null; OUTER: while(true) { for(int i=cur.numChildren() - 1; i>=0; i--) { @@ -882,7 +922,7 @@ public final class Box extends JS.Scope { if (child == null) continue; // since this method is unsynchronized, we have to double-check globalx += child.x; globaly += child.y; - if (!child.invisible && child.inside(x - globalx, y - globaly)) { cur = child; continue OUTER; } + if (!((child.flags & INVISIBLE_FLAG) != 0) && child.inside(x - globalx, y - globaly)) { cur = child; continue OUTER; } globalx -= child.x; globaly -= child.y; } @@ -948,7 +988,6 @@ public final class Box extends JS.Scope { } else if (SVG.colors.get(s) != null) newcolor = 0xFF000000 | ((Integer)SVG.colors.get(s)).intValue(); - // FIXME put named colors back in if (newcolor == b.fillcolor) return; b.fillcolor = newcolor; b.dirty(); @@ -960,6 +999,7 @@ public final class Box extends JS.Scope { public Object get(Box b) { return b.get("fillcolor"); } public void put(Box b, Object value) { b.put("fillcolor", value); } }); + specialBoxProperties.put("textcolor", new SpecialBoxProperty() { public Object get(Box b) { return b.get("strokecolor"); } public void put(Box b, Object value) { b.put("strokecolor", value); } @@ -970,13 +1010,30 @@ public final class Box extends JS.Scope { public void put(Box b, Object value) { String t = value == null ? "null" : value.toString(); if (t.equals(b.text)) return; - // FIXME text is broken + b.text = t; + if (t == null) { + if (b.textwidth != 0 || b.textheight != 0) MARK_FOR_REFLOW_b; + b.textwidth = b.textheight = 0; + } else { + try { + ImageDecoder id = org.xwt.translators.Font.render(new FileInputStream("COMIC.TTF"), 24, b.text, true); + if (id.getWidth() != b.textwidth || id.getHeight() != b.textheight) MARK_FOR_REFLOW_b; + b.textwidth = id.getWidth(); + b.textheight = id.getHeight(); + } catch (Exception e) { + Log.log(this, e); + } + } + b.dirty(); } }); + specialBoxProperties.put("font", new SpecialBoxProperty() { public Object get(Box b) { return b.font; } public void put(Box b, Object value) { b.font = value == null ? null : value.toString(); - //b.fontChanged(); + // FIXME: translate value into a resource + MARK_FOR_REFLOW_b; + b.flags |= FONT_CHANGED_FLAG; b.dirty(); } }); @@ -991,6 +1048,7 @@ public final class Box extends JS.Scope { specialBoxProperties.put("orient", new SpecialBoxProperty() { public Object get(Box b) { + Log.log(this, "warning: the orient property is deprecated"); if (b.redirect == null) return "horizontal"; else if (b.redirect != b) return get(b.redirect); else if (b.cols == 1) return "vertical"; @@ -998,6 +1056,7 @@ public final class Box extends JS.Scope { else return "grid"; } public void put(Box b, Object value) { + Log.log(this, "warning: the orient property is deprecated"); if (value == null) return; if (b.redirect == null) return; if (b.redirect != b) { put(b.redirect, value); return; } @@ -1026,17 +1085,17 @@ public final class Box extends JS.Scope { }); specialBoxProperties.put("shrink", new SpecialBoxProperty() { - public Object get(Box b) { return (b.vshrink && b.hshrink) ? Boolean.TRUE : Boolean.FALSE; } + public Object get(Box b) { return (((b.flags & HSHRINK_FLAG) != 0) || ((b.flags & VSHRINK_FLAG) != 0)) ? Boolean.TRUE : Boolean.FALSE; } public void put(Box b, Object value) { b.put("hshrink", value); b.put("vshrink", value); } }); - //#repeat hshrink/vshrink + //#repeat hshrink/vshrink HSHRINK_FLAG/VSHRINK_FLAG specialBoxProperties.put("hshrink", new SpecialBoxProperty() { - public Object get(Box b) { return new Boolean(b.hshrink); } + public Object get(Box b) { return new Boolean((b.flags & HSHRINK_FLAG) != 0); } public void put(Box b, Object value) { boolean newshrink = stob(value); - if (b.hshrink == newshrink) return; - b.hshrink = newshrink; + if (((b.flags & HSHRINK_FLAG) != 0) == newshrink) return; + if (newshrink) b.flags |= HSHRINK_FLAG; else b.flags &= ~HSHRINK_FLAG; MARK_FOR_REFLOW_b; } }); @@ -1046,18 +1105,17 @@ public final class Box extends JS.Scope { specialBoxProperties.put("x", new SpecialBoxProperty() { public Object get(Box b) { if (b.surface == null) return new Integer(0); - if (b.invisible) return new Integer(0); + if ((b.flags & INVISIBLE_FLAG) != 0) return new Integer(0); return new Integer(b.x); } public void put(Box b, Object value) { - if (!b.absolute) return; + if (!((b.flags & ABSOLUTE_FLAG) != 0)) return; int x = stoi(value); if (x == b.x) return; b.dirty(); b.x = x; if (b.parent == null && b.surface != null) { - // FIXME this gets hosed by the #repeat - //b.surface.setLocation(b.x, b.y); + b.surface.setLocation(); b.surface.centerSurfaceOnRender = false; } MARK_FOR_REFLOW_b; @@ -1075,6 +1133,7 @@ public final class Box extends JS.Scope { b.surface.setSize(); MARK_FOR_REFLOW_b; } else { + if (b.minwidth == b.width && b.maxwidth == b.width) return; b.minwidth = b.maxwidth = b.width; MARK_FOR_REFLOW_b; } @@ -1085,6 +1144,7 @@ public final class Box extends JS.Scope { specialBoxProperties.put("cols", new SpecialBoxProperty() { public Object get(Box b) { return new Double(b.cols); } public void put(Box b, Object value) { + if (b.cols == stoi(value)) return; b.cols = stoi(value); if (b.cols == 0 && b.rows == 0) b.rows = 1; if (b.cols != 0 && b.rows != 0) b.rows = 0; @@ -1095,30 +1155,33 @@ public final class Box extends JS.Scope { //#repeat colspan/rowspan specialBoxProperties.put("colspan", new SpecialBoxProperty() { public Object get(Box b) { return new Double(b.colspan); } - public void put(Box b, Object value) { b.colspan = stoi(value); MARK_FOR_REFLOW_b; } + public void put(Box b, Object value) { + if (b.colspan == stoi(value)) return; + b.colspan = stoi(value); + MARK_FOR_REFLOW_b; + } }); //#end specialBoxProperties.put("tile", new SpecialBoxProperty() { - public Object get(Box b) { return b.tile ? Boolean.TRUE : Boolean.FALSE; } + public Object get(Box b) { return ((b.flags & TILE_FLAG) != 0) ? Boolean.TRUE : Boolean.FALSE; } public void put(Box b, Object value) { - boolean newtile = stob(value); - if (newtile == b.tile) return; - b.tile = newtile; + if (((b.flags & TILE_FLAG) != 0) == stob(value)) return; + if (stob(value)) b.flags |= TILE_FLAG; else b.flags &= ~TILE_FLAG; b.dirty(); } }); specialBoxProperties.put("invisible", new SpecialBoxProperty() { public Object get(Box b) { - for (Box cur = b; cur != null; cur = cur.parent) { if (cur.invisible) return Boolean.TRUE; } + for (Box cur = b; cur != null; cur = cur.parent) { + if ((cur.flags & INVISIBLE_FLAG) != 0) return Boolean.TRUE; } return Boolean.FALSE; } public void put(Box b, Object value) { - boolean newinvisible = stob(value); - if (newinvisible == b.invisible) return; - b.invisible = newinvisible; + if (stob(value) == ((b.flags & INVISIBLE_FLAG) != 0)) return; + if (stob(value)) b.flags |= INVISIBLE_FLAG; else b.flags &= ~INVISIBLE_FLAG; if (b.parent == null) { - if (b.surface != null) b.surface.setInvisible(newinvisible); + if (b.surface != null) b.surface.setInvisible((b.flags & INVISIBLE_FLAG) != 0); } else { b.dirty(); MARK_FOR_REFLOW_b_parent; @@ -1127,12 +1190,11 @@ public final class Box extends JS.Scope { }}); specialBoxProperties.put("absolute", new SpecialBoxProperty() { - public Object get(Box b) { return b.absolute ? Boolean.TRUE : Boolean.FALSE; } + public Object get(Box b) { return ((b.flags & ABSOLUTE_FLAG) != 0) ? Boolean.TRUE : Boolean.FALSE; } public void put(Box b, Object value) { - boolean newabsolute = stob(value); - if (newabsolute == b.absolute) return; - b.absolute = newabsolute; - if (b.absolute) { b.x = 0; b.y = 0; } + if (stob(value) == ((b.flags & ABSOLUTE_FLAG) != 0)) return; + if (stob(value)) b.flags |= ABSOLUTE_FLAG; else b.flags &= ~ABSOLUTE_FLAG; + if ((b.flags & ABSOLUTE_FLAG) != 0) { b.x = 0; b.y = 0; } if (b.parent != null) MARK_FOR_REFLOW_b_parent; } }); @@ -1147,8 +1209,8 @@ public final class Box extends JS.Scope { if ((b.image = ImageDecoder.getPicture(s)) == null) { if (Log.on) Log.logJS(Box.class, "unable to load image " + s); } else { - b.minwidth = b.maxwidth = b.image.getWidth(); - b.minheight = b.maxheight = b.image.getHeight(); + b.minwidth = b.image.getWidth(); + b.minheight = b.image.getHeight(); MARK_FOR_REFLOW_b; } } @@ -1198,7 +1260,8 @@ public final class Box extends JS.Scope { }); specialBoxProperties.put("mouseinside", new SpecialBoxProperty() { - public Object get(Box b) { return b.mouseinside ? Boolean.TRUE : Boolean.FALSE; } + public Object get(Box b) { + return ((b.flags & MOUSEINSIDE_FLAG) != 0) ? Boolean.TRUE : Boolean.FALSE; } }); specialBoxProperties.put("numchildren", new SpecialBoxProperty() { @@ -1302,8 +1365,8 @@ public final class Box extends JS.Scope { }); // these are all do-nothings; just to prevent space from getting taken up in the params Hash. - specialBoxProperties.put("KeyPressed", new SpecialBoxProperty()); // FIXME should cascade - specialBoxProperties.put("KeyReleased", new SpecialBoxProperty()); // FIXME should cascade + specialBoxProperties.put("KeyPressed", new SpecialBoxProperty()); + specialBoxProperties.put("KeyReleased", new SpecialBoxProperty()); specialBoxProperties.put("PosChange", new SpecialBoxProperty()); specialBoxProperties.put("SizeChange", new SpecialBoxProperty()); @@ -1353,46 +1416,46 @@ public final class Box extends JS.Scope { } }); - /* - // FIXME: need to be able to read this back + // FEATURE: this still isn't totally harmonious; when you createSurface, these aren't checked specialBoxProperties.put("titlebar", new SpecialBoxProperty() { - public void put(Box b, Object value) { surface.setTitleBarText(value.toString()); } - public Object get(Box b) { return b.ti; } + public void put(Box b, Object value) { + if (b.surface != null) b.surface.setTitleBarText(value.toString()); + b.put_("titlebar", value); + } + public Object get(Box b) { return b.get_("titlebar"); } }); - // FIXME: need to be able to read this back specialBoxProperties.put("icon", new SpecialBoxProperty() { public void put(Box b, Object value) { - Picture pic = Box.getPicture(value.toString()); - if (pic != null) surface.setIcon(pic); + b.put_("icon", value); + if (b.surface == null) return; + Picture pic = ImageDecoder.getPicture(value.toString()); + if (pic != null) b.surface.setIcon(pic); else if (Log.on) Log.log(this, "unable to load icon " + value); } - public Object get(Box b) { return b.id; } + public Object get(Box b) { return b.get_("icon"); } }); - */ } + } - - /** helper that converts a String to a boolean according to JavaScript coercion rules */ - public static boolean stob(Object o) { - if (o == null) return false; - return Boolean.TRUE.equals(o) || "true".equals(o); - } + /** helper that converts a String to a boolean according to JavaScript coercion rules */ + public static boolean stob(Object o) { + if (o == null) return false; + return Boolean.TRUE.equals(o) || "true".equals(o); + } + /** helper that converts a String to an int according to JavaScript coercion rules */ + public static int stoi(Object o) { + if (o == null) return 0; + if (o instanceof Integer) return ((Integer)o).intValue(); + + String s; + if (!(o instanceof String)) s = o.toString(); + else s = (String)o; + try { return Integer.parseInt(s.indexOf('.') == -1 ? s : s.substring(0, s.indexOf('.'))); } + catch (NumberFormatException e) { return 0; } } - /** helper that converts a String to an int according to JavaScript coercion rules */ - public static int stoi(Object o) { - if (o == null) return 0; - if (o instanceof Integer) return ((Integer)o).intValue(); - - String s; - if (!(o instanceof String)) s = o.toString(); - else s = (String)o; - - try { return Integer.parseInt(s.indexOf('.') == -1 ? s : s.substring(0, s.indexOf('.'))); } - catch (NumberFormatException e) { return 0; } - } }