Results 1 to 3 of 3

Thread: typo 'sortabl'

  1. #1
    Sencha Premium Member
    Join Date
    Aug 2013
    Location
    Australia
    Posts
    306

    Default typo 'sortabl'

    Interesting that the code bug 'sortabl:true' (https://examples.sencha.com/extjs/6....ink/#form-grid) is in the Details-View, but the code actually works (Company is sortable).
    This means that you have to update code bug and/or documentation bug.

    On 17 Jul 2019 I reported that 7.0.0 documentation was pointing to 6.2 examples, which at the time I thought was okay (https://www.sencha.com/forum/showthread.php?472093)
    For a larger example, see the Basic Grid Example.
    https://docs.sencha.com/extjs/7.0.0/...nts/grids.html

    As it turns out the 7.0.0 document (https://docs.sencha.com/extjs/7.0.0/...nts/grids.html) that points to 6.2 code (For a larger example, see the Basic Grid Example) made 'sortable: false'.
    Maybe that was just to show a bad UI, but not a bug.

  2. #2
    Sencha Premium User mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    40,449

    Default

    The reason that the typo still works is because a column's sortable config default to true so that config being set to true wouldn't change anything anyway: https://docs.sencha.com/extjs/7.0.0/...l#cfg-sortable
    Mitchell Simoens @LikelyMitch
    Modus Create, Senior Fullstack Engineer
    ________________
    Modus Create is based on the model of an open source team. We’re a remote, global team of experts in our field. To find out more about the work we do, head over to our website.

    Check out my GitHub:
    https://github.com/mitchellsimoens

  3. #3
    Sencha Premium Member
    Join Date
    Aug 2013
    Location
    Australia
    Posts
    306

    Default

    Quote Originally Posted by mitchellsimoens View Post
    The reason that the typo still works is because a column's sortable config default to true so that config being set to true wouldn't change anything anyway: https://docs.sencha.com/extjs/7.0.0/...l#cfg-sortable
    Thanks for the review.
    I was looking at it as a typo, rather than some useless code where the typo was irrelevant.
    This reminds me that I am wasting time declaring variables their default value.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •