Skip to content

Feature: Frog attack pattern - #17

Open
SACKBOI123 wants to merge 7 commits into
developfrom
frog-attack-pattern
Open

Feature: Frog attack pattern#17
SACKBOI123 wants to merge 7 commits into
developfrom
frog-attack-pattern

Conversation

@SACKBOI123

Copy link
Copy Markdown
Collaborator

The frog has an attack pattern and also some changes to the duck to fix his sprite a bit

… for the floorLevel constant. Fix for some of how the duck is drawn. (added new gifs) Now storing z-indices in playdateConstants. Frog projectile (spitball) has been set up but is not used
@SACKBOI123
SACKBOI123 requested a review from brody2002 June 27, 2025 07:12
Fix z-indices

@brody2002 brody2002 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Well Done sir 🎉

Couple changes to be made 🤙

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why Did The same Duck Sprites get re-added? Are they resized?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Re-added Sprite?

tongueExtra:setZIndex(101)
spitball:setZIndex(150)

frog.enterSFX = sound.fileplayer.new("Sounds/SoundEffects/CTMR")

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Change Sound ? or keep it for now? 💀

@@ -27,28 +29,58 @@ Frog.create = function()
frog.direction = left
frog.onGround = true
frog.velocity = { x = 0, y = 0}

@brody2002 brody2002 Jun 27, 2025

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Not being used anywhere in the code

frog.velocity

Comment thread Source/Player/duck.lua
Comment on lines +38 to +39
duck.idleAnimation = AnimatedImage.new("Images/Duck/Gifs/Idle-1", {delay = 200, loop = true, first = 1, last = 2})
duck.walkingAnimation = AnimatedImage.new("Images/Duck/Gifs/Walking-1", {delay = 200, loop = true, first = 1, last = 2})

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Since you made the Frog Scope more than just the frog. Can we just make this the ticket that also handles all the sprites? That way the changes to the duck are traceable to this ticket as well.

  1. Delete the unused .gifs
  2. Remake all the .gifs 96x96
  3. Remove the -1s on the file name

Comment thread Source/Player/duck.lua
local initialImage = duck.currentAnimation:getImage()
duck:setImage(initialImage)
duck:moveTo(200, playdateConstants.floorLevel - duck.height)
-- For some reason the image is scaled down to 2/3 size ??????????????????

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Make the comment
".gif scaled by 2/3"

that comment is ass

local height = 240
local width = 400
local height_constant = 44
local height_constant = 16

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

height_constant doesn't really correctly portray what this value is used for. It should be something to do with the floor?

I also don't believe it's being used anywhere so maybe can delete it?


frog.enterSFX = sound.fileplayer.new("Sounds/SoundEffects/CTMR")

-- Don't know WHY THE FUCK there need to be scalars on these but they are necessary AFAIK

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Shitty Comment

Comment on lines 21 to +23
local frog = graphics.sprite.new()
local tongueExtra = graphics.sprite.new()
local spitball = graphics.sprite.new()

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I know it would be ass to redo your code to accomade a .png.

But since the duck is being resized, maybe resize your frog images so that there is less whitespace?

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