Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Created GridRegion #716

Draft
wants to merge 15 commits into
base: dev
Choose a base branch
from
Draft

Created GridRegion #716

wants to merge 15 commits into from

Conversation

Kevin-771
Copy link
Collaborator

@Kevin-771 Kevin-771 commented Feb 2, 2024

Description

Created a basic abstract GridRegion class

Contributes to #711

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (improvement to an already existing feature)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

charlestian23 and others added 4 commits January 26, 2024 13:03
* Revert "Bugfix 549 (#682)"

This reverts commit 5048ee6.

* Case rule test fix (#705)

Co-authored-by: Chase Grajeda <[email protected]>

* Rapid fix for STT case rules

Case rules broke at some point from legacy code or merge conflict. Provided is a quick fix in CaseRule and CaseRule_Generic

* Revert "Revert "Bugfix 549 (#682)"" (#706)

This reverts commit e9fe310.

---------

Co-authored-by: Chase-Grajeda <[email protected]>
Co-authored-by: Chase Grajeda <[email protected]>
added some basic functions to GridRegion
@Kevin-771 Kevin-771 changed the base branch from master to dev February 2, 2024 21:13
@Kevin-771 Kevin-771 changed the title Grid region Created GridRegion Feb 2, 2024
@Kevin-771 Kevin-771 marked this pull request as ready for review February 2, 2024 21:17
@charlestian23 charlestian23 marked this pull request as draft February 2, 2024 21:31
@charlestian23
Copy link
Collaborator

Making this a draft for now. We'll merge this into dev once we complete all the implementation.

Kevin-771 and others added 10 commits February 2, 2024 16:56
Added rippleeffect folder within puzzles so we can begin implementing and rendering the demo puzzle

Added RippleEffect.java
Added RippleEffectCell.java
Added RippleEffectImporter.java
Added RippleEffectView.java
Added more dependencies for RippleEffect board
Added config for RippleEffect editing support
Added RippleEffect Exporter template. At this time it should not be used for file IO activities.
…nsist of numbers. I have put no functions to change anything in the board after initialized. Each region has a different color with a max of 5 colors (5 regions) right now.
Initializing Grid Region Colors
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.

4 participants