From: Morten K. <mor...@ms...> - 2005-04-05 16:02:56
|
Minor changes, but very nice :) I have commited the changes and they will be included when plugin is released. Plugin is not updated on my homepage! Morten >From: Lukas Theussl <th...@tr...> >To: mav...@li... >Subject: [Maven-plugins-developer] proposed patch for maven-news-plugin >Date: Fri, 01 Apr 2005 17:04:26 +0000 > > >I would like to propose a few modifications to the maven-news-plugin that >was submitted recently by Morten Kristiansen. These modifications make the >plugin more flexible for the user. > >My motivations for these changes can be seen on this web page (which was >already generated by the new version of the plugin): >http://www.triumf.ca/people/theussl/JaxoDraw/Web/news-report.html > > > >The changes are: > >* introduced a maven.news.header property (Default="News") that can be >overridden by the user (there should be no hard-coded strings in .jsl >files). > >* replaced the "header" and "date" tags by just one "name" tag. This allows >the user to choose the format of the headers (I just wanted the date >without a text, others might prefer the date before the text, with a >different separator, etc). Note also that the xdoc plugin generates id's >for each sub/section from its "name" tag, which makes it possible to link >to a single news entry from other source documents, see eg the homepage of >my web page above. > >* add the possibility to include (optionally) a section (or whatever) >above/below the list of news entries. In my example above, this is a >"Current Status" section. > >* reorganized the structure of the news.xml source document like > ><news> > <entry name=""></entry> > ... ></news> > > >So the source code for my web page above looks like that: > ><body> > > <section name="Current Status (12.01.2005)"> > <p> > We are currently waiting for feedback ... > </p> > ... > </section> > > <news> > <entry name="07.02.2005"> > <p> > We have discovered quite a subtle bug ... > </p> > </entry> > ... > </news> > ></body> > > >A patch for these issues is attached. > >I think these are rather useful generalizations and I hope these changes >will be applied before the plugin gets published. > > > > > > ><< maven-news-plugin.patch >> |