[Dspace-tech] eperson popup error
Hi all, I did a bad thing :). I moved my search box to the upper right corner of the UI. The move of the form has now broken my eperson pop-ups Is there a patch/fix for this? I did find something about changing the javascript from to window.document.forms[1] from window.document.forms[0], however this didn't seem to work. Ideas? thank you! susan - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ___ DSpace-tech mailing list DSpace-tech@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dspace-tech
Re: [Dspace-tech] eperson popup error
> -Original Message- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf > Of Susan Teague Rector > Sent: Tuesday, April 03, 2007 2:11 PM > To: dspace-tech@lists.sourceforge.net > Subject: [Dspace-tech] eperson popup error > > Hi all, > > I did a bad thing :). > I moved my search box to the upper right corner of the UI. > The move of the form has now broken my eperson pop-ups > > Is there a patch/fix for this? I did find something about > changing the javascript from to window.document.forms[1] from > window.document.forms[0], however this didn't seem to work. > > Ideas? > > thank you! > > susan > Hi Susan! Have you made any headway on this issue? I'm looking at our install for reference, but I'm not sure what you mean by "eperson pop-ups". Gabe -- Gabriel Farrell Library Systems Developer Hagerty Library Drexel University [EMAIL PROTECTED] +1 215 895 1871 - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ___ DSpace-tech mailing list DSpace-tech@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dspace-tech
Re: [Dspace-tech] eperson popup error
Hi Gabe, Here's my hack that works! :) The eperson popups are on the admin pages. When you log into the admin portion of the app and click on E-people, click on 'Select e-person' and a page will pop-up. That pop-up window will not function properly if you've put the search box in the header as the search box become the first form on the page, thus breaking the Javascript. SoI created a new header for the Admin pages that doesn't include the search box so I wouldn't have to change all the Javascript. It's not the most elegant solution because I need to touch each admin jsp. However, from a usability standpoint for our administrators, it's a bit easier to figure out that you're in the admin screens when the header is different from the rest of the app. In each admin page, I have changed the header to point to /layout/admin-header.jsp (style="admin") On the default header, I now have something that looks like this, which includes the search box and logged in status: http://www.vcu.edu/"; title="Virginia Commonwealth University"> <%-- Search Box --%> onfocus="clearDefault(this)" name="query" id="tequery" value="Search Dspace" size="18"/> value="" /> <%if (locbar) {%> <% }%> <% if (user != null) { %> | key="jsp.layout.navbar-default.loggedin"><%= navbarEmail %> () <% } %> Then, on the admin header, I simply take out the search box div and just put 'Dspace admin' in it's place. The Javascript works fine for the E-people and E-group pages. Hope this makes sense! Susan Susan Rector Web Applications Manager Virginia Commonwealth University Libraries [EMAIL PROTECTED] Farrell,Gabriel wrote: -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Susan Teague Rector Sent: Tuesday, April 03, 2007 2:11 PM To: dspace-tech@lists.sourceforge.net Subject: [Dspace-tech] eperson popup error Hi all, I did a bad thing :). I moved my search box to the upper right corner of the UI. The move of the form has now broken my eperson pop-ups Is there a patch/fix for this? I did find something about changing the javascript from to window.document.forms[1] from window.document.forms[0], however this didn't seem to work. Ideas? thank you! susan Hi Susan! Have you made any headway on this issue? I'm looking at our install for reference, but I'm not sure what you mean by "eperson pop-ups". Gabe -- Gabriel Farrell Library Systems Developer Hagerty Library Drexel University [EMAIL PROTECTED] +1 215 895 1871 - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV ___ DSpace-tech mailing list DSpace-tech@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dspace-tech - Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV___ DSpace-tech mailing list DSpace-tech@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dspace-tech
Re: [Dspace-tech] eperson popup error
Susan and Gabe, Just to concur. I did the same thing here. We had a navigation bar across the top where I had put the search box and broken all the pop-ups in the admin pages. Someone (I forget who to give the credit to - sorry) mentioned about the pop-ups having to be the first form. After dickering with the pages a bit to see if I could overcome that I gave up and just removed the search box from the admin header and changed the jsps to use the edited one for admin use. It seems to work ok. == Mr. Glenn Bunton Head of Systems Development Old Dominion University Libraries Norfolk, Virginia 23529 [EMAIL PROTECTED] (757) 683-5952 === susan rector <[EMAIL PROTECTED] > To Sent by: "Farrell,Gabriel" dspace-tech-bounc <[EMAIL PROTECTED]>, [EMAIL PROTECTED] dspace-tech@lists.sourceforge.net rge.netcc Subject 04/09/2007 12:18 Re: [Dspace-tech] eperson popup PMerror Hi Gabe, Here's my hack that works! :) The eperson popups are on the admin pages. When you log into the admin portion of the app and click on E-people, click on 'Select e-person' and a page will pop-up. That pop-up window will not function properly if you've put the search box in the header as the search box become the first form on the page, thus breaking the Javascript. SoI created a new header for the Admin pages that doesn't include the search box so I wouldn't have to change all the Javascript. It's not the most elegant solution because I need to touch each admin jsp. However, from a usability standpoint for our administrators, it's a bit easier to figure out that you're in the admin screens when the header is different from the rest of the app. In each admin page, I have changed the header to point to /layout/admin-header.jsp (style="admin") On the default header, I now have something that looks like this, which includes the search box and logged in status: http://www.vcu.edu/"; title="Virginia Commonwealth University"> <%-- Search Box --%> " /> <%if (locbar) {%> <% }%> <% if (user != null) { %> | <%= navbarEmail %> () <% } %> Then, on the admin header, I simply take out the search box div and just put 'Dspace admin' in it's place. The Javascript works fine for the E-people and E-group pages. Hope this makes sense! Susan Susan Rector Web Applications Manager Virginia Commonwealth University Libraries [EMAIL PROTECTED] Farrell,Gabriel wrote: -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Susan Teague Rector Sent: Tuesday, April 03, 2007 2:11 PM To: dspace-tech@lists.sourceforge.net Subject: [Dspace-tech] eperson popup error Hi all, I did a bad thing :). I moved my search box to the upper right corner of the UI. The move of the form has now broken my eperson pop-ups Is there a patch/fix for this? I did find something about changing the javascript from to window.document.forms[1] from window.document.forms[0], however this didn't seem to work. Ideas? thank you! susan Hi Susan! H