Skip to content

Remove unecessary memo #154

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 21, 2024
Merged

Remove unecessary memo #154

merged 2 commits into from
Jun 21, 2024

Conversation

@dansreis
Copy link
Owner

@dansreis dansreis commented Jun 21, 2024

Description

Resolved the issue where the rectangle drawing did not show the helper.

Additional context

Also removed React "memo" and simplified code

What is the purpose of this pull request?

  • Bug fix

Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).

@dansreis dansreis merged commit 6eadc46 into main Jun 21, 2024
@dansreis dansreis deleted the remove-unecessary-memo branch June 21, 2024 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dansreis