Scarab commit: svn commit: r10823 - trunk/src/webapp/WEB-INF/templates/viewIssue/ViewIssueTab1.vm

Hussayn Dabbous <[email protected]>
Newsgroups gmane.comp.java.scarab.cvs
Message-ID <[email protected]>
Author: dabbous
Date: 2009-07-22 23:49:02-0700
New Revision: 10823

Modified:
   trunk/src/webapp/WEB-INF/templates/viewIssue/ViewIssueTab1.vm

Log:
SCB2986: Fixed a bug concerning visibility of "secnd column attributes".
The problem was with how the associated javascript function determines
when it has to set/unset display and visibility style.

Modified: trunk/src/webapp/WEB-INF/templates/viewIssue/ViewIssueTab1.vm
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/webapp/WEB-INF/templates/viewIssue/ViewIssueTab1.vm?view=diff&pathrev=10823&r1=10822&r2=10823
==============================================================================
--- trunk/src/webapp/WEB-INF/templates/viewIssue/ViewIssueTab1.vm	(original)
+++ trunk/src/webapp/WEB-INF/templates/viewIssue/ViewIssueTab1.vm	2009-07-22 23:49:02-0700
@@ -87,10 +87,10 @@
       #if ($onSameLine)
         #set ($isHidden =  !( $currentIssue.isRequiredAttributeFor($att, $user) || $rma.Visible || $currentColumn==1 ) )
         #if ($currentRow==0)
-          <tr id="${displayValue}:Container" #if ($isHidden) style="visibility:hidden;" #end>
+          <tr id="${displayValue}:Container" #if ($isHidden) style="visibility:hidden;" #else style="visibility:visible;" #end>
           <th style="text-align:right;">
         #else
-          <span id="${displayValue}:Container" #if ($isHidden) style="display:none;" #end>
+          <span id="${displayValue}:Container" #if ($isHidden) style="display:none;" #else style="display:inline;" #end>
         #end
         <span class="displayValue" style="white-space:nowrap;display:inline;margin-left:0px; margin-right:0px;">#drawRequiredMarkupAsNeeded($user $currentIssue $att)$displayValue</span>
         #if ($currentRow==0)
@@ -133,10 +133,10 @@
         #if ($onSameLine)
           #set ($isHidden =  !( $currentIssue.isRequiredAttributeFor($att, $user) || $rma.Visible || $currentColumn==1 ) )
           #if ($currentRow==0)
-            <tr id="${displayValue}:Container" #if ($isHidden) style="visibility:hidden;" #end>
+            <tr id="${displayValue}:Container" #if ($isHidden) style="visibility:hidden;" #else style="visibility:visible;" #end>
             <th style="text-align:right;">
           #else
-            <span id="${displayValue}:Container" #if ($isHidden) style="display:none;" #end>
+            <span id="${displayValue}:Container" #if ($isHidden) style="display:none;" #else style="display:inline;" #end>
           #end
           <span class="displayValue" style="white-space:nowrap;display:inline;margin-left:0px; margin-right:0px;">$displayValue</span>
           #if ($currentRow==0)

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=2374671
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.