public inbox for insight@sourceware.org
 help / color / mirror / Atom feed
From: Fernando Nasser <fnasser@cygnus.com>
To: Keith Seitz <keiths@cygnus.com>
Cc: Insight Maling List <insight@sources.redhat.com>
Subject: Re: [PATCH] Eliminate Tix from download dialog
Date: Mon, 16 Jul 2001 06:56:00 -0000	[thread overview]
Message-ID: <3B5168B0.2BE7D8F7@cygnus.com> (raw)
In-Reply-To: <Pine.GSO.4.33.0107131659050.16311-100000@makita.cygnus.com>

Keith Seitz wrote:
> 
> Hi,
> 
> I've committed a patch which removes our reliance of tix in the download
> dialog.
> 
> Unfortunately, there are some iwidgets bugs which prevent this from
> working just right. It is mostly fixed in a newer version of iwidgets, but
> for now, here's the patch for iwidgets to make this work properly. (This
> is only a cosmetic issue.)
> 
> Index: itcl/iwidgets3.0.0/generic/feedback.itk
> ===================================================================
> RCS file: /cvs/src/src/itcl/iwidgets3.0.0/generic/feedback.itk,v
> retrieving revision 1.1.1.1
> diff -u -p -r1.1.1.1 feedback.itk
> --- feedback.itk        2000/02/07 00:19:46     1.1.1.1
> +++ feedback.itk        2001/07/14 00:00:51
> @@ -172,6 +172,7 @@ configbody iwidgets::Feedback::steps {
>  #
> -----------------------------------------------------------------------------
>  body iwidgets::Feedback::_display {} {
>      set troughwidth [winfo width $itk_component(trough)]
> +    incr troughwidth [expr {-2 * [cget -borderwidth]}]
>      set _barwidth [expr $troughwidth.0/$itk_option(-steps)]
>      set fraction [expr int((1.0*$_stepval)/$itk_option(-steps)*100.0)]
> 
> @@ -203,5 +204,8 @@ body iwidgets::Feedback::step {{inc 1}}
>      }
> 
>      incr _stepval $inc
> +    if {$_stepval > $itk_option(-steps)} {
> +      set _stepval $itk_option(-steps)
> +    }
>      _display
>  }
> 
> We'll see about getting our itcl fixed and getting our change back into
> the itcl repository. (Or we'll write our own feedback widget and put it in
> libgui.)
> 
> Keith
> 

I am sure Michael will appreciate that you send the patch and will incorporate it.

It is nice to see the Tix -> iwidgets+BLT finally moving forward.  Thank you very much!


-- 
Fernando Nasser
Red Hat - Toronto                       E-Mail:  fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario   M4P 2C9

      reply	other threads:[~2001-07-16  6:56 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-07-13 17:02 Keith Seitz
2001-07-16  6:56 ` Fernando Nasser [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3B5168B0.2BE7D8F7@cygnus.com \
    --to=fnasser@cygnus.com \
    --cc=insight@sources.redhat.com \
    --cc=keiths@cygnus.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).