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

hidden columns shifting cells

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: 8.5.1.2
    • Fix Version/s: 8.5.2
    • Component/s: Components
    • Security Level: Jimmy
    • Environment:

      reproduced on FF 59/60/60.0.1

    • gh.sprint.customfield.default.name:
      ZK 8.5.2 S1, ZK 8.5.2 S2

      Description

      Steps to Reproduce

      run the attached example
      6 columns 4~9 are rendered hidden (visible=false)

      Current Result

      the border between the cells in col-3 and col-10 is appears with 2px width

      Expected Result

      single pixel border between all columns

      Debug Cause

      the red border styles only highlight the problem it also happens without the borders, just not obvious

      Root Cause

      the "hidden" style of 0.1px adds up to 0.6px rounding up to a full pixel

      Workaround

      adjust the width to a smaller value making it less likely to cause double pixel borders e.g. 0.001px

      	zk.afterLoad('zul.mesh', function() {
      		var xMeshWidget = {};
      		zk.override(zul.mesh.MeshWidget.prototype, xMeshWidget, {
      			bind_ : function() {
      				var result = xMeshWidget.bind_.apply(this, arguments);
      				var adjustWidth = function() {
      					if (this.style.width === '0.1px') {
      						this.style.width = '0.001px';
      					};
      				}
      				jq(this.ehdfaker).children().each(adjustWidth);
      				jq(this.ebdfaker).children().each(adjustWidth);
      				jq(this.eftfaker).children().each(adjustWidth);
      				return result;
      			}
      		});//zk.override
      	});//zk.afterLoad
      

      the goal should be to evaluate whether this workaround of only fake hidden columns is still necessary, and apply it only to browsers needed

        Issue Links

          Activity

          Hide
          Anonymous added a comment -

          Fixed since 2018/06/28

          Show
          Anonymous added a comment - Fixed since 2018/06/28

            People

            • Assignee:
              rudyhuang rudyhuang
              Reporter:
              cor3000 cor3000
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 1 day
                1d
                Remaining:
                Time Spent - 4 hours, 30 minutes Remaining Estimate - 3 hours, 30 minutes
                3h 30m
                Logged:
                Time Spent - 4 hours, 30 minutes Remaining Estimate - 3 hours, 30 minutes
                4h 30m

                  Agile