classification
Title: IDLE: Dedenting with Shift+Tab
Type: enhancement Stage: test needed
Components: IDLE Versions: Python 3.6
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: terry.reedy Nosy List: Sean.Wolfe, Todd.Rovito, csabella, gpolo, irdb, kamek, kbk, louielu, rhettinger, roger.serwy, taleinat, terry.reedy
Priority: normal Keywords: patch

Created on 2003-02-27 13:08 by kamek, last changed 2017-06-16 17:57 by csabella.

Files
File name Uploaded Description Edit
common_dedent.diff gpolo, 2009-03-29 13:15 review
Pull Requests
URL Status Linked Edit
PR 2210 open louielu, 2017-06-15 08:39
Messages (12)
msg53784 - (view) Author: Daniel (kamek) Date: 2003-02-27 13:08
IDLE already provides quick block indenting, by 
selecting one or more lines and pressing Tab. So, it 
would make sense to have Shift+Tab to do the opposite 
work. While there's already Alt+[ (and Alt+] to indent), 
Shift+Tab is more intuitive, especially to those who are 
used to other Windows developer tools, like Visual 
Studio or TextPad.
msg53785 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2003-07-20 01:20
Logged In: YES 
user_id=80475

While you get used to it quickly, I've bumped in to this also.  
It is nice to have the control keys work the same across 
applications.
msg53786 - (view) Author: Daniel (kamek) Date: 2004-01-01 19:16
Logged In: YES 
user_id=539453

This can be considered solved in the newest IDLE versions,
as you can set custom shortcuts.
msg53787 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2004-01-01 20:47
Logged In: YES 
user_id=80475

Evenso, it may be better to have it work right out of the box.
msg53788 - (view) Author: Kurt B. Kaiser (kbk) * (Python committer) Date: 2004-01-02 00:41
Logged In: YES 
user_id=149084

What I'd really like is smart tabs like python-mode, where a tab
(re)indents the line to the correct point based on the syntax,
and a backspace backs up to the previous level.  

Since IDLE already knows how to indent I'm hoping that that 
won't be to difficult to implement.
msg84392 - (view) Author: Guilherme Polo (gpolo) * (Python committer) Date: 2009-03-29 13:15
Patch added for allowing, and defining, Shift-Tab as the default binding
for dedenting.

The problem is that dedent-region may do more than one would expect.
msg150059 - (view) Author: Roger Serwy (roger.serwy) * (Python committer) Date: 2011-12-22 00:54
I applied the patch and encountered a problem. MultiCall.py is replacing "<Shift-Key-Tab>" with "<Shift-KeyPress-Tab>". When this happens, the logic for rebinding "<<PrevWindow>>" fails.

event_info('<<PrevWindow>>') returns ('<Key-ISO_Left_Tab>', '<Key-hpBackTab>', '<Shift-Key-Tab>') 

event_info('<<dedent-region>>') returns ('<Control-KeyPress-bracketleft>', '<Shift-KeyPress-Tab>')

This is with 3.3a0 on Ubuntu 11.04.
msg212906 - (view) Author: Sean Wolfe (Sean.Wolfe) Date: 2014-03-07 22:25
I did a couple tests and the shift-tab and tab work pretty much as expected. There's a small quirk for a single-line edit:

* place cursor on beginning of line
* tab forward
--> the text indents as expected
* shift-tab
--> the entire line is highlighted
--> the cursor now appears beneath the line
--> subsequent typing however affects the highlighted line

For the single-line case, it would be cleaner to have it stay on the same line without highlighting.

This is OSX 10.9, python 2.7.6+ and 3.4.rc1+
msg220193 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2014-06-10 19:51
Regarding Roger Serwy's comment, we could either:

1) normalize the event descriptions using MultiCall's _parse_sequence() and _triplet_to_sequence() functions
2) remove any Shift-Tab bindings from <<PrevWindow>>

I suggest #2.


Regarding the patch itself:

1) Was the existing default binding of Ctrl-[ removed on purpose in Lib/idlelib/configHandler.py? According to the same patch, in the config files it is still there along-side the new Shift-Tab binding. At the least, these should be consistent.

2) In the code re-binding the <<Prev-Window>> event in EditorWindow.py, shouldn't there be a "break" at the end of the "if" clause inside the "for" loop?
msg220197 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2014-06-10 21:27
The fact that Tab indents regions as well as lines seems not be documented. This should be added to Automatic Indentation (which should just be Indentation) with BackTab doc addition if not sooner.

On Windows, 3.4.1, shift-tab is the same as tab. The same is true in Notepad (not a model to follow), but Notepad++ dedents. Doing the same in Idle, within the editor, seems like a reasonable request.

The unplanned and strange OSX behavior should be fixed.

From the patch, I would gather that the problem is that tk hard-codes shift-Tab to <<prev-window (and, I would expect, tab to <<next-window>>). (This works but is currently useless in the dialogs.) If that is so, why do they currently act the same (for me) in the editor? Does tk automatically rebind Tab to indent-region in a text widget?

Tal: option 2 seems like something to try. If Shift-Tab is user configurable (and I think baked-in like Tab would be ok), then it should be *added* in both places. You might be right about break, except this is a moot point if the code is wrong.
msg296074 - (view) Author: Louie Lu (louielu) * Date: 2017-06-15 08:40
PR 2210 add the smart-dedent to shift-tab in Windows, macOS, Linux.

