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.net                                                    cc 
                                                                           
                                                                   Subject 
             04/09/2007 12:18          Re: [Dspace-tech] eperson popup     
             PM                        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.

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: 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



-------------------------------------------------------------------------
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

Reply via email to