Chris,<br><br>>>I'm wondering if this is adding too much complexity to the display of the test<br>results. Will many users really care to see more than say 5 of the testing<br>results? Does it help them to show them varying numbers of entries?
<br><br><br>I guess that's a matter of opinion for each user. I was looking through the page with variables of screen size, and I found that 5 results looks too limiting on 1280x1024, but lowering to 1024x768 or 800x600, it's just right. Allowing the user to choose how many to show based on system setup and preference might work.
<br><br>I guess before I fix the coding, I take it that you don't think this is a good idea?<br><br><div><span class="gmail_quote">On 2/10/06, <b class="gmail_sendername">Chris Morgan</b> <<a href="mailto:cmorgan@alum.wpi.edu">
cmorgan@alum.wpi.edu</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">I'm wondering if this is adding too much complexity to the display of the test
<br>results. Will many users really care to see more than say 5 of the testing<br>results? Does it help them to show them varying numbers of entries?<br><br><br>On Tuesday 07 February 2006 1:27 pm, Jason Weisberger wrote:
<br>> Allows the user to display the selected amount of testing results.<br>><br>> Changelog: include/testResults.php<br>><br>> Index: appdb/include/testResults.php<br>> ===================================================================
<br>> RCS file: /home/wine/appdb/include/testResults.php,v<br>> retrieving revision 1.9<br>> diff -u -r1.9 testResults.php<br>> --- appdb/include/testResults.php 28 Jan 2006 22:53:28 -0000 1.9<br>> +++ appdb/include/testResults.php 7 Feb 2006 18:23:52 -0000
<br>> @@ -349,23 +349,25 @@<br>> // Show the Test results for a application version<br>> function ShowVersionsTestingTable($iVersionId, $iCurrentTest, $link,<br>> $iDisplayLimit)<br>> {<br>> - $showAll = $_REQUEST['showAll'];
<br>> + $iDisplayLimit = $_REQUEST['iDisplayLimit'];<br>> +<br><br>Do you really want to override the display limit passed into the function?<br><br><br><br>> - echo '<form method=get action="'.$PHP_SELF.'">';
<br>> - echo '<input name="versionId" type=hidden value="',$iVersionId,'"<br>> />';<br>> - if($rowsUsed >= $iDisplayLimit && !is_string($showAll))<br>> - echo '<input class="button" name="showAll" type=submit
<br>> value="Show All Tests" />';<br>> + echo '<form method=get action="'.$PHP_SELF.'">';<br>> + echo '<input name="versionId" type=hidden value="',$iVersionId,'" />';
<br>> + if($rowsUsed >= 5) {<br>> + echo 'Tests:<select name="iDisplayLimit">';<br>> + echo '<option value="5"'; if($iDisplayLimit == 5) { echo '<br>> selected="selected"';} echo '>5</option>';
<br>> + echo '<option value="10"'; if($iDisplayLimit == 10) { echo '<br>> selected="selected"';} echo '>10</option>';<br>> + echo '<option value="15"'; if($iDisplayLimit == 15) { echo '
<br>> selected="selected"';} echo '>15</option>';<br>> + echo '<option value="20"'; if($iDisplayLimit == 20) { echo '<br>> selected="selected"';} echo '>20</option>';
<br>> + echo '<option value="ALL"'; if($iDisplayLimit == "ALL") { echo<br>> ' selected="selected"';} echo '>ALL</option>';<br>> + echo '</select>';
<br><br>You can use a for() loop to generate all of these entries.<br></blockquote></div><br><br clear="all"><br>-- <br>Jason Weisberger<br><a href="mailto:jbdubbs@gmail.com">jbdubbs@gmail.com</a>