Quipu Rojo#2473
Conversation
Duplicate and rename template and all files
Base del Minijuego 3
minijuego3 pre terminado
…uendia1y Minijuego 3 jugable y con mecánica
modificacion1 y 2
…mites Se modificó del temporizador, audio, terreno y se limitó el area del mapa
intro y outro subida
…x-dependencias Restauración de los archivos de la carpeta elementos del minijuego 3
Create folder actualizacion logic
|
@kevinKVN00 ¡hola! Por favor revisen https://github.com/endlessm/threadbare/wiki/StoryQuest-submission y marquen esto como "ready for review" cuando esté listo! |
|
Play this branch at https://play.threadbare.game/branches/GameLab6-UTP-Grupo5-Equipo5/main/. (This launches the game from the start, not directly at the change(s) in this pull request.) |
wjt
left a comment
wjt
left a comment
There was a problem hiding this comment.
Nice quest! I love how the two stealth stages are quite large, with a complicated maze structure. It makes for a different kind of challenge to a lot of those I've seen before in the game. The flow - stealth, codebreaking, stealth again but this time with time pressure - works well. I also particularly like the enemy art!
I have a number of comments about the level design, some technical points, and licensing questions. Please also review https://github.com/endlessm/threadbare/wiki/StoryQuest-submission.
Please forgive me for writing the whole review in English - I don't speak Spanish. Feel free to ask (here or on discord) if you need more clarification on any of the points here.
|
|
||
| [resource] | ||
| script = ExtResource("1_bi1xp") | ||
| title = "jueguito" |
There was a problem hiding this comment.
Please fill in the title, description, authors, etc. for this quest. Right now it looks rather empty in the storybook!
There was a problem hiding this comment.
Unused copy of assets/third_party/tiny-swords/UI/Banners/Banner_Horizontal.png, please remove.
|
|
||
| [params] | ||
|
|
||
| loop=false |
There was a problem hiding this comment.
Again, one typically wants background music to loop, and to loop cleanl.y
There was a problem hiding this comment.
I really like the timer in this scene: it adds some pressure and fits the plot well!
There was a problem hiding this comment.
Unused copy of res://assets/third_party/tiny-swords-non-cc0/Terrain/Resources/Wood/Trees/Stump 1.png, please remove, along with 2, 3 and 4.
There was a problem hiding this comment.
This (and the image it references) isn't used. Did you intend to use it somewhere in the scene?
There was a problem hiding this comment.
The music in this scene is played by an AudioStreamPlayer2D. I can see why you didn't use BackgroundMusic for this - you want the music to start at the same time as the counter. (It is possible to do this - see res://scenes/quests/lore_quests/quest_002/1_void_runner/void_runner.tscn for an example, where the music is an AudioStreamInteractive and transitions between different clips are triggered by Area2Ds in the world. You're welcome to set up the same thing here but you don't have to.)
However AudioStreamPlayer2D is the wrong node to use for background music. It is meant for sounds that exist in the game world, and depending on where the node is relative to the centre of the screen, the audio will be panned and/or attenuated. I can see that you set Max Distance to 24,000px so that it never fades to nothing; but it still pans to the left ear as soon as you run to the right.
Change the node type to AudioStreamPlayer (it has the same API minus the 2D bits). Change the "Bus" to "Music" so that it respects the volume control.
5 participants
No description provided.