Opened 16 years ago
Closed 16 years ago
#7634 closed defect (duplicate)
Browser folder expansion fails on custom template
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | version control/browser | Version: | 0.11.1 |
Severity: | minor | Keywords: | browser custom template expand |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
If a template is created that places the main contents into a table cell, the javascript that expands a folder in the browser view inserts the table rows into the topmost table rather than the browser table. I have attached an example theme.html file that exposes this problem. I have hacked a fix that works by changing the following line at line 43 of expand_dir.js:
var tr = expander.parents("tr");
to
var tdtemp = expander.parents("td.name") var tr = tdtmp.parent();
This problem could also easily be solved by adding a class to the browser table rows so it can be referenced directly.
Attachments (1)
Change History (2)
by , 16 years ago
Attachment: | theme.html added |
---|
comment:1 by , 16 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
Thanks for the report. This was already requested earlier (#7559), so it's about time to apply the patch ;-)