Uploaded image for project: 'ZK'
  1. ZK
  2. ZK-3804

Chrome meshwidget visible="false" column not hidden after ZK-3722

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: 8.5.0
    • Fix Version/s: 8.5.1
    • Component/s: Components
    • Security Level: Jimmy
    • Labels:
      None

      Description

      Steps to Reproduce

      Run fiddle:
      http://zkfiddle.org/sample/tj8bv6/4-Grid-850-colunm-hidden

      Current Result

      In ZK 8.0.5
      Visible="false" column is hidden.
      Grid body <colgroup> <col> element receives style="display:none"
      Each cell in the column also receive style="display:none"

      In ZK 8.5.0
      Visible="false" column header is hidden, but content (row cells) is not hidden.
      Grid body <colgroup> <col> element receives style="width:0px" (chrome doesn't parse width:0px well)

      Expected Result

      Grid body <col> should be hidden (either display none or width:0.1px)

      Debug Info

      https://github.com/zkoss/zk/blame/072daeafd9d60997b65889fd3ae0dc562c9e1076/zul/src/archive/web/js/zul/mesh/MeshWidget.js#L1315

      Root Cause

      Changes to Meshwidget in https://github.com/zkoss/zk/commit/a9d091ed356824be23c86e69261fbf4b7796e183
      Beforehand:
      Both the <col> and the <td> elements used by this column would receive "display:none".

      Workaround

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              wenninghsu wenninghsu
              Reporter:
              MDuchemin MDuchemin
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 30 minutes
                  30m