Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

Classe ls-modal #1700

Closed
lucasvmotta opened this issue Jul 6, 2016 · 4 comments
Closed

Classe ls-modal #1700

lucasvmotta opened this issue Jul 6, 2016 · 4 comments

Comments

@lucasvmotta
Copy link

Ao tentar utilizar um modal com a classe "ls-modal" , nada aparece na tela.
Vejo tambem a necessidade de um modal que cubra toda a tela, para exibição de tabelas grandes por exemplo.

@itumoraes
Copy link
Contributor

@lucasvmotta Você utilizando o HTML do modal, tal qual está descrito na documentação, com uma .ls-modal-box dentro da .ls-modal?

@lucasvmotta
Copy link
Author

@itumoraes isso foi sugerido pelo diego eis, segundo ele , deveria haver uma classe apenas ls-modal. A .ls-modal-box funciona corretamente

diegoeis added a commit that referenced this issue Jul 15, 2016
…#1700

Before, we need to use two classes to change the modal size:
`ls-modal-box and ls-modal-large` to make big modal, ou `ls-modal-box
ls-modal-small` to make small modal. Now, you can use only
`ls-modal-large` or `ls-modal-small`, and the old way works too.

TODO: we need to change where we put this classes. The right place is
add a classe ls-modal-large|small in the element ls-modal, because this
element is the parent of modal.

This Closes #1700
@deividmarques deividmarques added this to the 3.8.6 milestone Jul 15, 2016
@carlosbrunoro
Copy link

carlosbrunoro commented Jul 31, 2016

Existe uma inconsistência ao utilizar a class="ls-modal-box ls-modal-small" em dispositivos móveis.

Ao utilizar a classe, o modal não se redimensiona na tela corretamente (RESPONSIVE), somente quando a funcionalidade é removida.
screenshot_140

class="ls-modal-box"
screenshot_141

"**

@itumoraes
Copy link
Contributor

@diegoeis inseri no branch modal-size um commit que corrige o problema descrito acima

itumoraes added a commit that referenced this issue Aug 8, 2016
@deividmarques deividmarques removed this from the 3.8.6 milestone Aug 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants