src/motif/frame.cpp

   1 /////////////////////////////////////////////////////////////////////////////
   2 // Name:        motif/frame.cpp
   3 // Purpose:     wxFrame
   4 // Author:      Julian Smart
   5 // Modified by:
   6 // Created:     17/09/98
   7 // RCS-ID:      $Id$
   8 // Copyright:   (c) Julian Smart
   9 // Licence:     wxWindows licence
  10 /////////////////////////////////////////////////////////////////////////////
  11
  12 // ============================================================================
  13 // declarations
  14 // ============================================================================
  15
  16 // ----------------------------------------------------------------------------
  17 // headers
  18 // ----------------------------------------------------------------------------
  19
  20 #ifdef __GNUG__
  21     #pragma implementation "frame.h"
  22 #endif
  23
  24
  25 #ifdef __VMS
  26 #define XtDisplay XTDISPLAY
  27 #define XtWindow XTWINDOW
  28 #define XtScreen XTSCREEN
  29 #endif
  30
  31 #include "wx/frame.h"
  32 #include "wx/statusbr.h"
  33 #include "wx/toolbar.h"
  34 #include "wx/menu.h"
  35 #include "wx/settings.h"
  36 #include "wx/utils.h"
  37 #include "wx/log.h"
  38 #include "wx/app.h"
  39 #include "wx/icon.h"
  40
  41 #ifdef __VMS__
  42     #pragma message disable nosimpint
  43 #endif
  44
  45 #if defined(__ultrix) || defined(__sgi)
  46     #include <Xm/Frame.h>
  47 #endif
  48
  49 #include <Xm/Xm.h>
  50 #include <X11/Shell.h>
  51 #include <X11/Core.h>
  52 #if XmVersion >= 1002
  53     #include <Xm/XmAll.h>
  54 #else
  55     #include <Xm/Frame.h>
  56 #endif
  57 #include <Xm/MwmUtil.h>
  58 #include <Xm/BulletinB.h>
  59 #include <Xm/Form.h>
  60 #include <Xm/MainW.h>
  61 #include <Xm/RowColumn.h>
  62 #include <Xm/Label.h>
  63 #include <Xm/AtomMgr.h>
  64 #include <Xm/LabelG.h>
  65 #include <Xm/Frame.h>
  66 #if   XmVersion > 1000
  67     #include <Xm/Protocols.h>
  68 #endif
  69
  70 #ifdef __VMS__
  71     #pragma message enable nosimpint
  72 #endif
  73
  74 #include "wx/motif/private.h"
  75 #include "wx/unix/utilsx11.h"
  76
  77 // ----------------------------------------------------------------------------
  78 // private functions
  79 // ----------------------------------------------------------------------------
  80
  81 static void wxFrameMapProc(Widget frameShell, XtPointer clientData,
  82                            XCrossingEvent* event);
  83
  84 // ----------------------------------------------------------------------------
  85 // globals
  86 // ----------------------------------------------------------------------------
  87
  88 extern wxList wxModelessWindows;
  89 extern wxList wxPendingDelete;
  90
  91 // TODO: this should be tidied so that any frame can be the
  92 // top frame
  93 // static bool wxTopLevelUsed = FALSE;
  94
  95 // ----------------------------------------------------------------------------
  96 // wxWin macros
  97 // ----------------------------------------------------------------------------
  98
  99 BEGIN_EVENT_TABLE(wxFrame, wxFrameBase)
 100     EVT_ACTIVATE(wxFrame::OnActivate)
 101     EVT_SYS_COLOUR_CHANGED(wxFrame::OnSysColourChanged)
 102 END_EVENT_TABLE()
 103
 104 IMPLEMENT_DYNAMIC_CLASS(wxFrame, wxWindow)
 105
 106 // ============================================================================
 107 // implementation
 108 // ============================================================================
 109
 110 // ----------------------------------------------------------------------------
 111 // frame construction
 112 // ----------------------------------------------------------------------------
 113
 114 void wxFrame::Init()
 115 {
 116     m_iconized = FALSE;
 117
 118     //// Motif-specific
 119     m_frameShell = (WXWidget) NULL;
 120     m_mainWidget = (WXWidget) NULL;;
 121     m_workArea = (WXWidget) NULL;;
 122     m_clientArea = (WXWidget) NULL;;
 123     // m_visibleStatus = TRUE;
 124 }
 125
 126 bool wxFrame::Create(wxWindow *parent,
 127                      wxWindowID id,
 128                      const wxString& title,
 129                      const wxPoint& pos,
 130                      const wxSize& size,
 131                      long style,
 132                      const wxString& name)
 133 {
 134     if( !wxTopLevelWindow::Create( parent, id, title, pos, size, style,
 135                                    name ) )
 136         return FALSE;
 137
 138     m_backgroundColour =
 139         wxSystemSettings::GetColour(wxSYS_COLOUR_APPWORKSPACE);
 140     m_foregroundColour = *wxBLACK;
 141     m_font = wxSystemSettings::GetFont(wxSYS_DEFAULT_GUI_FONT);
 142
 143     int x = pos.x, y = pos.y;
 144     int width = size.x, height = size.y;
 145
 146     // Set reasonable values for position and size if defaults have been
 147     // requested
 148     //
 149     // MB TODO: something better than these arbitrary values ?
 150     // VZ       should use X resources for this...
 151     if ( width == -1 )
 152         width = 400;
 153     if ( height == -1 )
 154         height = 400;
 155
 156     int displayW, displayH;
 157     wxDisplaySize( &displayW, &displayH );
 158
 159     if ( x == -1 )
 160     {
 161         x = (displayW - width) / 2;
 162         if (x < 10) x = 10;
 163     }
 164     if ( y == -1 )
 165     {
 166         y = (displayH - height) / 2;
 167         if (y < 10) y = 10;
 168     }
 169
 170     SetTitle( title );
 171
 172     wxLogTrace(wxTRACE_Messages,
 173                "Created frame (0x%p) with work area 0x%p and client "
 174                "area 0x%p", m_mainWidget, m_workArea, m_clientArea);
 175
 176     XtAddEventHandler((Widget) m_clientArea, ExposureMask,FALSE,
 177         wxUniversalRepaintProc, (XtPointer) this);
 178
 179     if (x > -1)
 180         XtVaSetValues((Widget) m_frameShell, XmNx, x, NULL);
 181     if (y > -1)
 182         XtVaSetValues((Widget) m_frameShell, XmNy, y, NULL);
 183     if (width > -1)
 184         XtVaSetValues((Widget) m_frameShell, XmNwidth, width, NULL);
 185     if (height > -1)
 186         XtVaSetValues((Widget) m_frameShell, XmNheight, height, NULL);
 187
 188     ChangeFont(FALSE);
 189
 190     ChangeBackgroundColour();
 191
 192     PreResize();
 193
 194     wxSizeEvent sizeEvent(wxSize(width, height), GetId());
 195     sizeEvent.SetEventObject(this);
 196
 197     GetEventHandler()->ProcessEvent(sizeEvent);
 198
 199     return TRUE;
 200 }
 201
 202 bool wxFrame::DoCreate( wxWindow* parent, wxWindowID id,
 203                         const wxString& title,
 204                         const wxPoint& pos,
 205                         const wxSize& size,
 206                         long style,
 207                         const wxString& name )
 208 {
 209     static bool wxTopLevelUsed = FALSE; /* this is global */
 210     WXWidget frameShell;
 211
 212     if (wxTopLevelUsed)
 213     {
 214         // Change suggested by Matthew Flatt
 215         frameShell = (WXWidget)XtAppCreateShell( name,
 216                                                  wxTheApp->GetClassName(),
 217                                                  topLevelShellWidgetClass,
 218                                                  (Display*) wxGetDisplay(),
 219                                                  NULL, 0 );
 220     }
 221     else
 222     {
 223         frameShell = wxTheApp->GetTopLevelWidget();
 224         wxTopLevelUsed = TRUE;
 225     }
 226
 227     XtVaSetValues((Widget) frameShell,
 228         // Allows menu to resize
 229         XmNallowShellResize, True,
 230         XmNdeleteResponse, XmDO_NOTHING,
 231         XmNmappedWhenManaged, False,
 232         XmNiconic, (style & wxICONIZE) ? TRUE : FALSE,
 233         NULL);
 234
 235     m_frameShell = frameShell;
 236
 237     m_mainWidget = (WXWidget) XtVaCreateManagedWidget("main_window",
 238         xmMainWindowWidgetClass, (Widget) frameShell,
 239         XmNresizePolicy, XmRESIZE_NONE,
 240         NULL);
 241
 242     m_workArea = (WXWidget) XtVaCreateWidget("form",
 243         xmFormWidgetClass, (Widget) m_mainWidget,
 244         XmNresizePolicy, XmRESIZE_NONE,
 245         NULL);
 246
 247     m_clientArea = (WXWidget) XtVaCreateWidget("client",
 248         xmBulletinBoardWidgetClass, (Widget) m_workArea,
 249         XmNmarginWidth, 0,
 250         XmNmarginHeight, 0,
 251         XmNrightAttachment, XmATTACH_FORM,
 252         XmNleftAttachment, XmATTACH_FORM,
 253         XmNtopAttachment, XmATTACH_FORM,
 254         XmNbottomAttachment, XmATTACH_FORM,
 255         NULL);
 256
 257     XtVaSetValues((Widget) m_mainWidget,
 258         XmNworkWindow, (Widget) m_workArea,
 259         NULL);
 260
 261     XtManageChild((Widget) m_clientArea);
 262     XtManageChild((Widget) m_workArea);
 263
 264     XtTranslations ptr = XtParseTranslationTable( "<Configure>: resize()" );
 265     XtOverrideTranslations( (Widget) m_workArea, ptr );
 266     XtFree( (char *)ptr );
 267
 268     /* Part of show-&-hide fix */
 269     XtAddEventHandler( (Widget)frameShell, StructureNotifyMask,
 270                        False, (XtEventHandler)wxFrameMapProc,
 271                        (XtPointer)this );
 272
 273     XtRealizeWidget((Widget) frameShell);
 274
 275     wxAddWindowToTable( (Widget)m_workArea, this);
 276     wxAddWindowToTable( (Widget)m_clientArea, this);
 277
 278     wxModelessWindows.Append( this );
 279
 280     return TRUE;
 281 }
 282
 283 wxFrame::~wxFrame()
 284 {
 285     m_isBeingDeleted = TRUE;
 286
 287     if (m_clientArea)
 288     {
 289       XtRemoveEventHandler((Widget) m_clientArea, ExposureMask, FALSE,
 290           wxUniversalRepaintProc, (XtPointer) this);
 291     }
 292
 293     if (GetMainWidget())
 294         Show(FALSE);
 295
 296     if (m_frameMenuBar)
 297     {
 298         m_frameMenuBar->DestroyMenuBar();
 299
 300         // Hack to stop core dump on Ultrix, OSF, for some strange reason.
 301 #if MOTIF_MENUBAR_DELETE_FIX
 302         GetMenuBar()->SetMainWidget((WXWidget) NULL);
 303 #endif
 304         delete m_frameMenuBar;
 305         m_frameMenuBar = NULL;
 306     }
 307
 308     if (m_frameStatusBar)
 309     {
 310         delete m_frameStatusBar;
 311         m_frameStatusBar = NULL;
 312     }
 313 }
 314
 315 void wxFrame::DoDestroy()
 316 {
 317     Widget frameShell = (Widget)GetShellWidget();
 318
 319     XtRemoveEventHandler( frameShell, StructureNotifyMask,
 320                           False, (XtEventHandler)wxFrameMapProc,
 321                           (XtPointer)this );
 322
 323     if( m_clientArea )
 324     {
 325         wxDeleteWindowFromTable( (Widget)m_clientArea );
 326         XtDestroyWidget( (Widget)m_clientArea );
 327     }
 328
 329     if( m_workArea )
 330     {
 331         XtVaSetValues( (Widget)m_mainWidget,
 332                        XmNworkWindow, (Widget)NULL,
 333                        NULL );
 334
 335         wxDeleteWindowFromTable( (Widget)m_workArea );
 336         XtDestroyWidget( (Widget)m_workArea );
 337     }
 338
 339     if( m_mainWidget )
 340         XtDestroyWidget( (Widget)m_mainWidget );
 341
 342     if( frameShell )
 343         XtDestroyWidget( frameShell );
 344 }
 345
 346 // Get size *available for subwindows* i.e. excluding menu bar, toolbar etc.
 347 void wxFrame::DoGetClientSize(int *x, int *y) const
 348 {
 349     Dimension xx, yy;
 350     XtVaGetValues((Widget) m_workArea, XmNwidth, &xx, XmNheight, &yy, NULL);
 351
 352     if (m_frameStatusBar)
 353     {
 354         int sbw, sbh;
 355         m_frameStatusBar->GetSize(& sbw, & sbh);
 356         yy -= sbh;
 357     }
 358 #if wxUSE_TOOLBAR
 359     if (m_frameToolBar)
 360     {
 361         int tbw, tbh;
 362         m_frameToolBar->GetSize(& tbw, & tbh);
 363         if (m_frameToolBar->GetWindowStyleFlag() & wxTB_VERTICAL)
 364             xx -= tbw;
 365         else
 366             yy -= tbh;
 367     }
 368 #endif // wxUSE_TOOLBAR
 369     *x = xx; *y = yy;
 370 }
 371
 372 // Set the client size (i.e. leave the calculation of borders etc.
 373 // to wxWindows)
 374 void wxFrame::DoSetClientSize(int width, int height)
 375 {
 376     // Calculate how large the new main window should be
 377     // by finding the difference between the client area and the
 378     // main window area, and adding on to the new client area
 379     if (width > -1)
 380         XtVaSetValues((Widget) m_workArea, XmNwidth, width, NULL);
 381
 382     if (height > -1)
 383     {
 384         if (m_frameStatusBar)
 385         {
 386             int sbw, sbh;
 387             m_frameStatusBar->GetSize(& sbw, & sbh);
 388             height += sbh;
 389         }
 390 #if wxUSE_TOOLBAR
 391         if (m_frameToolBar)
 392         {
 393             int tbw, tbh;
 394             m_frameToolBar->GetSize(& tbw, & tbh);
 395             if (m_frameToolBar->GetWindowStyleFlag() & wxTB_VERTICAL)
 396                 width += tbw;
 397             else
 398                 height += tbh;
 399         }
 400 #endif // wxUSE_TOOLBAR
 401
 402         XtVaSetValues((Widget) m_workArea, XmNheight, height, NULL);
 403     }
 404     PreResize();
 405
 406     wxSizeEvent sizeEvent(wxSize(width, height), GetId());
 407     sizeEvent.SetEventObject(this);
 408
 409     GetEventHandler()->ProcessEvent(sizeEvent);
 410
 411 }
 412
 413 void wxFrame::DoGetSize(int *width, int *height) const
 414 {
 415     Dimension xx, yy;
 416     XtVaGetValues((Widget) m_frameShell, XmNwidth, &xx, XmNheight, &yy, NULL);
 417     *width = xx; *height = yy;
 418 }
 419
 420 void wxFrame::DoGetPosition(int *x, int *y) const
 421 {
 422     Window parent_window = XtWindow((Widget) m_frameShell),
 423         next_parent   = XtWindow((Widget) m_frameShell),
 424         root          = RootWindowOfScreen(XtScreen((Widget) m_frameShell));
 425
 426     // search for the parent that is child of ROOT, because the WM may
 427     // reparent twice and notify only the next parent (like FVWM)
 428     while (next_parent != root) {
 429         Window *theChildren; unsigned int n;
 430         parent_window = next_parent;
 431         XQueryTree(XtDisplay((Widget) m_frameShell), parent_window, &root,
 432             &next_parent, &theChildren, &n);
 433         XFree(theChildren); // not needed
 434     }
 435     int xx, yy; unsigned int dummy;
 436     XGetGeometry(XtDisplay((Widget) m_frameShell), parent_window, &root,
 437         &xx, &yy, &dummy, &dummy, &dummy, &dummy);
 438     if (x) *x = xx;
 439     if (y) *y = yy;
 440 }
 441
 442 void wxFrame::DoSetSize(int x, int y, int width, int height, int WXUNUSED(sizeFlags))
 443 {
 444     if (x > -1)
 445         XtVaSetValues((Widget) m_frameShell, XmNx, x, NULL);
 446     if (y > -1)
 447         XtVaSetValues((Widget) m_frameShell, XmNy, y, NULL);
 448     if (width > -1)
 449         XtVaSetValues((Widget) m_mainWidget, XmNwidth, width, NULL);
 450     if (height > -1)
 451         XtVaSetValues((Widget) m_mainWidget, XmNheight, height, NULL);
 452
 453     if (!(height == -1 && width == -1))
 454     {
 455         PreResize();
 456     }
 457 }
 458
 459 bool wxFrame::Show( bool show )
 460 {
 461     if( !wxTopLevelWindowMotif::Show( show ) )
 462         return FALSE;
 463
 464     m_isShown = show;
 465
 466     Widget shell = (Widget)GetShellWidget();
 467     if (!shell)
 468         return wxWindow::Show(show);
 469
 470     SetVisibleStatus(show);
 471     if (show)
 472     {
 473         XtMapWidget (shell);
 474         XRaiseWindow (XtDisplay(shell), XtWindow(shell));
 475     }
 476     else
 477     {
 478         XtUnmapWidget(shell);
 479     }
 480
 481     return TRUE;
 482 }
 483
 484 void wxFrame::SetTitle(const wxString& title)
 485 {
 486     wxString oldTitle = GetTitle();
 487     if( title == oldTitle )
 488         return;
 489
 490     wxTopLevelWindow::SetTitle( title );
 491
 492     if( !title.empty() )
 493         XtVaSetValues( (Widget)m_frameShell,
 494                        XmNtitle, title.c_str(),
 495                        XmNiconName, title.c_str(),
 496                        NULL );
 497 }
 498
 499 void wxFrame::DoSetIcon(const wxIcon& icon)
 500 {
 501     if (!m_frameShell)
 502         return;
 503
 504     if (!icon.Ok() || !icon.GetPixmap())
 505         return;
 506
 507     XtVaSetValues((Widget) m_frameShell, XtNiconPixmap, icon.GetPixmap(), NULL);
 508 }
 509
 510 void wxFrame::SetIcon(const wxIcon& icon)
 511 {
 512     SetIcons( wxIconBundle( icon ) );
 513 }
 514
 515 void wxFrame::SetIcons(const wxIconBundle& icons)
 516 {
 517     wxFrameBase::SetIcons( icons );
 518
 519     if (!m_frameShell)
 520         return;
 521
 522     DoSetIcon( m_icons.GetIcon( -1 ) );
 523     wxSetIconsX11(GetXDisplay(),
 524                   (WXWindow) XtWindow( (Widget) m_frameShell ), icons);
 525 }
 526
 527 void wxFrame::PositionStatusBar()
 528 {
 529     if (!m_frameStatusBar)
 530         return;
 531
 532     int w, h;
 533     GetClientSize(&w, &h);
 534     int sw, sh;
 535     m_frameStatusBar->GetSize(&sw, &sh);
 536
 537     // Since we wish the status bar to be directly under the client area,
 538     // we use the adjusted sizes without using wxSIZE_NO_ADJUSTMENTS.
 539     m_frameStatusBar->SetSize(0, h, w, sh);
 540 }
 541
 542 WXWidget wxFrame::GetMenuBarWidget() const
 543 {
 544     if (GetMenuBar())
 545         return GetMenuBar()->GetMainWidget();
 546     else
 547         return (WXWidget) NULL;
 548 }
 549
 550 void wxFrame::SetMenuBar(wxMenuBar *menuBar)
 551 {
 552     if (!menuBar)
 553     {
 554         m_frameMenuBar = NULL;
 555         return;
 556     }
 557
 558     // Currently can't set it twice
 559     //    wxASSERT_MSG( (m_frameMenuBar == (wxMenuBar*) NULL), "Cannot set the menubar more than once");
 560
 561     if (m_frameMenuBar)
 562     {
 563         m_frameMenuBar->DestroyMenuBar();
 564         delete m_frameMenuBar;
 565     }
 566
 567     m_frameMenuBar = menuBar;
 568     m_frameMenuBar->CreateMenuBar(this);
 569 }
 570
 571 // Responds to colour changes, and passes event on to children.
 572 void wxFrame::OnSysColourChanged(wxSysColourChangedEvent& event)
 573 {
 574     SetBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_APPWORKSPACE));
 575     Refresh();
 576
 577     if ( m_frameStatusBar )
 578     {
 579         wxSysColourChangedEvent event2;
 580         event2.SetEventObject( m_frameStatusBar );
 581         m_frameStatusBar->ProcessEvent(event2);
 582     }
 583
 584     // Propagate the event to the non-top-level children
 585     wxWindow::OnSysColourChanged(event);
 586 }
 587
 588 // Default activation behaviour - set the focus for the first child
 589 // subwindow found.
 590 void wxFrame::OnActivate(wxActivateEvent& event)
 591 {
 592     if (!event.GetActive())
 593         return;
 594
 595     for(wxWindowList::Node *node = GetChildren().GetFirst(); node;
 596         node = node->GetNext())
 597     {
 598         // Find a child that's a subwindow, but not a dialog box.
 599         wxWindow *child = node->GetData();
 600         if (!child->IsTopLevel())
 601         {
 602             child->SetFocus();
 603             return;
 604         }
 605     }
 606 }
 607
 608 void wxFrame::SendSizeEvent()
 609 {
 610     wxSizeEvent event(GetSize(), GetId());
 611     event.SetEventObject(this);
 612     GetEventHandler()->AddPendingEvent(event);
 613 }
 614
 615 #if wxUSE_TOOLBAR
 616
 617 wxToolBar* wxFrame::CreateToolBar(long style,
 618                                   wxWindowID id,
 619                                   const wxString& name)
 620 {
 621     if ( wxFrameBase::CreateToolBar(style, id, name) )
 622     {
 623         PositionToolBar();
 624     }
 625
 626     return m_frameToolBar;
 627 }
 628
 629 void wxFrame::SetToolBar(wxToolBar *toolbar)
 630 {
 631     wxFrameBase::SetToolBar(toolbar);
 632     SendSizeEvent();
 633 }
 634
 635 void wxFrame::PositionToolBar()
 636 {
 637     wxToolBar* tb = GetToolBar();
 638     if (tb)
 639     {
 640         int cw, ch;
 641         GetClientSize(& cw, &ch);
 642
 643         int tw, th;
 644         tb->GetSize(& tw, & th);
 645
 646         if (tb->GetWindowStyleFlag() & wxTB_VERTICAL)
 647         {
 648             // Use the 'real' position. wxSIZE_NO_ADJUSTMENTS
 649             // means, pretend we don't have toolbar/status bar, so we
 650             // have the original client size.
 651             th = ch + th;
 652         }
 653         else
 654         {
 655             // Use the 'real' position
 656             tw = cw;
 657         }
 658
 659         tb->SetSize(0, 0, tw, th, wxSIZE_NO_ADJUSTMENTS);
 660     }
 661 }
 662 #endif // wxUSE_TOOLBAR
 663
 664 //// Motif-specific
 665 bool wxFrame::PreResize()
 666 {
 667 #if wxUSE_TOOLBAR
 668     PositionToolBar();
 669 #endif // wxUSE_TOOLBAR
 670
 671 #if wxUSE_STATUSBAR
 672     PositionStatusBar();
 673 #endif // wxUSE_STATUSBAR
 674
 675     return TRUE;
 676 }
 677
 678 WXWidget wxFrame::GetClientWidget() const
 679 {
 680     return m_clientArea;
 681 }
 682
 683 void wxFrame::ChangeFont(bool WXUNUSED(keepOriginalSize))
 684 {
 685     // TODO
 686 }
 687
 688 void wxFrame::ChangeBackgroundColour()
 689 {
 690     if (GetClientWidget())
 691         DoChangeBackgroundColour(GetClientWidget(), m_backgroundColour);
 692 }
 693
 694 void wxFrame::ChangeForegroundColour()
 695 {
 696     if (GetClientWidget())
 697         DoChangeForegroundColour(GetClientWidget(), m_foregroundColour);
 698 }
 699
 700 /* MATTEW: Used to insure that hide-&-show within an event cycle works */
 701 static void wxFrameMapProc( Widget frameShell, XtPointer clientData,
 702                             XCrossingEvent* event )
 703 {
 704     wxFrame *tli = (wxFrame*)clientData;
 705
 706     XEvent *e = (XEvent *)event;
 707
 708     if( e->xany.type == MapNotify )
 709     {
 710         // Iconize fix
 711         XtVaSetValues( frameShell, XmNiconic, (Boolean)False, NULL );
 712         if( !tli->GetVisibleStatus() )
 713         {
 714             /* We really wanted this to be hidden! */
 715             XtUnmapWidget( frameShell );
 716         }
 717     }
 718     else if( e->xany.type == UnmapNotify )
 719         // Iconize fix
 720         XtVaSetValues( frameShell, XmNiconic, (Boolean)True, NULL );
 721 }