GNU nano - Bugs: bug #51335, in some terminals, the cursor can...
You are not allowed to post comments on this tracker with your current authentication level.
bug #51335: in some terminals, the cursor can be misplaced with --softwrap --nohelp --constantshow
Submitter: | David Lawrence Ramsey <dolorous> | ||
Submitted: | Wed 28 Jun 2017 04:54:16 PM UTC | ||
Severity: | 2 - Minor | Status: | Fixed |
Assigned to: | bens | Open/Closed: | Closed |
Jump to the original submission
Fri 27 Nov 2020 01:25:07 PM UTC, comment #9: |
Benno Schulenberg <bens> |
Fri 30 Jun 2017 08:33:06 AM UTC, comment #8: Fixed in git, b93767b5. Thanks for reporting. |
Benno Schulenberg <bens> |
Thu 29 Jun 2017 07:42:41 PM UTC, comment #7: Yes, it does. Thanks. |
David Lawrence Ramsey <dolorous> |
Thu 29 Jun 2017 07:32:12 PM UTC, comment #6: Okay. Attached is more economical version of the patch: it avoids an extra wrefresh() call. Does this fix the issue too? |
Benno Schulenberg <bens> |
Thu 29 Jun 2017 06:27:55 PM UTC, comment #5: Yes, it does fix the problem for both xfce-terminal and Konsole. And if the fix is that simple, I say go for it.
|
David Lawrence Ramsey <dolorous> |
Thu 29 Jun 2017 06:10:06 PM UTC, comment #4: The fix is easy enough: just do a cursor move and another screen update, just like at the end of bottombars(). See attached patch. Does this fix the problem also for your current Konsole?
|
Benno Schulenberg <bens> |
Thu 29 Jun 2017 04:24:39 PM UTC, comment #3: Good to know. If it's fixed on more modern systems, it may not be worth the effort, unless the fix turns out to be incredibly simple and won't unnecessarily clutter things.
|
David Lawrence Ramsey <dolorous> |
Thu 29 Jun 2017 08:15:21 AM UTC, comment #2: I've run the reproduction recipe
|
Benno Schulenberg <bens> |
Wed 28 Jun 2017 04:56:49 PM UTC, comment #1: Forgot one detail: since xfce-terminal is based on vte, the version of xfce-terminal that shows the bug is using vte 0.28.2. |
David Lawrence Ramsey <dolorous> |
Wed 28 Jun 2017 04:54:16 PM UTC, original submission:
This can be reproduced with current git (c0a5237) as follows:
|
David Lawrence Ramsey <dolorous> |
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 10 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-07-22 | bens | Open/Closed | Open | Closed | |
2017-06-30 | bens | Severity | 3 - Normal | 2 - Minor | |
Status | In Progress | Fixed | |||
Assigned to | None | bens | |||
Summary | in some terminals, the cursor can be misplaced with --softwrap --constant --nohelp | in some terminals, the cursor can be misplaced with --softwrap --nohelp --constantshow | |||
2017-06-29 | bens | Attached File | - | Added more-economical-version.patch, #41075 | |
2017-06-29 | bens | Attached File | - | Added defeat-another-VTE-KDE-bug.patch, #41074 | |
Status | None | In Progress | |||
Summary | in some terminals, the cursor can be misplaced with --softwrap --const --nohelp | in some terminals, the cursor can be misplaced with --softwrap --constant --nohelp | |||
2017-06-29 | bens | Summary | in some terminals. cursor can be misplaced with --softwrap, --const and --nohelp | in some terminals, the cursor can be misplaced with --softwrap --const --nohelp |
Powered by Savane 3.13-02a9.
Corresponding source code
Updated recipe... First comment out the lines in src/winio.c that "Defeat a VTE/Konsole bug" and recompile.
Then make your terminal 54 columns wide and run:
Then type <End>. See that the cursor is placed correctly.
Type: x ^S ^X, and run the same command again.
Type <End> and see that the cursor is placed correctly.
Do that last dance two more times. The cursor is placed fine. Both on Xfce Terminal 0.8.7.4 and Konsole 20.08.3.
So, for VTE, the bug has been fixed before libvte 0.52 (that is: before 2018). For Konsole, I'm going to guess that it was fixed before 2020, before ncurses-6.2, so that the workaround can be excluded for builds on modern systems.
The workaround has been conditionalized on ncurses before version 6.2 by commit fe22da6b.