Skip to content

Conversation

@adumesny
Copy link
Member

Description

  • went through new code for DD and made sure we have a type for everything
  • cleanup to match lib style (public vs private)
  • made all routines return 'this' for chainable calls. Abstract DDBaseImplement still return :void
    otherwise had to do
    return this as unknown as DDBaseImplement
    for all sub-classes which is annoying.
  • created DDUIData to hold what we need for callback data. Commented out part we don't use today...

* went through new code for DD and made sure we have a type for everything
* cleanup to match lib style (public vs private)
* made all routines return 'this' for chainable calls. Abstract DDBaseImplement still return :void
otherwise had to do
`return this as unknown as DDBaseImplement`
for all sub-classes which is annoying.
* created DDUIData to hold what we need for callback data. Commented out part we don't use today...
@adumesny adumesny merged commit c45f9a8 into gridstack:h5 Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant