refactor: form el for ConfirmorCancel #1617

[vaibhavgarg237]

Describe the changes you have made in this PR

Wrapped all the ConfirmorCancel components forms with forms and handling submit operation via onSubmit instead of onClick

Link this PR to an issue [optional]

Fixes #1617

Type of change

(Remove other not matching type)

• fix: Bug fix (non-breaking change which fixes an issue)

Screenshots of the changes [optional]

Refactored code

Checklist

• My code follows the style guidelines of this project and performed a self-review of my own code
• New and existing tests pass locally with my changes
• I checked if I need to make corresponding changes to the documentation (and made those changes if needed)

[github-actions]

🚀 Thanks for the pull request!

Here are the current build files for testing:

• chrome.zip.zip
• firefox.xpi.zip
• opera.crx.zip

Download and unzip the file for your browser. Refer to the readme for detailed install instructions.

Don't forget: keep earning sats!

[escapedcat]

ack

[vaibhavgarg237]

ack

You are a machine! 🚀🚀

[escapedcat]

@vaibhavgarg237 please remove the forms where we do not have any inputs. Talked to @im-adithya and we think this is confusing. But we should always use forms (and formfields if useful) if we actually have input-fields.

Thanks for your contribution!

See last comment

[vaibhavgarg237]

@vaibhavgarg237 please remove the forms where we do not have any inputs. Talked to @im-adithya and we think this is confusing. But we should always use forms (and formfields if useful) if we actually have input-fields.

Thanks for your contribution!

Alright, I have updated the required changes.