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

Konami code animation: issue #297 #486

Merged
merged 4 commits into from
Mar 13, 2024

Conversation

MattEzekiel
Copy link
Contributor

@MattEzekiel MattEzekiel commented Mar 13, 2024

Descripción

Se cambió la animación del konami code acorde a la propuesta de: #297

Cambios propuestos

Desktop.13-3-2024.11-41-53.mp4

Comprobación de cambios

  • He revisado que no haya ninguna PR (pull request) ya abierta con un problema similar, siguiendo el apartado de buenas prácticas
  • He revisado localmente los cambios para asegurarme de que no haya errores ni problemas.
  • He probado estos cambios en múltiples dispositivos y navegadores para asegurarme de que la landing page se vea y funcione correctamente.
  • He actualizado la documentación, si corresponde.

Enlaces útiles

Issue 297

Closes #297

Copy link

vercel bot commented Mar 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
la-velada-web-oficial ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 13, 2024 5:10pm

@midudev
Copy link
Owner

midudev commented Mar 13, 2024

¿Podrías dejar toda la funcionalidad que ya existía antes y añadir esta funcionalidad al escribir "midu" como nuevo konami code? :)

Así reutilizas el código de antes, dejamos los huevos de pascua de antes y sólo añadimos este nuevo.

@AlejandroSuero
Copy link
Contributor

¿Podrías dejar toda la funcionalidad que ya existía antes y añadir esta funcionalidad al escribir "midu" como nuevo konami code? :)

Así reutilizas el código de antes, dejamos los huevos de pascua de antes y sólo añadimos este nuevo.

@midudev, se podría hacer a la hora de mirar el código, si empieza con m que haga el de los contribuidores, y si empieza ArrowUp, se deje la funcionalidad del anterior no?

@JJozef
Copy link
Contributor

JJozef commented Mar 13, 2024

No es mejor que añadas simplemente el nuevo Konami Code?, algo asi:

// FILE: src/consts/konami-codes.ts

export const konamiCodes = {
	// dejas lo de antes,
	contributorsBubbles: ["","","",""],
}
// FILE: src/components/KonamiCode.astro

const { rotateY, gloveCursor, amongUsCursor, lolCursor, contributorsBubbles } = konamiCodes

document.addEventListener("keydown", ({ key }) => {
   // dejas los que ya estaban
    Konami({
	    code: contributorsBubbles,
	    keyPressed: key,
	    onFinishTime: 1500,
	    onSuccess: () => {
		    // aqui haces lo que tengas que hacer cuando se cumpla
	    },
	    onFinish: () => {
		   // aqui lo hace lo que se tenga que hacer, despues que termine el tiempo de ** onFinishTime: 1500,**
	    },
    })
})

Ahora quedaron ambos, el konami code y el miducode
@AlejandroSuero
Copy link
Contributor

@JJozef también es buena opción así, y el cursor que sea la cara de midu xD

@MattEzekiel
Copy link
Contributor Author

MattEzekiel commented Mar 13, 2024

Dejé el viejo konami code y agregué el miducode

Desktop.13-3-2024.14-01-00.mp4

@JJozef
Copy link
Contributor

JJozef commented Mar 13, 2024

@JJozef también es buena opción así, y el cursor que sea la cara de midu xD

Claro jajajaja, midu.cur is coming

Copy link
Contributor

@AlejandroSuero AlejandroSuero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🫡

@midudev midudev merged commit 3324ffa into midudev:main Mar 13, 2024
3 of 4 checks passed
@MattEzekiel MattEzekiel deleted the konami-code-animation branch March 13, 2024 17:56
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.

Propuesta: Mejorar Konami Code
4 participants