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

Mejoras visuales #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Mejoras visuales #36

wants to merge 1 commit into from

Conversation

Ricardoaar
Copy link
Member

  • Montañas con inclinacion correcta y opacidad del foreground restablecida a 1

image

  • Parallax:

    • Offset agregado como parametro en caso de algun calculo que no se sepa hacer (El parallax diagonal de las motanañas en este caso)
  • Los elementos de la escena ya van a la misma velocidad

Revisar la branch antes de hacer el merge
Para asegurar que no haya roto algo de la escena principal 😅

Copy link
Member

@LuqueDaniel LuqueDaniel left a comment

Choose a reason for hiding this comment

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

He estado probando en la escena final. Algunos obstáculos de la parte inferior de los patrones no se visualizan bien.

imagen

Enla escena queda así.

imagen

Probe modificnado el vector _initialPositionPat de PatternManager, pero eso provoca que los obstáculos se salgan de la pista por la parte superior.

¿Deberíamos rehacer los patrones/reajustar los patrones? o ¿o podriamos solucionarlo de otro modo?

@LuqueDaniel
Copy link
Member

Acabo de notar que hay patrones con distintos tamaños.

imagen

@Ricardoaar
Copy link
Member Author

@LuqueDaniel En cuanto a que se vean o no, con alejar un poco el foreground se verian sin problema.

Sobre el tamaño ... ¿Ya estaban asi desde antes o fue desde mi branch? No lo habia notado 🤔

@LuqueDaniel
Copy link
Member

@Run19 Creo que han estado así desde siempre. Yo tampoco lo había notado.

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.

2 participants