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

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

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Normal Normal
    • 8.5.1
    • 8.5.0
    • Components
    • Security Level: Jimmy
    • None
    • None

      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

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

              Created:
              Updated:
              Resolved:

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