[Dspace-tech] eperson popup error

2007-04-03 Thread Susan Teague Rector
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

2007-04-09 Thread Farrell,Gabriel
> -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

2007-04-09 Thread susan rector

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

2007-04-09 Thread Glenn Bunton
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