[Debian-med-packaging] Bug#815176: staden: Gap4 contig editor does not show sequences

[Andreas Tille]

Hi James,

if I understand you correctly you are suggesting a further change.  Could
you please confirm that this patch

    https://anonscm.debian.org/viewvc/debian-med/trunk/packages/staden/trunk/debian/patches/cope_with_modern_windowmanager.patch?view=markup

is implementing your suggestion?

Sorry for my Tcl/Tk ignorance

    Andreas.

On Mon, Feb 22, 2016 at 11:14:09AM +0000, James Bonfield wrote:
> On Mon, Feb 22, 2016 at 12:03:40PM +0100, Andreas Tille wrote:
> > I've applied the suggested patch (which can be found in the packaging
> > repository[1]).
> 
> I did manage to experiment with a few window managers, freezing my
> desktop in the process :-).
> 
> All worked in as far as the editor was usable, but some struggled with
> manual resizing - either not allowing it (twm) or fighting against the
> application (sawfish).  I also tweaked the resize callback to no
> longer have the after-idle bit.  This generally worked better, at the
> expense of not permitting resize on ancient window managers (eg twm).
> I really doubt that's an issue though.
> 
> *** contig_editor.tcl~	 Tue Nov 17 11:38:00 2015
> --- contig_editor.tcl	 Mon Feb 22 10:06:49 2016
> ***************
> *** 108,114 ****
>     xtoplevel $w
>     fix_maxsize $w
>       }
> !     wm resizable $w 1 0
>   
>       catch {set $w.Grab [grab current]}
>       frame $w.grab
> --- 108,114 ----
>     xtoplevel $w
>     fix_maxsize $w
>       }
> !     #wm resizable $w 1 0
>   
>       catch {set $w.Grab [grab current]}
>       frame $w.grab
> ***************
> *** 450,456 ****
>       #
>       bind $w <Any-Configure> {
>       if {[winfo toplevel %W] == "%W"} {
> !         after 1000 {if {[winfo exists %W]} {wm geometry %W {}}}
>      	  }
>       }
>       SetDefaultTags CONTIG_EDITOR.TAGS $editor
> --- 450,456 ----
>       #
>       bind $w <Any-Configure> {
>       if {[winfo toplevel %W] == "%W"} {
> !         wm geometry %W {}
>      	  }
>       }
>       SetDefaultTags CONTIG_EDITOR.TAGS $editor
> 
> > James, if you are interested there are some more patches[2] in the
> > packaging reporitory.  Some of them might be of interest - at least
> > the spelling.patch with some spelling fixes.
> 
> Thanks, I'll take a look.
> 
> James
> 
> -- 
> James Bonfield (jkb at sanger.ac.uk) | Hora aderat briligi. Nunc et Slythia Tova
>                                   | Plurima gyrabant gymbolitare vabo;
>   A Staden Package developer:     | Et Borogovorum mimzebant undique formae,
> https://sf.net/projects/staden/   | Momiferique omnes exgrabure Rathi. 
> 
> 
> -- 
>  The Wellcome Trust Sanger Institute is operated by Genome Research 
>  Limited, a charity registered in England with number 1021457 and a 
>  company registered in England with number 2742969, whose registered 
>  office is 215 Euston Road, London, NW1 2BE. 
> 

-- 
http://fam-tille.de