GNU nano - Bugs: bug #51923, cursor flashes to bottom of the...
You are not allowed to post comments on this tracker with your current authentication level.
bug #51923: cursor flashes to bottom of the screen when editing
Submitter: | Mike Frysinger <vapier> | ||
Submitted: | Sun 03 Sep 2017 12:44:44 AM UTC | ||
Severity: | 3 - Normal | Status: | Fixed |
Assigned to: | bens | Open/Closed: | Closed |
Jump to the original submission
Sun 19 Nov 2017 08:13:44 AM UTC, comment #20: |
Benno Schulenberg <bens> |
Wed 20 Sep 2017 07:59:03 PM UTC, comment #19: The flashing on the bottom row should be fixed in git: ae34825f. |
Benno Schulenberg <bens> |
Wed 20 Sep 2017 07:23:47 PM UTC, comment #18: Thanks for testing. But... the problem of ^R ^T ^W ^C not switching the cursor back on (when using --show) is present in master too. Please report it as a separate bug. And the problem of the cursor getting hidden/misplaced when doing <Up> at the top of the file list or <Down> at the bottom, is present in master also. Please report that as a separate bug too.
|
Benno Schulenberg <bens> |
Mon 18 Sep 2017 08:36:17 PM UTC, comment #17: ...and in (b), "the same problem in (a)" means that the cursor isn't on the file it should be. |
David Lawrence Ramsey <dolorous> |
Mon 18 Sep 2017 08:28:06 PM UTC, comment #16: Check that; effect 2 sometimes doesn't occur depending on whether the cursor is on or off. More precisely, run nano with the aforementioned options, press Ctrl-W Ctrl-T, and then press Ctrl-G immediately afterward. Or, run nano with the aforementioned options, press Ctrl-W Ctrl-T, and then press Ctrl-W Ctrl-G immediately afterward; the same problem occurs in the file search help browser.
|
David Lawrence Ramsey <dolorous> |
Mon 18 Sep 2017 08:14:31 PM UTC, comment #15: I've done some quick testing with the patch in comment #14. With the patch applied to git fd86696, run:
|
David Lawrence Ramsey <dolorous> |
Mon 18 Sep 2017 07:34:20 PM UTC, comment #14: Here is an updated patch that fixes the case of Unicode input. |
Benno Schulenberg <bens> |
Mon 18 Sep 2017 06:56:04 PM UTC, comment #13: Attached is a proposed patch. It works by keeping the cursor off all the time that we are not waiting for input.
|
Benno Schulenberg <bens> |
Fri 15 Sep 2017 12:05:50 AM UTC, comment #12: You can also run script outside of screen/tmux to see what the terminal emulator sees :) Of course it has its limitations and inconveniences too, I just wanted to make sure that you're aware of this useful tool.
|
Egmont Koblinger <egmont> |
Thu 14 Sep 2017 11:55:49 PM UTC, comment #11: script would work too, but it's not as useful (imo) for on the fly testing. it'll give you a full session, but doesn't make it easy to narrow down events.
|
Mike Frysinger <vapier> |
Thu 14 Sep 2017 08:20:10 PM UTC, comment #10: FYI: Instead of adding debug stuff or strace'ing, I'm wondering if you guys have heard of script(1), probably the easiest and simplest way of recording the app's output.
|
Egmont Koblinger <egmont> |
Wed 13 Sep 2017 07:52:14 PM UTC, comment #9: When I catch a strace, and press and hold j, sequences like these get repeated over and over again:
|
Benno Schulenberg <bens> |
Wed 13 Sep 2017 07:07:10 AM UTC, comment #8: i added debug code to the terminal that dumps the exact sequences transmitted :)
|
Mike Frysinger <vapier> |
Tue 12 Sep 2017 07:09:21 PM UTC, comment #7: How do you catch the codes that ncurses sends to the terminal? |
Benno Schulenberg <bens> |
Thu 07 Sep 2017 03:58:17 PM UTC, comment #6: here's exactly what the terminal sees when i press "j" once in an empty buffer. the terminal is 202 rows x 58 cols, and i'm not using screen/tmux/etc... TERM is set to xterm-256color and i'm using hterm.
|
Mike Frysinger <vapier> |
Tue 05 Sep 2017 08:38:41 PM UTC, comment #5: i had traced the escape sequences emitted a bit to try and double check it wasn't a terminal emulation bug. i'll give it another try to double check.
|
Mike Frysinger <vapier> |
Sun 03 Sep 2017 10:59:25 PM UTC, comment #4: For the record, I can't reproduce the original problem in the first place. I've tried with Xfce-terminal and Konsole (regarding the latter, I was finally able to get KDE5 installed), both with blinking cursors, and with xterm without a blinking cursor; I've also tried it with lines past 100000 columns. If it's an ncurses bug, I'm using ncurses 5.9. |
David Lawrence Ramsey <dolorous> |
Sun 03 Sep 2017 08:48:02 PM UTC, comment #3: Attached patch should reduce the distraction on regular-sized terminals, because normally the --constantshow line will fit fine on the status bar. So the patch activates the workaround only when the message is close to filling the entire row.
|
Benno Schulenberg <bens> |
Sun 03 Sep 2017 11:34:59 AM UTC, comment #2: Okay, I have been able to reproduce. But it is quite rare: in most cases the cursor is already beyond column 500 before I see the first flash. On my old laptop it is more frequent: half of the time the first flash is within a handful of seconds, and it occurs both on Gnome Terminal and on a Linux console (I didn't check on xterm). It also seems to occur more easily when using a blinking cursor (my default cursor is steady).
|
Benno Schulenberg <bens> |
Sun 03 Sep 2017 09:34:27 AM UTC, comment #1: I don't see this flashing. Not on an Xfce terminal, nor on an xterm nor on a Linux console (VT). What terminal are you using?
|
Benno Schulenberg <bens> |
Sun 03 Sep 2017 12:44:44 AM UTC, original submission:
since commit b93767b54fd55112ea091f0c906c9fc7089f263f (screen: defeat a VTE/Konsole bug also for the case of --constantshow), when editing files, i see the cursor briefly flash to the bottom left corner of the screen before moving back to its current position when editing content. before the commit, it would flash to the end of the status bar and back up which was less distracting.
|
Mike Frysinger <vapier> |
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-11-19 | bens | Open/Closed | Open | Closed | |
2017-09-20 | bens | Status | In Progress | Fixed | |
2017-09-18 | bens | Attached File | - | Added 0001-display-rework-the-switching-on-and-off-of-the-curso.patch, #41831 | |
2017-09-18 | bens | Attached File | - | Added 0001-display-rework-the-switching-on-and-off-of-the-curso.patch, #41830 | |
2017-09-13 | bens | Attached File | - | Added switch-cursor-on-AFTER-screen-has-been-updated.patch, #41799 | |
2017-09-03 | bens | Attached File | - | Added reduce-distraction.patch, #41725 | |
Status | None | In Progress | |||
Assigned to | None | bens | |||
2017-09-03 | bens | Attached File | - | Added dont-show-the-cursor-at-all.patch, #41721 | |
Carbon-Copy | - | Added dolorous |
Powered by Savane 3.13-d3ae.
Corresponding source code
Released in 2.9.0.