album select, treeview causes IE window send to background

buut
buut's picture

Joined: 2003-06-18
Posts: 196
Posted: Thu, 2005-04-28 18:33

In the album select module, tree view.
The Expand and Collapse links will put the active browser window to hide behind other open windows (send to background). This only occurs if the other windows are NOT minimized.

I'am using
G2 core: 0.9.14 (28 april)
Browser: IE v6

 
mindless
mindless's picture

Joined: 2004-01-04
Posts: 8601
Posted: Thu, 2005-04-28 21:37

sorry, gonna have to file an IE bug for that one.. I looked through dTree.prototype.oAll in modules/albumselect/dtree.js and I don't see anything about raising or lowering windows... if you find anything in there that causes this let us know.

 
buut
buut's picture

Joined: 2003-06-18
Posts: 196
Posted: Thu, 2005-04-28 23:07

mindless,
I'll wil file it.

But it does no raising or lowering (which I understand as Maximize and Minimize), the window size stays the same. It is just that the other window is made 'active'.

This conclusion is based on:
- Having multiple windows maximized in 'Windows OS'
- Gallery window is the active window.
- clicking 'Expand or Collaps'
- an other window becomes the active window.
- when the new active window is minimized the gallery window appaers in the same size as it was.

 
mindless
mindless's picture

Joined: 2004-01-04
Posts: 8601
Posted: Fri, 2005-04-29 01:49

looks like you filed a bug against G2.. where's the bug filed against IE?

raise/lower is not maximize/minimize.
raise means make the window appear in front of all other windows.
lower means make the window appear behind all other windows.

 
edriss
edriss's picture

Joined: 2004-07-19
Posts: 74
Posted: Fri, 2005-04-29 02:47
mindless wrote:
looks like you filed a bug against G2.. where's the bug filed against IE?

raise/lower is not maximize/minimize.
raise means make the window appear in front of all other windows.
lower means make the window appear behind all other windows.

<EDIT> Sorry, i miss-understood the original poster's issue, but the below issue is still valid </EDIT>

Hi,
I can "second" this behaviour. But only in Firefox (1.0.2)
Let me try to explain in a different way:

Using Firefox:

1) If you click on and drag a link into the main window (the "stop sign" [ie invalid action sign]) will appear. Now use the scroll wheel:
Result: Page (given it's long enough) scrolls up/down.

2) If you click on an image and do the same as above, the scroll wheel allowes for page scrolling.

3) **ISSUE** Click on a (+) sign in the tree. Use the scroll wheel ---> The page does not scroll up and/or down.

Note that this works for me in IE 6.

My setup:
Gallery version = 2.0-beta-2+ core 0.9.14
PHP version = 4.3.11 isapi
Webserver = Microsoft-IIS/5.1
Database = mysqlt 5.0.4-beta-nt
Toolkits = ImageMagick, NetPBM
Operating system = Windows XP
Browser = Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.6) Gecko/20050317 Firefox/1.0.2

 
buut
buut's picture

Joined: 2003-06-18
Posts: 196
Posted: Fri, 2005-04-29 11:59

mindless,

Removing this.blur() twice in the albumtree.tpl did the trick for me. I don't know what the impact is of this change.

Orignal line
<a href="javascript: albumTree.openAll(); this.blur()">{g->text text="Expand"}</a>

new line
<a href="javascript: albumTree.openAll();">{g->text text="Expand"}</a>

So tell me that this isn't so bad for somebody who didn't write any code in about 6 a 7 Years. (and certainly does not have ANY experience with java or php) ;-)

 
mindless
mindless's picture

Joined: 2004-01-04
Posts: 8601
Posted: Fri, 2005-04-29 14:35

buut, nice work.. apparently "this" is different in IE between href and onclick.. I moved this.blur() to onclick and it seems ok.

edriss, unless you are able to find something specific in albumtree that we change to workaround that browser issue (as buut did for his issue) there isn't anything we can do.