Package: tpp
Version: 1.3-1
Severity: wishlist
Tags: patch

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

I discovered tpp today and I think it's really a charming piece of
software :-)

What irritated me is that the boxes created with --begin(shell)ouput are not
closed, i.e. they only have a full left margin, a half top and bottom
border and no right border.

Therefor I tried to add the "missing" parts; the attached patch seems
to work quite well.

I know this is a matter of taste but I'd like to ask you and Andreas
to inlucde this functionality.

Thanks,
gregor

- -- System Information:
Debian Release: 4.0
  APT prefers unstable
  APT policy: (990, 'unstable'), (500, 'experimental'), (500, 'testing'), (500, 
'stable')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.18.200612171741
Locale: LANG=C, [EMAIL PROTECTED] (charmap=ISO-8859-15)

Versions of packages tpp depends on:
ii  libncurses-ruby1.8            1.1-1      ruby Extension for the ncurses C l
ii  ruby                          1.8.2-1    An interpreter of object-oriented 

tpp recommends no packages.

- -- no debconf information

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFFrSZgOzKYnQDzz+QRAksaAJ937Ry1XlVrCJWVTmCYhyICHHHFbwCbBdSk
pzpN3L4oF5WQrPq0t8mnb58=
=dW4+
-----END PGP SIGNATURE-----
--- /usr/bin/tpp        2005-06-23 20:46:29.000000000 +0200
+++ /tmp/tpp    2007-01-16 19:21:26.000000000 +0100
@@ -672,2 +672,6 @@
       @screen.addstr(l)
+      if @output or @shelloutput then
+        @screen.move(@cur_line,@termwidth - @indent - 2)
+        @screen.addstr(" |")
+      end
       @cur_line += 1
@@ -687,5 +691,8 @@
       end
-      x = (@termwidth - l.length - 2)
+      x = (@termwidth - l.length - 5)
       @screen.move(@cur_line,x)
       @screen.addstr(l)
+      if @output or @shelloutput then
+        @screen.addstr(" |")
+      end
       @cur_line += 1
@@ -733,3 +740,4 @@
     @screen.addstr(".")
-    ((@termwidth - @indent)/2).times { @screen.addstr("-") }
+    (@termwidth - @indent*2 - 2).times { @screen.addstr("-") }
+    @screen.addstr(".")
     @output = true
@@ -741,3 +749,4 @@
     @screen.addstr(".")
-    ((@termwidth - @indent)/2).times { @screen.addstr("-") }
+    (@termwidth - @indent*2 - 2).times { @screen.addstr("-") }
+    @screen.addstr(".")
     @shelloutput = true
@@ -750,3 +759,4 @@
       @screen.addstr("`")
-      ((@termwidth - @indent)/2).times { @screen.addstr("-") }
+      (@termwidth - @indent*2 - 2).times { @screen.addstr("-") }
+      @screen.addstr("'")
       @output = false
@@ -787,3 +797,4 @@
       @screen.addstr("`")
-      ((@termwidth - @indent)/2).times { @screen.addstr("-") }
+      (@termwidth - @indent*2 - 2).times { @screen.addstr("-") }
+      @screen.addstr("'")
       @shelloutput = false
@@ -950,3 +961,3 @@
       @screen.move(@cur_line,@indent)
-      if @output or @shelloutput then
+      if (@output or @shelloutput) and ! @slideoutput then
         @screen.addstr("| ")
@@ -960,2 +971,6 @@
       end
+      if (@output or @shelloutput) and ! @slideoutput then
+        @screen.move(@cur_line,@termwidth - @indent - 2)
+        @screen.addstr(" |")
+      end
       @cur_line += 1

Reply via email to