Opened 17 years ago
Closed 17 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 , 17 years ago
| Attachment: | theme.html added | 
|---|
comment:1 by , 17 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 ;-)