enthought / enthought/traitsui

Qt ListStrEditor and TabularEditor setx methods use non existing _no_notify attribute

Open
#867 1 comment 0 reactions 0 assignees View on GitHub
toolkit: Qt
Dominant language
Python
Stars
306
Forks
99
PR merge metrics
No merged PRs in 30d

Description

Instead of `_no_notify` this should use `_no_update`, which is the attribute used elsewhere in the editor to prevent updates.

https://github.com/enthought/traitsui/blob/835ed3516def0e961d437e43f36c033ba3a00cb1/traitsui/qt4/list_str_editor.py#L238-L247

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the linked section of traitsui/qt4/list_str_editor.py and locate the Qt ListStrEditor and TabularEditor setx methods. Verify that both refer to the editor attribute used elsewhere to prevent updates, then run the relevant editor tests to confirm the methods no longer reference a nonexistent attribute.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
desktop
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.