[PATCH] Re: "Installation finished"

[Igor Peshansky <pechtcha at cs dot nyu dot edu>]

On Thu, 22 Mar 2007, Brian Dessent wrote:

> Corinna Vinschen wrote:
>
> > Question to the setup.exe maintainers:
> >
> > Do we really need the "Installation finished." message box at the end?
> > I find it increasingly annoying that I have to confirm the end of the
> > installation on a centered button when all the rest of the "OK" buttons
> > were at the bottom right of the prevous dialogs.
> >
> > Wouldn't it suffice to rename the "OK" button in the last dialog to
> > "Finish"?
>
> You're right, I hate that too and it's been on the todo list to remove
> this for a long time.

And here is a patch.  It does a bit more than remove the final mbox -- it
also moves the status information into the "Create Icons" dialog.  It
should handle the resizing of that dialog correctly, but please test it
anyway.  ChangeLog is below.  Comments welcome.
	Igor
==============================================================================
ChangeLog:
2007-03-28  Igor Peshansky  <pechtcha@cs.nyu.edu>

	* LogFile.cc (LogFile::exit): Write exit_msg directly to log, without
	the message box.
	* resource.h (IDC_DESKTOP_SEPARATOR,IDC_STATUS_HEADER,IDC_STATUS): New
	control ids.
	* res.rc (IDD_DESKTOP): Change the caption.  Add status information.
	* desktop.h (DesktopSetupPage::DesktopSetupPage): Define constructor
	in the implementation.
	* desktop.cc (DesktopControlsInfo): Position specifiers for status
	information items.
	(DesktopSetupPage::DesktopSetupPage): Register position specifiers.
	(set_status): New static method.
	(load_dialog): Set status information, if available.
	(DesktopSetupPage::OnInit): Set status header font.

-- 
				http://cs.nyu.edu/~pechtcha/
      |\      _,,,---,,_	    pechtcha@cs.nyu.edu | igor@watson.ibm.com
ZZZzz /,`.-'`'    -.  ;-;;,_		Igor Peshansky, Ph.D. (name changed!)
     |,4-  ) )-,_. ,\ (  `'-'		old name: Igor Pechtchanski
    '---''(_/--'  `-'\_) fL	a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

Freedom is just another word for "nothing left to lose"...  -- Janis Joplin

Attachment: setup-remove-final-mbox.patch
Description: Text document