After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 792122 - XHTML pages are broken several ways
XHTML pages are broken several ways
Status: RESOLVED OBSOLETE
Product: doxygen
Classification: Other
Component: general
1.8.14
Other Windows
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
[moved_to_github]
Depends on:
Blocks:
 
 
Reported: 2018-01-02 10:46 UTC by Jeroen Van Antwerpen
Modified: 2018-07-30 10:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Reproduction issue menu.js and annotated.xhtml (101.72 KB, application/x-rar)
2018-01-03 10:11 UTC, Jeroen Van Antwerpen
Details

Description Jeroen Van Antwerpen 2018-01-02 10:46:09 UTC
Upgrading from 1.8.11 to 1.8.14 gave us several issues we had to work around by patching the generated output. This because some important pages suddenly became invalid XML.

Important config options:
HTML_FILE_EXTENSION    = .xhtml
SEARCHENGINE           = YES
SERVER_BASED_SEARCH    = YES
EXTERNAL_SEARCH        = YES



Search.xhtml

Contains:
            <input type="text" id="MSearchField" name="query" value="" size="20" accesskey="S" onfocus="searchBox.OnSearchFieldFocus(true)" onblur="searchBox.OnSearchFieldFocus(false)"/>
            </form>
          </div><div class="right"></div>
        </div>
      </li>
    </ul>
  </div>

None of these ending tags are ever started.
If we explicitly update to start them, we have 2 search boxes of which 1 is on a really strange place. (This one)


menu.js

Contains <form ...><input ...></form>
Changed this to <form ...><input .../></form>

With this we were able to view the main page.

On several pages, we have a table in markdown, this seems to generate:
<tr class="markdownTableBody" class="markdownTableRowEven">
however, firefox/IE only accept
<tr class="markdownTableBody,markdownTableRowEven">


Lastly, the annotated.xhtml page is broken to (this one also in doxygen 1.8.11),
whenever someone adds complicated elements picked up by auto-brief.
(QT_AUTOBRIEF           = YES)

This is especially the case for comments starting with
/*!
  * Text
 */
or with
Variant 1
=========


Please let me know if you extra information to reproduce this, however, note that this might take a while since it takes several hours to run.
Comment 1 Dimitri van Heesch 2018-01-02 20:13:12 UTC
Hi Jeroen,

Can you please attach a self contained example (source+config file in a tar or zip) that allows me to reproduce the problem myself?
Comment 2 Jeroen Van Antwerpen 2018-01-03 10:11:13 UTC
Created attachment 366241 [details]
Reproduction issue menu.js and annotated.xhtml

To reproduce, you can open the files in firefox and check the console
Comment 3 Jeroen Van Antwerpen 2018-01-03 10:12:51 UTC
Note, going to http://localhost/bug/search.xhtml?query=other will give you the search.xhtml problem
Comment 4 albert 2018-03-04 12:15:42 UTC
We have here multiple problems listed in 1 report:
- search problem (menu.js)
- problem with markdown and tables, please provide separate example
- problem with "Text", I was not able to reproduce it, please provide separate example
- problem with "Variant 1" and "======", this has to do with the way the comment is processed and the moment that the markdown processing is done. This can be overcome by placing the \details command before the "Variant 1". For this last problem I've just pushed a proposed patch to github (pull request 674, https://github.com/doxygen/doxygen/pull/674).
Comment 5 albert 2018-04-22 12:30:27 UTC
Code has been integrated in master on github.
Comment 6 Marcin Kasperski 2018-05-16 10:10:47 UTC
The fix introduced some problems, see bug 796110
Comment 7 albert 2018-05-20 13:41:54 UTC
Fix gave some problems wit h[TOC], this has been fixed in the proposed patch (pull request 735, https://github.com/doxygen/doxygen/pull/735).
Comment 8 albert 2018-06-10 10:00:42 UTC
2018/06/10 New patch (pull request 735) has been integrated on github
Comment 9 André Klapper 2018-07-30 10:43:16 UTC
As discussed in https://github.com/doxygen/doxygen/pull/734 , Doxygen has moved its issue tracking to 

   https://github.com/doxygen/doxygen/issues

All Doxygen tickets in GNOME Bugzilla have been migrated to Github. You can subscribe and participate in the new ticket in Github. You can find the corresponding Github ticket by searching for its Bugzilla ID (number) in Github.

Hence I am closing this GNOME Bugzilla ticket.
Please use the corresponding ticket in Github instead. Thanks a lot!