Ticket #3202 (closed defect: fixed)

Opened 6 months ago

Last modified 4 months ago

svn urls has bogus first line for table

Reported by: k0s Assigned to: k0s
Priority: normal Component: SvnUrlsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

For any subdirectory, the first entry .., for the parent directory contains a bogus svn url link. The link points to the right place, but it is in the "Last Changes" column and should either be moved to the right place or removed

see (e.g.)

http://trac.openplans.org/trac/browser/23Screens/wp-admin

Attachments

Change History

(follow-up: ↓ 3 ) 07/30/08 01:43:18 changed by extemporalgenome@gmail.com

This is because the .. column has a colspan attribute hard coded to be 5. The best method would be to submit a patch to vanilla trac that looks at the number of columns in the header, and sets the colspan dynamically. From there, it would be a simple fix (via negation in an xpath expression) to have SvnUrlsPlugin avoid transforming that row.

08/04/08 07:38:20 changed by k0s

  • status changed from new to closed.
  • resolution set to fixed.

(In [4112]) unmark the colspan attribute from the "../" link; fixes #3202

(in reply to: ↑ 1 ) 08/04/08 07:40:04 changed by k0s

Replying to extemporalgenome@gmail.com:

This is because the .. column has a colspan attribute hard coded to be 5. The best method would be to submit a patch to vanilla trac that looks at the number of columns in the header, and sets the colspan dynamically. From there, it would be a simple fix (via negation in an xpath expression) to have SvnUrlsPlugin avoid transforming that row.

i did this by just unmarking the colspan attribute on this entry (see r4112); ideally, the colspan should be reset on the subsequent entry, but i punted on this for now.

Add/Change #3202 (svn urls has bogus first line for table)




Change Properties
Action