Issue 24825: visual margin indicator for breakpoints in IDLE (original) (raw)

Created on 2015-08-07 14:16 by markroseman, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Messages (4)
msg248194 - (view) Author: Mark Roseman (markroseman) * Date: 2015-08-07 14:16
Right now breakpoints can only be set/cleared by using a context menu on a line in the editor. I discovered this entirely by reading through the bug database, as right-click doesn't work on OS X (#24801). Some other tools use an indicator (e.g. stop sign) in the left margin, which can be toggled on and off by a simple click. This could exist in conjunction with line numbering (#17535)
msg248223 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2015-08-07 20:07
#24801 needs to be fixed so we can depend on right clicking. I think SET and CLEAR should be replace by TOGGLE (#22083). The patch for #17535 already includes marking breakpoints in the margin. I am opposed to a putting a symbol on every line. Too noisy and too confusing to beginners for an advanced feature. There are other things that could be added to a line-number context menu, such as cutting or copying a line without having to drag select it (which is too easy to mess up).
msg248226 - (view) Author: Mark Roseman (markroseman) * Date: 2015-08-07 20:17
oh agree, definitely don't want a symbol on every line. i was already thinking implementation where you'd probably have an object matching the background for hit detection! :-) but yes, visually nothing there but empty space when there isn't a breakpoint, and a symbol there when the breakpoint is set
msg248234 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2015-08-07 23:10
I believe one of Saimadhav's original patches used a symbol. I suggested color highlighting instead, and it seems to work better. In any case, debating such details is part of an issue.
History
Date User Action Args
2022-04-11 14:58:19 admin set github: 69013
2015-08-07 23:10:33 terry.reedy set status: open -> closedsuperseder: IDLE editor line numbersmessages: + resolution: duplicatestage: resolved
2015-08-07 20:17:54 markroseman set messages: +
2015-08-07 20:07:46 terry.reedy set messages: +
2015-08-07 14:16:34 markroseman create