Bug 183993
Summary: | Enabling reverse video mode through the menu (ctrl + middle mousebutton) leaves cursor black together with background being black | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Joni Yrjana <joyr> |
Component: | xterm | Assignee: | Miroslav Lichvar <mlichvar> |
Status: | CLOSED ERRATA | QA Contact: | |
Severity: | low | Docs Contact: | |
Priority: | medium | ||
Version: | 4 | CC: | dickey, jessethilo |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | i386 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-08-17 15:54:46 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Joni Yrjana
2006-03-04 18:31:46 UTC
Seconded, in a slightly different situation. I use a white-on-black set of defaults, and my cursor always turns black at some point (turning on reverse video exposes the cursor). This didn't happen in xterm-208-1. $ xrdb -query | grep VT100 *VT100*backarrowKey: false *VT100*background: black *VT100*boldColors: true *VT100*boldFont: false *VT100*boldMode: off *VT100*colorBDMode: off *VT100*colorBLMode: off *VT100*colorULMode: off *VT100*cursorColor: grey *VT100*dynamicColors: false *VT100*eightBitInput: false *VT100*foreground: grey *VT100*geometry: 80x24 *VT100*jumpScroll: true *VT100*loginShell: true *VT100*ptyInitialErase: true *VT100*reverseWrap: true *VT100*saveLines: 2500 *VT100*scrollBar: true *VT100*underLine: on *VT100*utf8: true Yes, I see the problem - this is because of the fix to bug 178302, which was a complaint about xterm not setting the cursor color if *vt100*cursorColor is explicitly set to be the same as the foreground color . xterm was testing if the cursorColor is the same as the foreground color, and not setting the cursor to that color if so. But when ReverseVideo is enabled, the cursor color IS the same as the foreground color by default, so xterm would not set the cursor color to that value, leaving it the same as the background. I think xterm's logic needs to be a bit more sophisticated than just refusing to set the cursorColor if it is the same as the foreground . Perhaps xterm should only refuse to set the cursorColor to the foreground if ReverseVideo is in effect. I'll be fixing this with xterm-208-4.FC4 today . Something more complicated than what you're suggesting, but taking that into account. I've had a couple other complaints about this recently, will look into a short #210 to address this area. Looked into it - no: the bug is in a different place (changing set_cursor_gcs is incorrect). I'll add a call to set_cursor_gcs from ReverseVideo instead. Looks like this was an old bug exposed by the changes to set_cursor_gcs rather than a new bug. Many thanks Thomas - placed a call to set_cursor_gcs in ReverseVideo and the problem is now fixed with xterm-208-4.FC4 , shortly to be released to FC-4 updates / testing. From User-Agent: XML-RPC xterm-208-4.FC4 has been pushed for FC4, which should resolve this issue. If these problems are still present in this version, then please make note of it in this bug report. xterm-208-4.FC4 has been pushed for FC4, which should resolve this issue. If these problems are still present in this version, then please make note of it in this bug report. |