Author Saimadhav.Heblikar
Recipients JayKrish, Saimadhav.Heblikar, Todd.Rovito, jesstess, ned.deily, rhettinger, roger.serwy, serhiy.storchaka, taleinat, terry.reedy
Date 2014-08-16.06:24:51
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1408170293.45.0.896126445141.issue17535@psf.upfronthosting.co.za>
In-reply-to
Content
@ Ned Deily:
Thank you for the comment's.

1. I documented it in the config-extensions.def how to make it visible on startup(or not). In the new patch, it is not visible by default.

2. I have made the changes.

4. I'll explain the reason why this argument is required:
Without this argument, if a user "selects" the sidebar Text widget, a highlight can be seen around the selected area. Adding this argument prevents it showing up.
The new patch prevents this unknown option error, by adding the inactiveselectbackground only if Tk version >= 8.5

(Because of this issue which you raised, I found a new bug and fixed it - see below.)

5. Done

6. I added a comment in the config-extensions.def file. The enable_shell option has been removed for now. It is not related to linenumbering.

@Terry Reedy:

1. Done. Disabled by default

4. I mentioned the reason why it is required above.

------------
New bug which was discovered because of 4 and fixed:
Scrolling on the sidebar text widget using mouse by dragging would scroll just the sidebar text widget and not the main text widget. This has been fixed by adding  self.text.yview_moveto(args[0]) in vbar_set.
History
Date User Action Args
2014-08-16 06:24:53Saimadhav.Heblikarsetrecipients: + Saimadhav.Heblikar, rhettinger, terry.reedy, taleinat, ned.deily, roger.serwy, jesstess, Todd.Rovito, serhiy.storchaka, JayKrish
2014-08-16 06:24:53Saimadhav.Heblikarsetmessageid: <1408170293.45.0.896126445141.issue17535@psf.upfronthosting.co.za>
2014-08-16 06:24:53Saimadhav.Heblikarlinkissue17535 messages
2014-08-16 06:24:53Saimadhav.Heblikarcreate