[issue40511] IDLE: properly handle '(' and ')' within calls

[wyz23x2]

wyz23x2  added the comment:

Thanks! :D

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[Terry J. Reedy]

Terry J. Reedy  added the comment:

wyz, thank you for the report.  It should be fixed now.

--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[miss-islington]

miss-islington  added the comment:


New changeset 1341582e165841810e2fbf89e23be0e92b4a9fdd by Miss Skeleton (bot) 
in branch '3.8':
bpo-40511: Stop unwanted flashing of IDLE calltips (GH-20910)
https://github.com/python/cpython/commit/1341582e165841810e2fbf89e23be0e92b4a9fdd


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[miss-islington]

miss-islington  added the comment:


New changeset 79e9f06149f92798a8e11e3f1c62dad171312ab3 by Miss Skeleton (bot) 
in branch '3.9':
bpo-40511: Stop unwanted flashing of IDLE calltips (GH-20910)
https://github.com/python/cpython/commit/79e9f06149f92798a8e11e3f1c62dad171312ab3


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[miss-islington]

Change by miss-islington :


--
pull_requests: +22009
pull_request: https://github.com/python/cpython/pull/23094

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[Terry J. Reedy]

Terry J. Reedy  added the comment:


New changeset da7bb7b4d769350c5fd03e6cfb16b23dc265ed72 by Tal Einat in branch 
'master':
bpo-40511: Stop unwanted flashing of IDLE calltips (GH-20910)
https://github.com/python/cpython/commit/da7bb7b4d769350c5fd03e6cfb16b23dc265ed72


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[miss-islington]

Change by miss-islington :


--
nosy: +miss-islington
nosy_count: 4.0 -> 5.0
pull_requests: +22008
pull_request: https://github.com/python/cpython/pull/23093

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[Tal Einat]

Tal Einat  added the comment:

See PR GH-20910 with a fix.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[Tal Einat]

Change by Tal Einat :


--
keywords: +patch
pull_requests: +20092
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/20910

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[Tal Einat]

Tal Einat  added the comment:

Confirmed on latest master on macOS 10.15.5 with Tcl/Tk 8.6.10.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[Tal Einat]

Tal Einat  added the comment:

> OMG, request too large. I can't upload mp4

You can try to reduce the video's resolution and/or framerate in order to 
reduce its size.

Usually screen captures are taken at high resolution and framerates by default, 
making the resulting video files very large, which is a sensible default but 
not great for attaching to bug reports.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[Tal Einat]

Tal Einat  added the comment:

I agree that this should be improved.

Terry, on the technical side, IDLE already has good tools for parsing the 
current code line/block and figuring out whether the cursor is in a string, a 
comment or parentheses. This should be possible to solve in a robust way 
without writing any new parsing logic.

I'll try to take a look at this soon.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40511] IDLE: properly handle '(' and ')' within calls

[Terry J. Reedy]

Terry J. Reedy  added the comment:

Each  '(' and ')' in an argument string such as in 
f('()(()()())') causes a blink.  Ditto with a comment withing 
the call.  A related bug is that any unquoted '(', as when one enters 
f((a+b)*c), dismisses the tip. 

Proposed partial solution: before displaying a tip, record the opening index 
(before the opening '(').  When there is an opening index, and '(' is keyed, 
ignore it; no parsing.  When the tip is closed (properly), removed the open 
index.  I believe paren matching would work properly.

The cause of blinks for ')' in strings and comments would need to be found and 
remedied.

--
nosy: +cheryl.sabella, taleinat
title: IDLE yellow hint frame blinks when entering () in strings in 
functions/classes -> IDLE: properly handle '(' and ')' within calls
versions: +Python 3.10 -Python 3.7, Python 3.8, Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com