Home My Page Projects FusionForge
Summary Activity Forums Tracker Lists News SCM Files Mediawiki Hudson/Jenkins

[#670] docman: list of documents has height=0 when there's a pending document

Date:
2014-05-05 12:17
Priority:
3
State:
Closed
Submitted by:
Sylvain Beucler (beuc-inria)
Assigned to:
Sylvain Beucler (beuc-inria)
Target Release:
5.3.1
Found in Version:
5.3
Severity:
normal
Resolution:
Fixed
Summary:
docman: list of documents has height=0 when there's a pending document

Detailed description
The docman home for a project, which should list the documents, is currently hidden (height=0px) if there's a pending document ($foundFiles == "01").

The presence of a pending document generates an hidden 'editfile' div which is apparently used to compute said height, whie it's usually not generated at all.

We need to fix this so that the height of the list of documents is correcty computed in all cases.
Message  ↓
Date: 2014-05-13 08:42
Sender: Sylvain Beucler

The JS comment is not a fix in itself, it's a bug report.
If everything worked with the "faulty" JS, I expect that fixing the faulty JS will have a bad side effect.

Date: 2014-05-12 16:53
Sender: Franck Villaume

I'm fine with the fix.
Thanks for spotting it.
I commited the js fix too (0a5f5b83da48e32cc4419eef886647aae0216275)
I set the target release to 5.3.1

Date: 2014-05-05 13:08
Sender: Sylvain Beucler

Fix for review at:
https://fusionforge.org/plugins/scmgit/cgi-bin/gitweb.cgi?p=fusionforge/fusionforge.git;a=commitdiff;h=ac8637700c88cdbf23c31659a075bfbb7f85ab1f;hp=e0fa55ba0086f8077da66cbde536c9205dd5dbf5

Date: 2014-05-05 12:48
Sender: Sylvain Beucler

Note: in "if (jQuery('#editFile'))", jQuery('#editFile') is always true (it's a valid selector object).
jQuery('#editFile').length isn't.

Field Old Value Date By
status_idOpen2014-05-12 16:53nerville
close_dateNone2014-05-12 16:53nerville
Target Release5.32014-05-12 16:53nerville
SeverityNone2014-05-12 16:53nerville
ResolutionAwaiting Response2014-05-12 16:53nerville
assigned_tonone2014-05-05 13:08beuc-inria
ResolutionNone2014-05-05 13:08beuc-inria