This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author terry.reedy
Recipients Saimadhav.Heblikar, Sarah, r.david.murray, terry.reedy
Date 2014-12-19.04:53:58
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1418964839.56.0.301471546789.issue18590@psf.upfronthosting.co.za>
In-reply-to
Content
I am narrowing the scope of this issue to Replace dialogs and widening the scope to output as well as edit windows. The attached patch solves the issue as redefined.  (It has a temporary hack to pass the unittests.)

For edit windows, the problem is that the default tag stacking order seems to be alphabetical.  No tag, 'builtin, 'comment', and 'definition, are followed and dominated by 'hit'; 'keyword' and 'string' come after and dominate 'hit'.  The solution is to raise 'hit' to the top.  The test (from msg225543), which should be added to htest), is that all 6 'i's in "def i(): this list is 'is' # is not" are both found and highlighted.  They are with the patch.

For output windows, the problem, mentioned in msg225382, is that the 'hit' tag is configured in ColorDelegator, which is not used in output windows.  Ths solution, also mentioned there, is to move the configuration to SearchDialogBase.  The path does this and Replace dialog Find work for Output Windows.

I tested on Windows.  I am 99.9% sure there should be no problem on other systems, but would like confirmation on other systems before or after committing an expanded patch with test changes added.
History
Date User Action Args
2014-12-19 04:53:59terry.reedysetrecipients: + terry.reedy, r.david.murray, Sarah, Saimadhav.Heblikar
2014-12-19 04:53:59terry.reedysetmessageid: <1418964839.56.0.301471546789.issue18590@psf.upfronthosting.co.za>
2014-12-19 04:53:59terry.reedylinkissue18590 messages
2014-12-19 04:53:58terry.reedycreate