Skip to content

Conversation

@titan73
Copy link
Contributor

@titan73 titan73 commented Oct 16, 2025

Currently clicking on a violation in the drc viewer zooms to the violation in the renderer and switch to the inspector.
When quickly browsing several violations, we always have to switch back to the drc viewer for the next violation which is annoying.

This PR makes the changes:

  • single click : Only zooms to the violation
  • double click : Zooms to the violation and switch to the inspector

…in drc viewer.

Signed-off-by: Christian COSTA <christian.costa@st.com>
@titan73
Copy link
Contributor Author

titan73 commented Oct 16, 2025

I can add an option to enable the new behavior if needed. Let me know.

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@maliberty
Copy link
Member

This makes sense to me. @gadfort any concern?

@gadfort
Copy link
Collaborator

gadfort commented Oct 16, 2025

@maliberty no, makes sense me as well

@maliberty maliberty merged commit 66bcec9 into The-OpenROAD-Project:master Oct 16, 2025
13 checks passed
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.

3 participants