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.

So....I 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")
<dspace:layout titlekey="jsp.dspace-admin.eperson-edit.title"
              navbar="admin"
              locbar="link"
              style="admin"
              parenttitlekey="jsp.administer"
              parentlink="/dspace-admin"
              nocache="true">

On the default header, I now have something that looks like this, which includes the search box and logged in status:
  <div id="container">
          <div class="pageBanner">
              <div id="vcubar">
<a href="http://www.vcu.edu/"; title="Virginia Commonwealth University"> <img src="/dspace/image/vcu_logo.jpg" alt="Virginia Commonwealth University Logo" border="0" /></a></div> <div id="logo"> <a href="<%= request.getContextPath()%>" title="VCU Dspace" id="vcudspace"></a> <div id="topnav"> <%-- Search Box --%> <form method="get" action="<%= request.getContextPath() %>/simple-search"> <input type="text" onfocus="clearDefault(this)" name="query" id="tequery" value="Search Dspace" size="18"/>&nbsp;<input type="submit" name="submit" value="<fmt:message key="jsp.layout.navbar-default.go"/>" />
                               </form>
                         </div>
           </div>
           <div id="utilnav">
               <%if (locbar) {%>
               <dspace:include page="/layout/location-bar.jsp" />
               <% }%>
               &nbsp;
<a href="<%= request.getContextPath() %>/mydspace"><fmt:message key="jsp.layout.navbar-default.users"/></a>
               <% if (user != null) { %>
| <span class="loggedIn"><fmt:message key="jsp.layout.navbar-default.loggedin"><fmt:param><%= navbarEmail %></fmt:param></fmt:message></span> (<a class="loggedIn" href="<%= request.getContextPath() %>/logout"><fmt:message key="jsp.layout.navbar-default.logout"/></a>) <% } %> </div>
</div> <!-- close pageBanner-->


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: [email protected]
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
[email protected]
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/dspace-tech

Reply via email to