[Htmlparser-developer] Re: More patches for HTMLParser
Brought to you by:
derrickoswald
From: Ian M. <ian...@gm...> - 2005-11-01 10:15:56
|
This has been checked into CVS. On 10/27/05, Ian Macfarlane <ian...@gm...> wrote: > Forwarded again as SourceForge blocked the zip attachments on the previou= s mail. > > ---------- Forwarded message ---------- > From: Ian Macfarlane <ian...@gm...> > Date: Oct 27, 2005 1:35 PM > Subject: More patches for HTMLParser > To: Der...@ro... > Cc: htm...@li... > > > Dear Derrick, > > I've modified the custom definitions I had to write for p tags and > definition-list-related tags (dl,dd,dt). > > Notes: > > - ParagraphTag: There are a lot of tags listed as closing the P tag - > these all appear to be correct however - I have tested it using the > splitting_test.html file attached (sample testing div splitting p) and > Firefox's DOM inspector, and went through every tag in the HTML 4 > specification. All the listed tag names caused the P tag to close in > Firefox. > > - Definition list stuff: I did not simply add these as entries to the > Bullet and BulletList classes, as from testing (a few examples shown > in the attached list_test.html) showed that they in fact behaved > differently to normal lists (for example a dt closes a dd, and vice > versa, but a li does not close either of these, nor do they close an > li. > > A question about PrototypicalNodeFactory, is there a reason that the > div, span, body, head and html tag registration in method registerTags > () are at the end rather than in alphabetical order like the other > tags? > > Kind regards, > > Ian Macfarlane > > PS: I've tried the following command to add a file to CVS: > > cvs -d :ext:ian...@cv...:/cvsroot/htmlparser > add src/org/htmlparser/tags/ParagraphTag.java > > And I get this response: > > cvs add: in directory .: > cvs [add aborted]: there is no version here; do 'cvs checkout' first > > I'm new to using CVS, so I'm not entirely sure what I'm doing wrong > here. Do you have any suggestions? > > > |