This is a multi-part message in MIME format.
--------------F5713E889B90896C59279E58
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit


goran.hultgren at bluefish.se wrote:
> 
> In version 0.96 of "SM Package Loader" there are two buglets:
> 
> ...

A few more buglets/problems:

1. The current window color is still too light.  This might seem a bit picky, but with the default Squeak preferences (alternativeWindowLook on, etc.), the scrollbar is so faint that it's difficult to see against its background.  Actually, on a laptop LCD screen (which tends to distinguish poorly between light colors), it's pretty much impossible to see the scrollbar against its background... a significant problem, IMHO.

To fix this, I would go with a somewhat darker yellow color.  You could change SMLoader>>paneColor to return "Color yellow muchLighter duller", which would give you a beige color.  Or you could use "Color yellow lighter duller" for a golden yellow color.

2. A minor thing (since you're already making other tweaks :-) )... The new help link to the SqueakMap swiki page is cool, but I would separate the last sentence more and clarify it a bit more.  Put a couple CRs in front of it, and change it to something like: "Would you like to view more detailed help on the SqueakMap swiki page?"

3. Hm, I just noticed that the source code in SMLoader seems to have extra characters (probably LFs) embedded at the beginning of each line of code.  Might want to do a "remove line feeds" on your next release changeset.

Other than that, things are looking pretty good...

Attached is a changeset of fixlets for #1 and #2.

- Doug Way
  dway at riskmetrics.com
--------------F5713E889B90896C59279E58
Content-Type: application/x-gzip;
 name="SMLoader096Fixlets-dew.cs.gz"
Content-Transfer-Encoding: base64
Content-Disposition: inline;
 filename="SMLoader096Fixlets-dew.cs.gz"

H4sIAAAAAAAAAKVSTW/bMAw95+D/wGAHt0DrJE6/lsN2GFBgRYsekqGHogMUiY6FSKKnj3j5
96OcZEWx426iSD6+R77y3pOF5a+EYjuvaqAGZjN4SGYP9XRaw6sREUOE1Cl+LODT1efrmzcg
B/UdPMtIa/SHShFhVi+ubhfTa+hsOS6K8fLpkYTiAouxJRXuyS+g3CQN66SN0m5TQojCdvyr
sIfZdFLfTQa0DDVnkBZNV4zOApoGJLlGe8vFL2gkWYRIEFs80n8SHXRCbsUGwQxzKyhWnHbC
YsjKjtkAwiM0ZAz1qGC9hzPtmIcxHO3QB83yLr/AUfrx55zRvjd5HjfrAI4YxlN/AXtKHv5F
oKH4LyVuyeEBtBqIyRbldk2/eT8ugY5oQ15jLltTjHwYgzHDgyWlmz30rZbtu4yCU2VpDATE
ClZiiyBYOm2PKBYu+VJ8ypCX9UF9aKl3TFpHzaz3x01RF5k5V+yENmJt8pY8Y540KOzQqXx9
bmfC0ItwSqrDfjJbowcmpy6GaMWO5ZJloTHkfh0voDMoQv52UchYZNkisVF87shRGO4Klsmw
Vxg2Lw6KF0pGvQ9iE+w0u8eSzwwjV/MVsnHyoI/+CL3eaia2wa/lORQj3ax8Ylu/LiXbkJ1K
LPDZ/fCGXdbG2C0mE6udo76Sstrw7WRboUqTA7VJfVvflCACN7yN4f89P5+y5zvh8BsZ8sXo
JwwP2GM2K9gk20e9aSPjq8Ru88PQP36Mc1XFAwAA
--------------F5713E889B90896C59279E58--