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.

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

Remove unused CloseButton.jsx #7411

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Remove unused CloseButton.jsx #7411

wants to merge 1 commit into from

Conversation

pjonsson
Copy link
Contributor

What this PR does

There is a CloseButton.tsx
in the same directory, so remove
the jsx file.

Test me

I believe this is tested by CI.

Checklist

  • There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
  • I've updated relevant documentation in doc/.
  • I've updated CHANGES.md with what I changed.
  • I've provided instructions in the PR description on how to test this PR.

@pjonsson pjonsson mentioned this pull request Jan 2, 2025
Open
4 tasks
@pjonsson pjonsson mentioned this pull request Feb 28, 2025
Merged
4 tasks
@pjonsson
Remove unused CloseButton.jsx
8bb5997 
There is a CloseButton.tsx
in the same directory, so remove
the jsx file.
@pjonsson pjonsson force-pushed the remove-unused-closebutton branch from b4340c0 to 8bb5997 Compare March 8, 2025 10:41
@pjonsson pjonsson mentioned this pull request Mar 8, 2025
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zoran995 zoran995 zoran995 approved these changes

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
@pjonsson @zoran995