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

Fix middle-click copy #782

Merged
merged 2 commits into from
Oct 20, 2020

Conversation

DiegoTavares
Copy link
Collaborator

Fixes issue preventing the middle-click copy feature from working on JobTree widgets

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 18, 2020

CLA Check
The committers are authorized under a signed CLA.

  • ✅ Diego Tavares da Silva (924dd62)

(cherry picked from commit 402169aa959f0426883ceecce1e6564efeff81d9)
@@ -186,9 +186,9 @@ def __itemSingleClickedCopy(self, item, col):
@param item: The item clicked on
@type col: int
@param col: The column clicked on"""
selected = [job.data.name for job in self.selectedObjects() if cuegui.Utils.isJob(job)]
selected = [job.data.name for job in self.selectedObjects() if cuegui.Utils.isJob(job)]
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: Extra whitespace at the end of this line, looks like.

@bcipriano bcipriano merged commit babaa38 into AcademySoftwareFoundation:master Oct 20, 2020
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.

None yet

2 participants