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

Cannot Delete Bug Report: Unable to Hover, Select, or Delete Wire Between Global Input and Output Pins #21

Open
EV-OD opened this issue Jul 16, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@EV-OD
Copy link
Owner

EV-OD commented Jul 16, 2024

Bug Report: Unable to Hover, Select, or Delete Wire Between Global Input and Output Pins

Description:
Currently, it is not possible to hover over, select, or delete a wire that connects a global input pin to a global output pin within the application.

Steps to Reproduce:

  1. Open the application and create a global input pin.
  2. Connect the global input pin to a global output pin using a wire.
  3. Attempt to hover over the wire, select it, or delete it using standard UI interactions.

Expected Behavior:

  • Users should be able to hover over the wire to view its details or properties.
  • Users should be able to select the wire to modify or delete it.
  • There should be no restrictions on interacting with wires between global input and output pins compared to other wire connections.

Actual Behavior:

  • Hovering over the wire between global input and output pins does not display any details or properties.
  • There is no option to select the wire to modify or delete it directly from the interface.

Environment:

  • OS: Win 11

Additional Context:
This issue limits the ability to manage connections between global pins, which is crucial for designing and debugging complex circuits. It affects usability and workflow efficiency.

Screenshots or Videos:
image

Possible Solutions:

  • Ensure that wires between global input and output pins support standard hover, selection, and deletion functionalities.
  • Review and update the wire interaction logic to include global pin connections in all relevant UI operations.

Development Notes:

  • Verify that the fix does not introduce regressions or affect other wire connections.
  • Test the solution thoroughly across different scenarios and user workflows to ensure consistent behavior.
@EV-OD EV-OD added the bug Something isn't working label Jul 16, 2024
@pk-021
Copy link
Contributor

pk-021 commented Jul 29, 2024

#28 Fixes this issue by completing the relevant code. The code changes can be reviewed in the pull request.
The changes have been made ensuring to new bugs are introduced and the wire deletions and corresponding binds are properly handled to prevent memory leaks and unwanted bugs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants