I'll answer because I coded it :P
Le 05-11-27 à 16:21, Youness Alaoui a écrit :

hey,
look at that :
checkbutton $w.top.remember -text [trans remembersetting] - variable [::config::getVar remember]

the config variable to "remember my setting" on whether to close a tab or all tabs is "remember"... if you look at "remember" as a config key (if you look at config.xml file for example) you won't understand what it is... I suggest we change this before 0.95 is released, how about "close_container_action"...
I agree, I never tought of that when I coded it.

About what the guy said...when I coded it, I added -default to the button "No" and I tought that TK will automatically handle "Enter" for "no"...seems not!
only problem is that it seems that instead of having a variable that would have "closetab"/"closeall"/"ask" values, we have a boolean variable that defines whether we close a tab or all and another one to whether we remember the setting or not...
need to be changed.. post-0.95

KaKaRoTo

------- Forwarded message -------
From: "SourceForge.net" <[EMAIL PROTECTED]>
To: [EMAIL PROTECTED]
Subject: [ amsn-Patches-1177260 ] Close tabs confirmation
Date: Sun, 27 Nov 2005 16:13:38 -0500

Patches item #1177260, was opened at 2005-04-05 13:56
Message generated for change (Comment added) made by kakaroto
You can respond by visiting:
https://sourceforge.net/tracker/? func=detail&atid=472657&aid=1177260&group_id=54091

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Harry de Boer (harrydb)
Assigned to: Alaoui Youness (kakaroto)
Summary: Close tabs confirmation

Initial Comment:
When closing a container, none of the buttons in the
confirmation dialog is highlighted which make it quite
impossible to close a container using the keyboard.

I decided not to be lazy for a time and write this
little piece of code myself (ContainerClose procedure
from chatwindow.tcl).
The version I wrote uses the ::amsn::MessageBox
procedure to show a message box. The code is in fact a
little smaller and more pretty than the code currently
in cvs.

I attached the modified ContainerClose procedure from
chatwindow.tcl, I hope it is useful.

----------------------------------------------------------------------

Comment By: Alaoui Youness (kakaroto)
Date: 2005-11-27 16:13

Message:
Logged In: YES
user_id=686750

Hi,
this doesn't allow us to have the "remember my choice"
checkbox.. and the window that appears now is nicer than
before... but yes, there is no highlighting when you tab
through the buttons...

KaKaRoTo

----------------------------------------------------------------------

Comment By: Harry de Boer (harrydb)
Date: 2005-04-07 09:50

Message:
Logged In: YES
user_id=845823

Hm, this might belong in the "patches" section... (and my
other post too..)

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/? func=detail&atid=472657&aid=1177260&group_id=54091



--
KaKaRoTo


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the web. DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel



-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_idv37&alloc_id865&op=click
_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel

Reply via email to