[Aptitude-svn-commit] r3502 - in branches/aptitude-0.3/aptitude: . src/vscreen/config

Daniel Burrows dburrows@costa.debian.org
Tue, 28 Jun 2005 14:06:22 +0000


Author: dburrows
Date: Tue Jun 28 14:06:20 2005
New Revision: 3502

Modified:
   branches/aptitude-0.3/aptitude/ChangeLog
   branches/aptitude-0.3/aptitude/src/vscreen/config/colors.cc
Log:
Don't make the cursor invisible, at the cost of losing black-on-black text.

Modified: branches/aptitude-0.3/aptitude/ChangeLog
==============================================================================
--- branches/aptitude-0.3/aptitude/ChangeLog	(original)
+++ branches/aptitude-0.3/aptitude/ChangeLog	Tue Jun 28 14:06:20 2005
@@ -1,5 +1,10 @@
 2005-06-28  Daniel Burrows  <dburrows@debian.org>
 
+	* src/vscreen/config/colors.cc:
+
+	  Don't make the cursor invisible, at the cost of losing
+	  black-on-black text.
+
 	* src/vscreen/vs_editline.cc, src/vscreen/vs_editline.h:
 
 	  Fix the line editor widget to support wide characters properly

Modified: branches/aptitude-0.3/aptitude/src/vscreen/config/colors.cc
==============================================================================
--- branches/aptitude-0.3/aptitude/src/vscreen/config/colors.cc	(original)
+++ branches/aptitude-0.3/aptitude/src/vscreen/config/colors.cc	Tue Jun 28 14:06:20 2005
@@ -36,7 +36,11 @@
     for(short bg=0; bg<COLORS; ++bg)
       {
 	if(fg==0 && bg==0)
-	  assume_default_colors(0, 0);
+	  // do nothing; on some terminals, doing this causes the
+	  // cursor to become INVISIBLE, and black-on-black text is a
+	  // bad idea anyway..
+	  ;
+	  /*assume_default_colors(0, 0);*/
 	else
 	  init_pair(fg*COLORS+bg, fg, bg);
       }