I can't saw <<prev-window>> nor <<next-window>> in 3.7 (master) source code, so I'll take it as removed in IDLE somewhere else.
msg296139 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2017-06-16 00:18
<<PrevWindow>> is a predefined virtual event. 
https://www.tcl.tk/man/tcl8.6/TkCmd/event.htm
I can't find <<NextWindow>> listed there. I don't remember if I actually saw it before mentioning it.  While I like mimicking smart_indent, I don't know if we can dismiss Roger Serwy's concern in msg150059.

Tk on Ubuntu may report key name ISO_Left_Tab msg150059, but it is not cross-platform.  On Win 10 with either tk 8.5.15 (2.7) or 8.6.6 (3.6):
  _tkinter.TclError: bad event type or keysym "ISO_Left_Tab"
I don't think we need to use the only sometimes valid synonym.

As Sean Wolfe reported msg212906, Shift-Tab already dedents a single line by treating it as a region.  This issue is about removing the extra behavior we don't want, and blocking Control-[ from doing simplefied non-region single line dedents.  Smart_indent does this by intercepting Tab (which I believe need not be bound to <<indent-region>>.

When there is a selection within a line, smart_indent does not call dedent_region but deletes the selection and insert \t or spaces, but in the latter case, the cursor may not be left on a multiple of indent spaces.  This does not seem very useful. It is even less so because if the selection is within a word, tab invokes autocomplete instead of smart_indent.  This needs discussion on another issue.
---

Text editing tests: how to we do them both thoroughly and efficiently (in terms of both human writing and machine execution time)?

White box approach: Set up a class test fixture that defines cls.text = <MultiCall(Text) instance>; put text in known state (content, cursor position, selection); call edit function (smart_indent, proposed smart dedent, etc); check resulting state (content, cursor, selection.  Repeat for multiple input state - output state pairs.  These tests break if the function name is changed and only test the function, which is, however, the main thing to be tested. 

Gray-box approach: Replace function calls with pseudoevents generated on the text widget.  Such tests would survive function name changes, break on pseudoevent name changes, and add a test of pseudoevent -- function binding. A black-box approach replaces pseudoevents with concrete events -- simulated key presses, mouse clicks, and menu selections.  These tests survive pseudoevent name changes and add a test of concrete -- pseudo event linkage.  They should only break if we change the user visible interface, which we seldom do. 

I think we should use all approaches in two sets of tests.  The first set would test multiple before and after states in subtests with a function call.  This would define and test the intended behavior of the function.  The second set would test one before and after state successively with a function call, pseudoevent generation, and concrete event generations, in this order.  (If the function call fails, the event tests are bound to fail.)  This would test the user interface and the links to the underlying function.

There are over 30 editing functions to test, and writing the above for all would be tedious.  After writing tests for a couple, we should write helper functions that embody the repetitive boilerplate.

An additional factor: The text edit functions, properly bound with a pseudoevent to the text instance, are not methods of the text instance (though they should be), but of the EditorWindow that also contains a Toplevel; Menu hierarchy; and a Frame that contains Scrollbars and a StatusBar, along with, finally, the MultiCall(Text) instance.  So the fixture setup must also provide provide an attribute for the method container.  Let us call it cls.widget.

In order to put EditFrames (a new subclass of Frame) on tabs, the Text methods will have to at least be transferred to the EditFrame.  It would be even better to transfer them to an IdleText subclass of MultiCall(Text) or possibly of Text.  (See Mark Roseman's suggestion of a MultiCall alternative in msg272070.)  After the setup code is changed to assign the new method container to cls.widget, the test methods should still run as they are.
History
Date User Action Args
2017-06-16 17:57:29csabellasetnosy: + csabella
2017-06-16 00:18:13terry.reedysetpriority: low -> normal
versions: + Python 3.6, - Python 2.7, Python 3.4, Python 3.5, Python 3.7
messages: + msg296139

assignee: terry.reedy
stage: patch review -> test needed
2017-06-15 08:40:40louielusetnosy: + louielu

messages: + msg296074
versions: + Python 3.7
2017-06-15 08:39:03louielusetpull_requests: + pull_request2254
2016-02-13 12:03:17irdbsetnosy: + irdb
2014-06-10 21:27:47terry.reedysetassignee: kbk -> (no value)
messages: + msg220197
versions: + Python 2.7, Python 3.4, Python 3.5, - Python 3.2
2014-06-10 19:51:33taleinatsetnosy: + taleinat
messages: + msg220193
2014-03-07 22:25:13Sean.Wolfesetnosy: + Sean.Wolfe
messages: + msg212906
2014-03-05 23:38:07terry.reedysetnosy: + terry.reedy
2013-03-25 02:10:50Todd.Rovitosetnosy: + Todd.Rovito
2011-12-22 00:54:14roger.serwysetnosy: + roger.serwy
messages: + msg150059
2010-08-09 04:37:58terry.reedysettitle: Dedenting with Shift+Tab -> IDLE: Dedenting with Shift+Tab
stage: patch review
versions: + Python 3.2, - Python 3.1, Python 2.7
2009-03-29 13:16:10gpololinkissue1562092 superseder
2009-03-29 13:15:03gpolosetfiles: + common_dedent.diff
versions: + Python 3.1, Python 2.7, - Python 2.6
nosy: + gpolo

messages: + msg84392

keywords: + patch
2008-01-05 18:09:32christian.heimessetpriority: normal -> low
versions: + Python 2.6
2003-02-27 13:08:54kamekcreate