Chart feature broken (in 1.9.4 dev. version)

Issue #178 resolved
Christopher Kramer created an issue

Originally reported on Google Code with ID 178

The chart feature is broken in the current development version.
The problem is that the JS does document.getElementById('content') and there is no
element with id=content (anylonger).

This issue was introduced with revision #279 (which aimed at issue #94).

If I see this correctly, the element that is meant here (that was <div id="content">
before) is now the <td class="right_td">.

Maybe we should use names like "content" again instead of "right_td". And ids instead
of classes for stuff that is unique (there cannot be a 2nd right_td).

Reported by crazy4chrissi on 2013-02-20 14:16:39

Comments (2)

  1. Christopher Kramer reporter
    Hmm. Maybe introducing "content" again is not a good idea as it would break all themes
    currently available. They all contain something like:
    #content { overflow:hidden; padding-left:10px; }
    
    With the current td-solution, we don't want this anylonger.
    

    Reported by crazy4chrissi on 2013-02-20 14:23:47

  2. Christopher Kramer reporter
    This issue was closed by revision r341.
    

    Reported by crazy4chrissi on 2013-02-20 14:27:49 - Status changed: Fixed

  3. Log in to comment