amsul / amsul/pickadate.js

update docs to reflect that `container` option can be JQuery

Aberta
#698 0 comentários 0 reações 1 responsável Reivindicada por @josephrexme Ver no GitHub
Linguagem predominante
JavaScript
Estrelas
7.6k
Forks
981
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

I am currently passing a `JQuery` element as the `container` argument when instantiating: http://amsul.ca/pickadate.js/date/#container

This is important since we have a reusable widget, so any selector within its scope will return multiple elements. However, `localVariable.find(selector)` will return a valid `JQuery` instance.

Assuming this is supported (it works currently since the pickadate source code accesses `container` via `$`), can you please add it to the docs so that we can be sure we're using the library in a prescribed manner?

I'll also update the typescript definitions, provided this is supported.

Thanks!

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.