ferrannl / Dimitri

Project Software Architectuur
0 stars 0 forks source link

Feature/level transition #63

Closed xiadevisser closed 3 years ago

xiadevisser commented 3 years ago

Adds opacity support to textures in the engine, by default the opacity is 100. Adds level transition before opening the levelview:

Assets are updated with running sound and black background image.

Billyyy013 commented 3 years ago

Heb het getest, als je in die transitie zit en op esc drukt en dan weer op play drukt gaat hij stuk. Zou dan een extra is_active case in de openhomeview zetten dat je in de transitie niet op esc kan drukken.

xiadevisser commented 3 years ago

Heb het getest, als je in die transitie zit en op esc drukt en dan weer op play drukt gaat hij stuk. Zou dan een extra is_active case in de openhomeview zetten dat je in de transitie niet op esc kan drukken.

Dit is gefixt in #67

xiadevisser commented 3 years ago

Ik wilde het PR gaan nakijken en tijdens het testen kwam ik deze error tegen, weet jij hier meer van toevallig?

Ik kan zelf de bug niet na creëren maar ik heb een idee een poging gedaan om het op te lossen. Hopelijk werkt het nu wel.

xiadevisser commented 3 years ago

Ik krijg nu deze error, de vorige error heb ik niet meer weten te creëren! Ook is me opgevallen dat als je op c drukt tijdens de transitie dat hij dat niet heel leuk vind en als ik op h druk dan skipt hij de transitie Ik heb deze error gekregen door gewoon vaak op play te klikken en dan de transitie op verschillende momenten skippen en dan op esc drukken en het proces opnieuw

De knoppen zoals 'C', 'H' and 'Esc' zijn er al uitgehaald in #67 dus voor dit PR mag je daar ook van uit gaan dat ze geen factor spelen. Verder heb ik meerdere keren geprobeerd de error te creëren, maar het is mij nog niet gelukt. Misschien kunnen we morgen samen hiernaar kijken.