Tapestry
  1. Tapestry
  2. TAPESTRY-2293

BeanDisplay formatting is off for fields with null values

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.11
    • Fix Version/s: 5.0.12
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      I believe it would be fixed if the t-beandisplay-value <div> was forced to contain a   for blank values.

      Ideally, the markup <em>null</em> for a null field, and perhaps <em>blank</em> for a blank field, would work best.

      1. index.html
        0.8 kB
        Martijn Brinkers

        Issue Links

          Activity

          Howard M. Lewis Ship created issue -
          Hide
          Howard M. Lewis Ship added a comment -

          A simpler solution is:

          DIV.t-beandisplay-row

          { clear: both; }
          Show
          Howard M. Lewis Ship added a comment - A simpler solution is: DIV.t-beandisplay-row { clear: both; }
          Howard M. Lewis Ship made changes -
          Field Original Value New Value
          Status Open [ 1 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Fix Version/s 5.0.12 [ 12313048 ]
          Assignee Howard M. Lewis Ship [ hlship ]
          Hide
          Martijn Brinkers added a comment - - edited

          It seems the previous fix no longer works with 5.0.14. If a property returns the empty string value "" the label and values are shifted and no longer line-up.

          FF2 the lines do not line-up.
          IE7 seems fine.

          Show
          Martijn Brinkers added a comment - - edited It seems the previous fix no longer works with 5.0.14. If a property returns the empty string value "" the label and values are shifted and no longer line-up. FF2 the lines do not line-up. IE7 seems fine.
          Hide
          Martijn Brinkers added a comment -

          Example static HTML file showing that FF2 uses the dd from another row when the dd is empty

          Show
          Martijn Brinkers added a comment - Example static HTML file showing that FF2 uses the dd from another row when the dd is empty
          Martijn Brinkers made changes -
          Attachment index.html [ 12387982 ]
          Martijn Brinkers made changes -
          Link This issue is related to TAPESTRY-2578 [ TAPESTRY-2578 ]
          Mark Thomas made changes -
          Workflow jira [ 12427131 ] Default workflow, editable Closed status [ 12568654 ]
          Mark Thomas made changes -
          Workflow Default workflow, editable Closed status [ 12568654 ] jira [ 12591685 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          43m 43s 1 Howard M. Lewis Ship 22/Mar/08 00:49

            People

            • Assignee:
              Howard M. Lewis Ship
              Reporter:
              Howard M. Lewis Ship
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development