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

Typo in Borderlayout animation.disabled property

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: 8.5.1
    • Fix Version/s: 8.5.2
    • Component/s: None
    • Security Level: Jimmy
    • Labels:
      None
    • gh.sprint.customfield.default.name:
      ZK 8.5.2 S1, ZK 8.5.2 S2

      Description

      https://www.zkoss.org/wiki/ZK_Component_Reference/Layouts/Borderlayout#AnimationDisabled

      Current Result

      current property name is:
      org.zkoss.zul.borderlayout.animation.disabed (missing l)

      Expected Result

      should be
      org.zkoss.zul.borderlayout.animation.disabled

      Debug Info

      https://github.com/zkoss/zk/blob/884e074c3ab999798dcf38b01564ec1f4c940818/zul/src/org/zkoss/zul/Borderlayout.java#L121

      Just in case this setting is already used in a project, it would be good to have either retro-compatibility or a warning message if the property name changes

        Activity

        Hide
        JamesChu JamesChu added a comment -

        Fixed since 2018/06/29

        Show
        JamesChu JamesChu added a comment - Fixed since 2018/06/29

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 4 hours
              4h
              Remaining:
              Time Spent - 1 hour Remaining Estimate - 3 hours
              3h
              Logged:
              Time Spent - 1 hour Remaining Estimate - 3 hours
              1h

                Agile