Edgewall Software

Opened 7 years ago

Last modified 4 years ago

#12752 closed defect

Broken layout of TicketQuery macro in grouped table mode — at Initial Version

Reported by: Christian Boos Owned by:
Priority: normal Milestone: 1.3.4
Component: query system Version: 1.3dev
Severity: normal Keywords: layout
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Looking at TracDev/ReleaseNotes/1.2#DetailedUserVisibleChanges, we see that the first <h2> for the Component: admin/console group doesn't have the same positioning as the others.

This is not so surprising considering the generated HTML:

<h2 class="section" id="DetailedUserVisibleChanges">Trac 1.2: Detailed User Visible Changes</h2>
<p>
</p>

<div>


    <h2 class="report-result" >
Component: admin/console     <span class="numrows">
      (2 matches)
    </span>
    </h2>


  <table class="listing tickets">
    <thead>
            <tr class="trac-columns">
        <th class="id asc">
          <a title="Sort by Ticket (descending)"
             href="/query?status=closed&amp;milestone=1.2&amp;milestone=1.1.1&amp;milestone=1.1.2&amp;milestone=1.1.3&amp;milestone=1.1.4&amp;milestone=1.1.5&amp;milestone=1.1.6&amp;severity=!trivial&amp;severity=!minor&amp;changelog=!&amp;group=component&amp;max=0&amp;col=id&amp;col=changelog&amp;desc=1&amp;order=id">Ticket</a>
        </th>
        <th class="changelog">
          <a title="Sort by Release Notes (ascending)"
             href="/query?status=closed&amp;milestone=1.2&amp;milestone=1.1.1&amp;milestone=1.1.2&amp;milestone=1.1.3&amp;milestone=1.1.4&amp;milestone=1.1.5&amp;milestone=1.1.6&amp;severity=!trivial&amp;severity=!minor&amp;changelog=!&amp;group=component&amp;max=0&amp;col=id&amp;col=changelog&amp;order=changelog">Release Notes</a>
        </th>
      </tr>

    </thead>

    <tbody>
      <tr class="odd prio3">
        <td class="id"><a href="/ticket/11333" title="View ticket" class="closed">#11333</a></td>
        <td class="changelog">
          <p>
The <a class="wiki" href="/wiki/TracAdmin">TracAdmin</a> <code>initenv</code> command accepts an optional parameter <code>--config</code>, the path to a file containing a configuration to be used when creating the environment.
</p>

        </td>
      </tr>      <tr class="even prio3">
        <td class="id"><a href="/ticket/11938" title="View ticket" class="closed">#11938</a></td>
        <td class="changelog">
          <p>
Improved error message when adding and removing an invalid action with <a class="wiki" href="/wiki/TracAdmin">TracAdmin</a>.
</p>

        </td>
      </tr>      </tbody>
    <tbody>
      <tr class="trac-group">
        <th colspan="2">
            <h2 class="report-result" >
Component: admin/web     <span class="numrows">
      (7 matches)
    </span>
    </h2>

        </th>
      </tr>
            <tr class="trac-columns">
        <th class="id asc">
          <a title="Sort by Ticket (descending)"
             href="/query?status=closed&amp;milestone=1.2&amp;milestone=1.1.1&amp;milestone=1.1.2&amp;milestone=1.1.3&amp;milestone=1.1.4&amp;milestone=1.1.5&amp;milestone=1.1.6&amp;severity=!trivial&amp;severity=!minor&amp;changelog=!&amp;group=component&amp;max=0&amp;col=id&amp;col=changelog&amp;desc=1&amp;order=id">Ticket</a>
        </th>
        <th class="changelog">
          <a title="Sort by Release Notes (ascending)"
             href="/query?status=closed&amp;milestone=1.2&amp;milestone=1.1.1&amp;milestone=1.1.2&amp;milestone=1.1.3&amp;milestone=1.1.4&amp;milestone=1.1.5&amp;milestone=1.1.6&amp;severity=!trivial&amp;severity=!minor&amp;changelog=!&amp;group=component&amp;max=0&amp;col=id&amp;col=changelog&amp;order=changelog">Release Notes</a>
        </th>
      </tr>

    </tbody>

IIRC, this odd output (already there on 1.2.x and probably from the start) was motivated by the need to define a correct <thead> with only the column headings.

So we either need to fix the CSS to harmonize this (the .wikipage table h1, .wikipage table h2, .wikipage table h3 { margin-left: -2rem; ... is the problem), or come with a better HTML output.

I don't have a solution in mind yet.

Change History (0)

Note: See TracTickets for help on using tickets.