I am sorry. It was a direct email from Matt. Here is the email that I saved in my folder:
-----Original Message----- From: Matt Raible [SMTP:[EMAIL PROTECTED] Sent: Wednesday, February 04, 2004 1:32 PM To: Pingili, Madhupal Subject: RE: [ 873930 ] TabbedMenuDisplayer doesn't support "target" attribute I wouldn't truts SF's CVS. Here's the latest snapshot. Matt > -----Original Message----- > From: Pingili, Madhupal [mailto:[EMAIL PROTECTED] > Sent: Wednesday, February 04, 2004 10:56 AM > To: 'Matt Raible' > Subject: RE: [ 873930 ] TabbedMenuDisplayer doesn't support > "target" attribute > > > Hi Matt, > I have just browsed CVS and did not find any comments about > changes. And also, last change made was 4 months back to > TabbedMenuDisplayer.java file. Is there any other place to > look for? Also, how do I download binary version(2.0) that > contains these changes? I have never downloaded from CVS. Any > help is appreciated. > > Thanks > Reddy Pingili > > > -----Original Message----- > > From: Matt Raible [SMTP:[EMAIL PROTECTED] > > Sent: Wednesday, February 04, 2004 12:19 PM > > To: Pingili, Madhupal > > Subject: RE: [ 873930 ] TabbedMenuDisplayer doesn't > support "target" > > attribute > > > > I've already fixed this in CVS - you might want to verify > that the fix > > works. If so, I'll close this issue. > > > > Matt > > > > > -----Original Message----- > > > From: Pingili, Madhupal [mailto:[EMAIL PROTECTED] > > > Sent: Wednesday, February 04, 2004 9:34 AM > > > To: '[EMAIL PROTECTED]' > > > Subject: [ 873930 ] TabbedMenuDisplayer doesn't support > > > "target" attribute > > > > > > > > > Hi Matt, > > > I am working on resolving following issue that I created on > > > http://struts-menu.sf.net/ . [ 873930 ] TabbedMenuDisplayer > > > doesn't support "target" attribute > > > > > > I am planning to change class > > > net.sf.navigator.displayer.TabbedMenuDisplayer > > > > > > method displayComponents() as below: > > > get target and change all print statements to pass this > > > target. And change DisplayStrings.properties as follows: > > > #tabbedmenu HTML tmd.menu.tab=<a href="{0}" title="{1}" > > > target="{2}">{3}</a>\n\t<ul class="submenu">\n > > > tmd.menu.item=\t<li><a href="{0}" title="{1}" > > > target="{2}">{3}</a></li> > > > > > > After completing these changes, what is the best possible way to > > > compile all the classes and build struts-menu-2.0.jar and test it? > > > > > > Do you have a Ant build.xml and a list of dependecies that I > > > should download? > > > > > > OR if I just email you these changes(one Java class and > > > properties files), can you > > > build a struts-menu-2.0.jar file and email to me, so that I > > > can test it here? > > > > > > Thank you, > > > Reddy Pingili > > > > > > > -----Original Message----- > > > > From: Matt Raible [SMTP:[EMAIL PROTECTED] > > > > Sent: Wednesday, January 07, 2004 11:27 AM > > > > To: [EMAIL PROTECTED] > > > > Subject: Re: [struts-menu] A question on struts menus > > > and frames... > > > > > > > > It doesn't look like this Displayer supports a "target" > attribute. > > > > You > > > > can enter this as a bug if you'd like to see it on there. > > > If you can, > > > > you could use the VelocityDisplayer with the tabs.html > > > template - then > > > > you'll have full control over the HTML it produces. > > > > > > > > HTH, > > > > > > > > Matt > > > > > > > > On Jan 7, 2004, at 8:45 AM, Pingili, Madhupal wrote: > > > > > > > > > I am using TabbedMenuDisplayer as follows: > > > > > > > > > > <Displayer name="TabbedMenu" > > > > > > > > > > type="net.sf.navigator.displayer.TabbedMenuDisplayer"/> > > > > > > > > > > JSP page contents: > > > > > > > > > > <menu:useMenuDisplayer name="TabbedMenu" > > > > > bundle="org.apache.struts.action.MESSAGE"> > > > > > <menu:displayMenu name="<%=menu%>"/> > > > > > </menu:useMenuDisplayer> > > > > > > > > > > Thanks > > > > > Reddy Pingili > > > > > > > > > >> -----Original Message----- > > > > >> From: Matt Raible [SMTP:[EMAIL PROTECTED] > > > > >> Sent: Wednesday, January 07, 2004 10:36 AM > > > > >> To: [EMAIL PROTECTED] > > > > >> Subject: Re: [struts-menu] A question on struts menus > > > and frames... > > > > >> > > > > >> Which displayer are you using? > > > > >> > > > > >> Matt > > > > >> > > > > >> On Jan 7, 2004, at 8:06 AM, Pingili, Madhupal wrote: > > > > >> > > > > >>> Hi Matt, > > > > >>> I have tested with static HTML and it replaces > whole body of > > > > >>> the window with target link results with the > following HTML in > > > header (top) > > > > >>> frame: > > > > >>> <a href="Help.html#Login" target="_top">Help</a> > > > > >>> OR > > > > >>> <a href="Help.html#Login" target="_parent">Help</a> > > > > >>> > > > > >>> But, When I put target="_parent" OR target="_top" in > > > > >>> menu-config.xml for menu items, it is not added in > HTML like > > > > >>> below: > > > > >>> > > > > >>> <li><a href="/docQuery/searchDocs.do" title="Advanced > > > > >>> Search">Advanced Search</a></li> > > > > >>> > > > > >>> So, If Struts-menu can add that target="_top" as part > > > of link like > > > > >>> below: > > > > >>> <li><a href="/loanQuery/searchDocs.do" target="_top" > > > > >>> title="Advanced Search">Advanced Search</a></li> that > > > will solve > > > > >>> my problem. > > > > >>> > > > > >>> Thanks > > > > >>> Reddy Pingili > > > > > > -----Original Message----- > From: Wendy Smoak [SMTP:[EMAIL PROTECTED] > Sent: Thursday, March 11, 2004 3:04 PM > To: [EMAIL PROTECTED] > Subject: Re: [struts-menu] Need the tabbed menus to have an onclick > attribute > > From: "Pingili, Madhupal" <[EMAIL PROTECTED]> > > I have added "target" attribute to TabbedMenuDisplayer. > > Please see the thread with subject > > "[ 873930 ] TabbedMenuDisplayer doesn't support "target" attribute" . > > You will see the details of changes required to add an attribute. > > If you need source code, let me know. > > Sorry, I'm confused. (And, embarrassingly enough, my mailbox was full, so > if there was a reply prior to this one I've missed it.) > > I can't find a thread with that title on mail-archive.com in > struts-menu-user or struts-menu-dev, but I did find this one: > http://www.mail-archive.com/[EMAIL PROTECTED]/msg00037.html > > Are you saying that you've patched the source code locally to allow a > 'target' attribute? Have your changes been committed? I don't mind using > nightly builds, but I'm unsure about maintaining a patched version > locally. > Is that what I'd have to do to get an 'onclick' attribute, or have you > already fixed that, too? > > Thanks, > -- > Wendy Smoak > > > > ------------------------------------------------------- > This SF.Net email is sponsored by: IBM Linux Tutorials > Free Linux tutorial presented by Daniel Robbins, President and CEO of > GenToo technologies. Learn everything from fundamentals to system > administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click > _______________________________________________ > struts-menu-user mailing list > [EMAIL PROTECTED] > https://lists.sf.net/lists/listinfo/struts-menu-user ------------------------------------------------------- This SF.Net email is sponsored by: IBM Linux Tutorials Free Linux tutorial presented by Daniel Robbins, President and CEO of GenToo technologies. Learn everything from fundamentals to system administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click _______________________________________________ struts-menu-user mailing list [EMAIL PROTECTED] https://lists.sf.net/lists/listinfo/struts-menu-user
