-
Notifications
You must be signed in to change notification settings - Fork 140
Conversation
Adding the necessary infos function to create the index and title mobile attributes and the mobile event to open and close the menu.
function bindMobileMenuClick() { | ||
$(config.selectors.mobile).on('click.steps', function() { | ||
$(config.selectors.nav).toggleClass(config.status.active); | ||
$(config.selectors.nav).slideToggle(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@itumoraes precisa confirmar com a equipe se utlizaremos o método slideToggle() ou o toogleClass(), já que a animação ficaria por conta do CSS (usamos assim em outros componentes).
@deividmarques, conversei com o Italo Oliveira de UX e o mesmo definiu que a seta deve apontar para cima quando o menu estiver aberto e a cor da seta vai mudar para a mesma que é utilizada no dropdown: Quanto a utilização do slideToggle(), acertei com o @diegoeis que vamos mudar para toggleClass() e deixar o CSS3 cuidar da animação. |
…transition instead of slideToggle().
…ing the .ls-active class on steps to control more items and including the mobile button arrow icon rotate animation.
Quando estou no Passo 3 e volto para o passo 1, não consigo voltar para o passo 3 novamente. Somente quando clico em avançar no passo 2. É esse comportamento mesmo? |
@palloi sim, você só navega entre os passos já preenchidos. |
👍 |
Foi desenvolvido o novo passo a passo mobile com base no layout apresentado na issue #1554.
Conversei com o Italo de UX e foi definido que na versão desktop, a cor do item que estiver selecionado será a mesma do tema.
close #1554