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

Inventory and spellview modifications #211

Merged
merged 12 commits into from
Nov 13, 2019

Conversation

Derillo
Copy link
Contributor

@Derillo Derillo commented Nov 6, 2019

Una gran labor del equipo por el entusiasmo de uno de nuestros colaboradores, lanzó una propuesta para mejorar el uso del Inventario y el libro de hechizos dentro del juego.
Después de varios intentos, vamos con uno más.
Este PR integra:

  • Un inventario y Libro de hechizos extendido configurable desde teclado
  • Se agregaron las traducciones para los códigos
  • Se cambiaron las imágenes de los hechizos.
    Revisar:
  • El uso de los módulos SpellSlot y SpellSlotEC si se puede usar uno solo

Luego de la integración ajustar #212

@Derillo Derillo added the feature Enhancement or new feature label Nov 6, 2019
@Derillo Derillo force-pushed the inventory-and-spellview-modifications branch from e066418 to 5418aaf Compare November 6, 2019 03:24
Copy link
Contributor

@guidota guidota left a comment

Choose a reason for hiding this comment

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

procuren usar todos el mismo formatter, el IDE trae uno para que el código se vea uniforme, claro y siempre igual

client/src/game/managers/AOInputProcessor.java Outdated Show resolved Hide resolved
}


private void addSpellSVE(Slot slot){ //funcion provisoria hasta q encuentre como hacerlo a desde el servidor
Copy link
Contributor

Choose a reason for hiding this comment

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

podes tomar como ejemplo cuando se agarra un item

Copy link
Contributor

Choose a reason for hiding this comment

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

ok gracias despues le echo un ojo a ver si puedo resolver esto con mis escasos conocimientos y neuronas XD

if (icon == null) {
icon = new Texture(Gdx.files.local(Resources.GAME_SPELL_ICONS_PATH + spell.getIconGrh() + ".png"));
}
icon = new Texture(Gdx.files.local(Resources.GAME_SPELL_ICONS_PATH + spell.getId () + ".png"));
Copy link
Contributor

Choose a reason for hiding this comment

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

esto va a cargar la imagen cada vez que se llama al metodo getSpellIcon, por eso estaba el if, para que se cargue una unica vez

Copy link
Contributor

Choose a reason for hiding this comment

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

En este en especifico lo saque porque sino al asignar un nuevo hechizo a uno de los 6 slots que ya posee un hechizo no re dibuja la imagen, osea quedaba el hechizo nuevo con la imagen del hechizo que estaba antes.

client/src/game/ui/SpellSlotEC.java Outdated Show resolved Hide resolved
client/src/game/ui/SpellViewExpanded.java Outdated Show resolved Hide resolved
Copy link
Contributor

@guidota guidota left a comment

Choose a reason for hiding this comment

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

👌

bestiardo and others added 9 commits November 12, 2019 22:28
agrega
*barra rapida de inventario 6 slots
*Teclas rapidas del 1 al 6, se asignan manteniendo presionado el Shift izq
*inventario de echisos expandido 5x5 slots (por el maximo del spellbook actual)
*boton bajo la barra rapida para mostrar y esconder los inventarios expandidos

se modifico
*Inventario de 6 slots a  5x4 slots (por el maximo de inventory actual )
*la posision del inventario fue reemplasada por la barra rapida
Modificaciones / agregados
* Se pueden asignar hechisos al "spellview"
* Shift derecho + numero 1-6 agrega asigna el hechizo seleccionado en el Libro de Hechizos Expandido al Spellview
* Se agrego la funcionalidad de los pergaminos de hechisos. (al usarlos agrega Hechisos al LHE)
* imagenes para los Hechisos del battle + del original para cubrir los 45 Hechisos
* modificado parametro de asignacion de las imagenes de los hechisos.
- Reemplacé los For por ForEach porque son mas faciles de leer, si preferis usar For sentite libre de revertir este commit.
- Saque un For vacío.
- Uso System.arraycopy()
- Agrego un chequeo en getSelected()
removido el switchbotton que usaba como relleno reemplasado por add()
agregado a isover que faltavan para que no tire mensajes no ves nada en la consola cada que se clickea en el boton de expancion o en los hechisos del expandido
…se vea.

cualquiera de los 2 Shift + Numero añade items a la barra rapida si estamos viendo el inventario o hechizos a la barra de  hechizos listos si estamos viendo el libro de hechizos
@joaquinodz joaquinodz force-pushed the inventory-and-spellview-modifications branch from 6bb5070 to 2e9cf78 Compare November 13, 2019 01:31
Ahora la JVM no queda abierta al usar gradle por primera vez.
@Derillo
Copy link
Contributor Author

Derillo commented Nov 13, 2019

@bestiardo se te ocurre como solucionarlo?
Parece que las notificaciones de que ya tienes el hechizo no funcionan y cuando asignas repetidamente el hechizo al seleccionar uno se seleccionan todos los del mismo.

Captura de pantalla de 2019-11-12 22-39-56

@bestiardo
Copy link
Contributor

@bestiardo se te ocurre como solucionarlo?
Parece que las notificaciones de que ya tienes el hechizo no funcionan y cuando asignas repetidamente el hechizo al seleccionar uno se seleccionan todos los del mismo.

Captura de pantalla de 2019-11-12 22-39-56

Lo de ya tienes el hechizo solo esta implementado para el expandido ya que allí ocuparía un espacio que de momento no se puede reemplazar. Para el spellview no lo toque porque supuse que el usuario lo puede asignar a gusto y si quiere tener el mismo hechizo en esa barra ya es problema de el, ya que en dicha barra se puede asignar los hechizos a voluntad dentro de los que tenes en el expandido. va eso desde mi punto de vista.

- Ahora se ve más claro que hechizo esta seleccionado
- Ahora la barra circular de experiencia es violeta
@Derillo Derillo merged commit 176e631 into master Nov 13, 2019
@delete-merged-branch delete-merged-branch bot deleted the inventory-and-spellview-modifications branch November 13, 2019 15:39
@Derillo Derillo mentioned this pull request Nov 13, 2019
Derillo added a commit that referenced this pull request Nov 13, 2019
* Volvemos agregar soporte mediante KOFI

* Quitamos la comprobación de clean

* Volvemos agregar la barra circular de experiencia
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Enhancement or new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants