Author terry.reedy
Recipients cheryl.sabella, terry.reedy
Date 2017-11-21.18:46:00
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1511289960.9.0.213398074469.issue32100@psf.upfronthosting.co.za>
In-reply-to
Content
The issue is this:
1. PathBrowser subclasses ModuleBrowser.
2. ModuleBrowser.init() has the common code for initializing both.
3. #31460 changed the signature of ModuleBrowser.__init__ *and* .init.
4. I must not have tested pathbrowser after the .init change.  Bad.
5. test_pathbrowser needs a new gui test class that at least calls PathBrowser.__init__, which will call ModuleBrowser.init

3.6.3 is OK, 3.7.0a2 has this bug.  3.7.0a3 is due 11/27.  Patch should be merged before this with or without new unittests.
---

Module items can be expanded either by clicking [+] or double-clicking.  The latter also opens the file so that lines can be highlighted.  I have thought about disabling both, and opening a module browser instead, but I can see the usefulness of expanding several files one level deep to get an overview of parts or all of a package.  On the other hand, we might have double-click open both the module and a module browser.
History
Date User Action Args
2017-11-21 18:46:00terry.reedysetrecipients: + terry.reedy, cheryl.sabella
2017-11-21 18:46:00terry.reedysetmessageid: <1511289960.9.0.213398074469.issue32100@psf.upfronthosting.co.za>
2017-11-21 18:46:00terry.reedylinkissue32100 messages
2017-11-21 18:46:00terry.reedycreate