Skip to content

Resolve issue where highlighting Job Output text can cause the event popup#298

Merged
cigamit merged 2 commits intomainfrom
react-clickable
Apr 9, 2026
Merged

Resolve issue where highlighting Job Output text can cause the event popup#298
cigamit merged 2 commits intomainfrom
react-clickable

Conversation

@cigamit
Copy link
Copy Markdown
Contributor

@cigamit cigamit commented Apr 9, 2026

SUMMARY

Its always been a problem that if I wanted to highlight a small section of text in a Job Output event, if that event is clickable (has even data), then highlighting the text will cause the even popup to fire.

This resolves this by checking for this scenario. It will still fire if you highlight the text and then deliberately click the event. It will not fire if only highlighting the text.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • UI

@cigamit cigamit requested a review from TheWitness April 9, 2026 07:38
@cigamit cigamit self-assigned this Apr 9, 2026
@cigamit cigamit added the bug Something isn't working label Apr 9, 2026
Copilot AI review requested due to automatic review settings April 9, 2026 07:38
@cigamit cigamit added the javascript Pull requests that update javascript code label Apr 9, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes an AWX UI Job Output interaction bug where selecting/highlighting text in a clickable job event line unintentionally triggers the host event popup.

Changes:

  • Adds a custom click handler that checks for active text selection before invoking the job event click action.
  • Updates the JobEvent component to use the new handler on clickable event lines.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread awx/ui/src/screens/Job/JobOutput/JobEvent.js
Comment thread awx/ui/src/screens/Job/JobOutput/JobEvent.js
@cigamit cigamit merged commit 2571e80 into main Apr 9, 2026
@cigamit cigamit deleted the react-clickable branch April 9, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working javascript Pull requests that update javascript code

Development

Successfully merging this pull request may close these issues.

3 participants