original ) (raw )Created on 2013-04-03 11:27 by rhettinger , last changed 2022-04-11 14:57 by admin . This issue is now closed .
Messages (10)
msg185917 - (view) Author: Raymond Hettinger (rhettinger) *
Date: 2013-04-03 11:27
In an editor window, select Edit-Replace and click ReplaceAll. The dialog window stays open. Formerly, it cleared immediately which was useful. The new behavior is confusing and less useful.
msg185924 - (view) Author: Roger Serwy (roger.serwy) *
Date: 2013-04-03 14:11
That behavior was introduced as part of the patch I wrote a long time ago for #13052 . In retrospect that extra functionality went beyond the scope of that issue and should not have been included. Does anyone else think that the new behavior ought to be reverted? I don't mind reverting it. A related issue is #17511 which wants to keep the Find dialog open instead of closing.
msg185981 - (view) Author: Roger Serwy (roger.serwy) *
Date: 2013-04-04 00:15
The attached patch restores the original behavior.
msg185987 - (view) Author: Raymond Hettinger (rhettinger) *
Date: 2013-04-04 00:51
Thanks. Please go ahead and apply it. I'm teaching some Python classes right now using the 2.7.4rc1 and the students are tripping over the new behavior.
msg185991 - (view) Author: Roger Serwy (roger.serwy) *
Date: 2013-04-04 01:11
If students are having trouble, then it should be changed. This behavior is also in 3.2. Do applied patches in 2.7 tip go toward 2.7.4 or 2.7.5? (I don't know the mechanics of the release candidate process just yet.)
msg185996 - (view) Author: Benjamin Peterson (benjamin.peterson) *
Date: 2013-04-04 02:33
I'll handle it.
msg185997 - (view) Author: Roundup Robot (python-dev)
Date: 2013-04-04 02:38
New changeset edcfd18840cc by Benjamin Peterson in branch '2.7': close search and replace dialog after it is used (closes #17625 ) http://hg.python.org/cpython/rev/edcfd18840cc New changeset 36130b84cd22 by Benjamin Peterson in branch '3.3': close search and replace dialog after it is used (closes #17625 ) http://hg.python.org/cpython/rev/36130b84cd22 New changeset a6c2210220dd by Benjamin Peterson in branch 'default': merge 3.3 (#17625 ) http://hg.python.org/cpython/rev/a6c2210220dd
msg185998 - (view) Author: Benjamin Peterson (benjamin.peterson) *
Date: 2013-04-04 02:39
Over to Georg for 3.3 cherry picking.
msg186015 - (view) Author: Georg Brandl (georg.brandl) *
Date: 2013-04-04 09:08
Done, thanks.
msg186127 - (view) Author: Roundup Robot (python-dev)
Date: 2013-04-06 14:43
New changeset 13e5664c5d19 by Benjamin Peterson in branch '3.2': close search and replace dialog after it is used (closes #17625 ) http://hg.python.org/cpython/rev/13e5664c5d19 New changeset 7746d238c4bb by Benjamin Peterson in branch '3.3': close search and replace dialog after it is used (closes #17625 ) http://hg.python.org/cpython/rev/7746d238c4bb
History
Date
User
Action
Args
2022-04-11 14:57:43
admin
set
github: 61825
2013-04-06 14:43:56
python-dev
set
messages: +
2013-04-04 09:08:10
georg.brandl
set
messages: +
2013-04-04 02:39:15
benjamin.peterson
set
assignee: roger.serwy -> georg.brandl messages: +
2013-04-04 02:38:11
python-dev
set
status: open -> closednosy: + python-dev messages: + resolution: fixedstage: commit review -> resolved
2013-04-04 02:33:35
benjamin.peterson
set
messages: +
2013-04-04 01:11:06
roger.serwy
set
stage: patch review -> commit reviewmessages: + versions: + Python 3.2
2013-04-04 00:52:07
rhettinger
set
priority: high -> release blockernosy: + georg.brandl , larry , benjamin.peterson
2013-04-04 00:51:48
rhettinger
set
priority: normal -> highmessages: +
2013-04-04 00:16:00
roger.serwy
set
files: + issue17625.patch keywords: + patch messages: + stage: needs patch -> patch review
2013-04-03 14:11:07
roger.serwy
set
type: behaviorstage: needs patchmessages: + versions: + Python 3.3, Python 3.4
2013-04-03 11:27:24
rhettinger
create
