Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrected problems with new job wizard. #262

Merged
merged 8 commits into from
Feb 18, 2016

Conversation

RaulMonteroC
Copy link
Member

What's New

  • Removed dots at the bottom of the wizard
  • Added visual aid to show user the current step and the remaining ones (view screen shot)
  • Corrected bug that allowed user to change pages without validating the current one.
  • Added google's recaptcha to view

What I need help with

  • Form is sumitted even if captcha is not valid
  • New round step buttons not aligning properly on mobile.

Related issues

capture

@jgtaveras
Copy link
Member

+1 al PR

@eatskolnikov
Copy link
Collaborator

@RaulMonteroC el captcha se tiene que validar en el servidor, el va a hacer el post como sea.

Solo deberías agregar:

[CaptchaValidator(RequiredMessage = "Por favor confirma que no eres un robot")]

en el metodo post del wizard.

@luis-ramirez
Copy link
Member

New round step buttons not aligning properly on mobile.
No muestres este div en mobile :), en cambio, agrega algo que se vea amigable en mobile como : Paso 1 de 3 etc, etc...

@eatskolnikov
Copy link
Collaborator

@luis-ramirez full,
@RaulMonteroC solo sería agregar la clase _hidden-sm _ al div

@RaulMonteroC
Copy link
Member Author

Ya esta corregido el problema del reCaptcha.

Ahora con el tema del layout de los botones, esta el problema que eso esta harcodeado. Si se fijan la posición esta explicita usando absolute position y left para alinearlo. Osea que salvo la resolución para la cuál la trabaje se va a ver descuadrada.

Necesito encontrar una manera para que esos circulos se ajusten más o menos aunque en la versión mobile se oculte.

@eatskolnikov
Copy link
Collaborator

Yo lo veo bien +1 RT Like, comment and Subscribe

@jugomez
Copy link
Collaborator

jugomez commented Feb 18, 2016

+1

eatskolnikov added a commit that referenced this pull request Feb 18, 2016
Corrected problems with new job wizard.
@eatskolnikov eatskolnikov merged commit e9c8d3d into developersdo:Development Feb 18, 2016
@RaulMonteroC RaulMonteroC deleted the bug/wizard branch February 20, 2016 15:01
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.

5 participants