Skip to content

test custom attack effect - #561

Open
flaribbit wants to merge 1 commit into
26F-Studio:mainfrom
flaribbit:main
Open

flaribbit wants to merge 1 commit into
26F-Studio:mainfrom
flaribbit:main

Conversation

@flaribbit

@flaribbit flaribbit commented Dec 21, 2021 •

Copy link
Copy Markdown
Member
2021-12-21.23-54-33.mp4

@flaribbit flaribbit added the hang on Won't resolve/close this for now label Dec 21, 2021
@Not-A-Normal-Robot

Copy link
Copy Markdown
Member

it looks good enough but let's wait until OP tells us it's ready before it's merged

@Not-A-Normal-Robot Not-A-Normal-Robot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

image
I like this

@Not-A-Normal-Robot

Copy link
Copy Markdown
Member

image
A lot of people like this ✅

@Not-A-Normal-Robot

Copy link
Copy Markdown
Member

merge when @MrZ626

@Not-A-Normal-Robot

Copy link
Copy Markdown
Member

been almost a year... will this ever get merged

@MrZ626

MrZ626 commented Oct 17, 2022

Copy link
Copy Markdown
Member

🤔it just change the effect to old one, but I think new one is better
Maybe someone want change it himself so I keep the code here

@flaribbit

Copy link
Copy Markdown
Member Author

🤔it just change the effect to old one, but I think new one is better Maybe someone want change it himself so I keep the code here

maybe a setting item should be added for players to choose

@ParticleG

Copy link
Copy Markdown
Member

I think polygon attacks are COOL

@ParticleG
ParticleG force-pushed the main branch 2 times, most recently from 68d0b76 to ed11faf Compare June 15, 2023 13:36
@MrZ626
MrZ626 force-pushed the main branch 2 times, most recently from 33e873c to 4768df6 Compare September 26, 2024 19:50

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hang on Won't resolve/close this for now

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants