-
Notifications
You must be signed in to change notification settings - Fork 2
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
Ocultar texto introductivo #21
Comments
Sería bueno simplemente eliminar el texto cuando se haga la request al servidor con replace? o eliminarlo cuando se escriba en el textarea? |
La primera opción si se implementa en client side entonces sí, pero veo más elegante la segunda |
Tenía pensado hacerlo en el client side, pero voy a probar la segunda a ver que tal queda |
Veo más factible la segunda, o si no simplemente bloquear que se guarde si esta ese texto. |
La segunda de prueba para ver como queda: 312a41b |
(ya está en producción) |
Te devuelve el cursor al final donde hay el último LF cuando escribes a la primera o segunda línea. Aquí el newline no hace falta ya si total el texto va a desaparecer. Mira el último commit y válida si funciona correctamente. |
Hmm, creo que mejor dejarlo al final del comentario, qué opináis? |
Teniendo en cuenta que se va a boorar al principio opino que queda mejor porque al final queda aleatorio que vuelva hacia delante. |
Si se pudiera ocultar yo creo que quedaria bastante bien (y luego hacerlo aparecer de nuevo), pero sino que el cursor vaya al final |
Y si se hace un aviso arriba cuando se abra la web? Que diga eso mismo y que haga dismiss cuando empieces a escribir? |
No me convence tener ventanas, además que así se ve más limpio y sin distracciones |
Eso es cierto, pero oculto no liaría? |
Creo que ahora debería estar bien |
al menos a mi me convence |
A mi también
|
Yup, ya más adelante se repasará ese detalle |
Se podría añadir ese texto de fondo si no hay nada en el Monaco? Así también se previene que alguien guarde sin parar con ese texto predeterminado.
Originally posted by @inetol in #17 (review)
The text was updated successfully, but these errors were encountered: