Edgewall Software
Modify

Opened 14 years ago

Closed 14 years ago

#7634 closed defect (duplicate)

Browser folder expansion fails on custom template

Reported by: pdoup@… 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)

theme.html (3.4 KB ) - added by pdoup@… 14 years ago.

Download all attachments as: .zip

Change History (2)

by pdoup@…, 14 years ago

Attachment: theme.html added

comment:1 by Christian Boos, 14 years ago

Resolution: duplicate
Status: newclosed

Thanks for the report. This was already requested earlier (#7559), so it's about time to apply the patch ;-)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The ticket will remain with no owner.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none) to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.