Bug 79926 - Quanta automatically puts <fieldset /> when i try to type <fieldset>, <fieldset /> is not valid html/xhtml
Summary: Quanta automatically puts <fieldset /> when i try to type <fieldset>, <fields...
Status: RESOLVED FIXED
Alias: None
Product: quanta
Classification: Miscellaneous
Component: general (show other bugs)
Version: 3.2.1
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: András Manţia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-04-19 01:34 UTC by Ed Martin
Modified: 2004-05-08 16:10 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Martin 2004-04-19 01:34:44 UTC
Version:           3.2.1 (using KDE 3.2.1, compiled sources)
Compiler:          gcc version 3.3.3
OS:          Linux (i686) release 2.6.4

When I'm typing some HTML and try to do the fieldset tag Quanta thinks that the fieldset tag is like the img tag where it ends in /> and has no end tag, that is not valid HTML, it must have an end tag, when it does this its auto tag editing changes the name of the tag before it if i try to edit making it a lot of work to add the end tag, example

when i type this
<fieldset>
it ends up as this
<fieldset />
typing the </ closes the tag before the fieldset instead of the fieldset tag, trying to edit the name of that close tag renames the tag before the fieldset tag


the w3c validator says the <fieldset /> is invalid for all HTML and XHTML

and if this helps
the loaded plugins are:
cvs management
kfilereplace
xslt debugger

and when i tried this i was using the XHTML 1.0 transitional DTD
Comment 1 András Manţia 2004-05-08 16:10:27 UTC
CVS commit by amantia: 

<fieldset> is not a single tag.

CCMAIL: 79926-done@bugs.kde.org


  M +3 -0      ChangeLog   1.205.2.36
  M +1 -1      quanta/data/tags/xhtml/fieldset.tag   1.1.4.1
  M +1 -1      quanta/data/tags/xhtml-frameset/fieldset.tag   1.1.4.1
  M +1 -1      quanta/data/tags/xhtml-strict/fieldset.tag   1.1.4.1
  M +1 -1      quanta/data/tags/xhtml11/fieldset.tag   1.2.4.1
  M +0 -1      quanta/parser/node.cpp   1.39.2.7
  M +0 -2      quanta/parser/parser.cpp   1.159.2.17
  M +1 -1      quanta/parser/tag.cpp   1.52.2.7


--- quanta/ChangeLog  #1.205.2.35:1.205.2.36
@@ -16,4 +16,7 @@
         - remove the backup files in every case when the document was saved and don't warn
           about their presence on the next startup.
+        - don't crash when saving files while the structure tree is visible [#79803]
+        - fix a major memory leak, cause of many crashes and instability
+        - <fieldset> is not a single tag [#79926]
 
     - improvements:

--- quanta/quanta/data/tags/xhtml/fieldset.tag  #1.1:1.1.4.1
@@ -2,5 +2,5 @@
 <!DOCTYPE TAGS>
 <TAGS>
-<tag name="fieldset" hasScript="1" hasCore="1" hasI18n="1" single="1">
+<tag name="fieldset" hasScript="1" hasCore="1" hasI18n="1">
 
 <children>

--- quanta/quanta/data/tags/xhtml-frameset/fieldset.tag  #1.1:1.1.4.1
@@ -2,5 +2,5 @@
 <!DOCTYPE TAGS>
 <TAGS>
-<tag name="fieldset" hasScript="1" hasCore="1" hasI18n="1" single="1">
+<tag name="fieldset" hasScript="1" hasCore="1" hasI18n="1">
 
 <children>

--- quanta/quanta/data/tags/xhtml-strict/fieldset.tag  #1.1:1.1.4.1
@@ -2,5 +2,5 @@
 <!DOCTYPE TAGS>
 <TAGS>
-<tag name="fieldset" hasScript="1" hasCore="1" hasI18n="1" single="1">
+<tag name="fieldset" hasScript="1" hasCore="1" hasI18n="1">
 
 <children>

--- quanta/quanta/data/tags/xhtml11/fieldset.tag  #1.2:1.2.4.1
@@ -2,5 +2,5 @@
 <!DOCTYPE TAGS>
 <TAGS>
-<tag name="fieldset" hasScript="1" hasCore="1" single="1">
+<tag name="fieldset" hasScript="1" hasCore="1">
 
 <children>

--- quanta/quanta/parser/node.cpp  #1.39.2.6:1.39.2.7
@@ -177,5 +177,4 @@ void Node::setNodeValue(QString value)
   {
     tag = new Tag();
-    kdDebug(24000) << "node:176" << endl;
   }
   tag->setStr(value);

--- quanta/quanta/parser/parser.cpp  #1.159.2.16:1.159.2.17
@@ -280,6 +280,4 @@ Node *Parser::parseArea(int startLine, i
           //a trick here: replace the node's DTD with this one //Note: with the new SAParser, the top level nodes must be Tag::ScriptTag-s!
          // const DTDStruct *savedDTD = node->tag->dtd;
-         if (!dtd)
-           kdDebug(24000) << "parser.cpp:283: " << endl;
           node->tag->setDtd(dtd);
           node->tag->type = Tag::ScriptTag;

--- quanta/quanta/parser/tag.cpp  #1.52.2.6:1.52.2.7
@@ -38,5 +38,5 @@ Tag::Tag(const AreaStruct &area, Documen
   QString s = write->text(area);
   m_area = area;
-  assert(dtd);
+//  assert(dtd);
   m_dtd = dtd;
   if (doParse)