Skip to content

Conversation

@tesselslate
Copy link
Contributor

This PR adds bubble columns. They look like water (with moving bubble particles), and can push you in any direction with a configurable strength.

Requested by me

@tesselslate tesselslate added the Core Gameplay This request is core to puzzle design label Jan 20, 2020
@0x0ade 0x0ade requested a review from Cruor January 21, 2020 23:51
else
WindMovers.Add(mover, 0f);
} else {
if(WindMovers.ContainsKey(mover)) {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
if(WindMovers.ContainsKey(mover)) {
if (WindMovers.ContainsKey(mover)) {

class BubblePushField : Entity {
public new int Width;

public new int Height;
Copy link
Member

Choose a reason for hiding this comment

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

This kiiinda begs for issues...

Position = BubbleField.Position;

Rand = BubbleField.Rand;
Texture = GFX.Game["particles/SpringCollab2020/bubble_" + new string[] { "a", "b" }[Rand.Next(0, 1)]];
Copy link
Member

Choose a reason for hiding this comment

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

Array should be stored in a static readonly field.

Copy link
Contributor

@Cruor Cruor left a comment

Choose a reason for hiding this comment

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

Ahorn plugin looks good to me functionality wise.
Please fix the mixed usage of spaces and tabs for indentation though.

Fix inconsistent usage of space/tabs in Ahorn plugin
Small code tweaks in C# portion
@tesselslate
Copy link
Contributor Author

@Cruor @0x0ade
Should be good to go now, I think.

Copy link
Contributor

@Cruor Cruor left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@Cruor Cruor merged commit 48e87fa into master Jan 24, 2020
@tesselslate tesselslate deleted the bubble_column branch January 25, 2020 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Core Gameplay This request is core to puzzle design

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants