Skip to content

Conversation

@JerryVincent
Copy link
Collaborator

@JerryVincent JerryVincent commented Jan 21, 2026

Type of Change

  • Dependency upgrade
  • [ x ] Bug fix (non-breaking change)
  • Breaking change
    • e.g. a fixed bug or new feature that may break something else
  • New feature
  • Code quality improvements
    • e.g. refactoring, documentation, tests, tooling, ...

Implementation

In the app\components\device\new\new-device-stepper.tsx file I have modified the TooltipTrigger Component in line no:329 to make it a button, other than considering it as a submit button. In the app\components\device\new\general-info.tsx file, I did the same in line no: 124 and made the TooltipTrigger a normal button and prevented it from triggering the unwanted submits. It might seem like the entire file has been modified, but it is because of the formatting issue. Please ignore it..😌

Checklist

  • [ x ] I gave this pull request a meaningful title
  • [ x ] My pull request is targeting the dev branch
  • [ x ] I have added documentation to my code
  • [ x ] I have deleted code that I have commented out

Additional Information

…he info button, while creating a new device.
Copy link
Contributor

@jona159 jona159 left a comment

Choose a reason for hiding this comment

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

Code changes are approved! Please make sure to only write comments that provide additional information that cannot be derived from the code itself :)

e.stopPropagation()
}}
>
{/*Treat it as a normal button, rather than treating it as a Submit button*/}
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as below

e.preventDefault()
e.stopPropagation()
}}
>{/*Covert it to button, so that it is not confused as the Submit button*/}
Copy link
Contributor

Choose a reason for hiding this comment

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

The comment is not necessary here as the code change itself is self-explanatory, please remove it

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Alright Thanks:)

@JerryVincent JerryVincent merged commit 961e404 into dev Jan 21, 2026
6 checks passed
@JerryVincent JerryVincent deleted the fix/PreventUnwantedJumps branch January 21, 2026 11:46
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.

device creation: click on info icon jumps to Location-step

3 participants