mistake in Flowable Toolbar

[gocher]

The toolbar will re-flow when you resize the window, but not accurate.

[gocher]

look at this

[gocher]

If a button was/is active the button gets a border, and this border changes the size of the table/div!

[mokhet]

this "patch" fix the problem here (add solid in the border declaration in the skin :) ​example

this is not happening (at least on my side) with original colors.

Index: skins/blue-look/skin.css
===================================================================
--- skins/blue-look/skin.css	(revision 187)
+++ skins/blue-look/skin.css	(working copy)
@@ -19,7 +19,7 @@
   width:20px;
   height:20px;
   padding:0px;
-  border:1px #f7f8fd;
+  border:1px solid #f7f8fd;
 }
 
 .buttonImageContainer

[gocher]

if the background is one color it's ok, but if it's a gradient?

[gogo]

changeset:194

[gogo]

And changeset:195

[anonymous]

this is still happening in yesterday's download of nightly build 20050727.
i tried with mozilla 1.7.3 (Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.3) Gecko/20040910)

at the same time it doesn't happen on firefox 1.0.6 on the same OSX plateform.

[gogo]

I think the trouble is that we cannot specify a height on the selects, which stuffs things up. the reason we can't specify a height is that it triggers a bug in Gecko whereby the selects will "grow" horizontally as they are reflowed.

[jkronika@…]

I haven't tracked Xinha for long, but I do work with CSS a lot. Perhaps fiddling with the font-size for the SELECTs would help? It might require some browser sniffing to get it to work properly on all browsers/platforms, but this is a potential solution. For instance:

var selects = document.getElementById(TOOLBAR_AREA_ID_HERE).getElementsByTagName('select');
var agent = navigator.userAgent.toLowerCase(),
    is_mac = agent.indexOf('mac') > -1,
    is_win = agent.indexOf('win') > -1,
    is_ff = agent.indexOf('firefox') > -1,
    is_saf = agent.indexOf('safari') > -1,
    is_op = agent.indexOf('opera') > -1,
    is_ie = agent.indexOf('msie') > -1 && !is_op,
    is_moz = agent.indexOf('mozilla') > -1 && !is_ff && !is_saf;
for (var i = 0; i < selects.length; i++) {
 if(is_mac) {
  if(is_ff) selects[i].style.fontSize = 12px;
  else if(is_saf) ...
  ...
 } else if(is_win) {
  ...
 } else { ... }
}

[gocher]

fixed at revision: 321