enthought / enthought/traitsui

Check for dragability in TabularEditor should not construct drag item

Open
#675 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
306
Forks
99
PR merge metrics
No merged PRs in 30d

Description

Currently with the qt4 toolkit, the check for whether an item can be dragged actually calls `get_drag` and checks if the result is None:

https://github.com/enthought/traitsui/blob/dad91d3079725c4495f760884a463d2750a40bff/traitsui/qt4/tabular_model.py#L157-L161

This can lead to inefficiencies when constructing the drag item is expensive.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.