// update EVERYTHING the first time!!
- // the state is kinda assumed to be normal. is this right? XXX
- _wmstate = NormalState; _iconic = false;
- // no default decors or functions, each has to be enabled
- _decorations = _functions = 0;
+ // we default to NormalState, visible
+ _wmstate = NormalState;
// start unfocused
_focused = false;
// not a transient by default of course
getType();
getMwmHints();
- setupDecorAndFunctions();
-
getState();
getShaped();
updateProtocols();
- getGravity(); // get the attribute gravity
+
+ // got the type, the mwmhints, and the protocols, so we're ready to set up
+ // the decorations/functions
+ setupDecorAndFunctions();
+
+ getGravity(); // get the attribute gravity
updateNormalHints(); // this may override the attribute gravity
- updateWMHints();
+ // also get the initial_state and set _iconic if we aren't "starting"
+ // when we're "starting" that means we should use whatever state was already
+ // on the window over the initial map state, because it was already mapped
+ updateWMHints(openbox->state() != Openbox::State_Starting);
updateTitle();
updateIconTitle();
updateClass();
updateStrut();
+ // this makes sure that these windows appear on all desktops
+ if (_type == Type_Dock || _type == Type_Desktop)
+ _desktop = 0xffffffff;
+
+ // set the desktop hint, to make sure that it always exists, and to reflect
+ // any changes we've made here
+ otk::Property::set(_window, otk::Property::atoms.net_wm_desktop,
+ otk::Property::atoms.cardinal, (unsigned)_desktop);
+
changeState();
}
// these values should not be persisted across a window unmapping/mapping
otk::Property::erase(_window, otk::Property::atoms.net_wm_desktop);
otk::Property::erase(_window, otk::Property::atoms.net_wm_state);
+ } else {
+ // if we're left in an iconic state, the client wont be mapped. this is
+ // bad, since we will no longer be managing the window on restart
+ if (_iconic)
+ XMapWindow(**otk::display, _window);
}
}
// defaults to the current desktop
_desktop = openbox->screen(_screen)->desktop();
- if (!otk::Property::get(_window, otk::Property::atoms.net_wm_desktop,
- otk::Property::atoms.cardinal,
- (long unsigned*)&_desktop)) {
- // make sure the hint exists
- otk::Property::set(_window, otk::Property::atoms.net_wm_desktop,
- otk::Property::atoms.cardinal, (unsigned)_desktop);
- }
+ otk::Property::get(_window, otk::Property::atoms.net_wm_desktop,
+ otk::Property::atoms.cardinal,
+ (long unsigned*)&_desktop);
}
{
// start with everything (cept fullscreen)
_decorations = Decor_Titlebar | Decor_Handle | Decor_Border |
- Decor_Iconify | Decor_Maximize;
+ Decor_AllDesktops | Decor_Iconify | Decor_Maximize;
_functions = Func_Resize | Func_Move | Func_Iconify | Func_Maximize |
Func_Shade;
+ if (_delete_window) {
+ _decorations |= Decor_Close;
+ _functions |= Func_Close;
+ }
switch (_type) {
case Type_Normal:
void Client::getState()
{
_modal = _shaded = _max_horz = _max_vert = _fullscreen = _above = _below =
- _skip_taskbar = _skip_pager = false;
+ _iconic = _skip_taskbar = _skip_pager = false;
unsigned long *state;
unsigned long num = (unsigned) -1;
_modal = true;
else if (state[i] == otk::Property::atoms.net_wm_state_shaded)
_shaded = true;
+ else if (state[i] == otk::Property::atoms.net_wm_state_hidden)
+ _iconic = true;
else if (state[i] == otk::Property::atoms.net_wm_state_skip_taskbar)
_skip_taskbar = true;
else if (state[i] == otk::Property::atoms.net_wm_state_skip_pager)
int num_return = 0;
_focus_notify = false;
- _decorations &= ~Decor_Close;
- _functions &= ~Func_Close;
+ _delete_window = false;
if (XGetWMProtocols(**otk::display, _window, &proto, &num_return)) {
for (int i = 0; i < num_return; ++i) {
if (proto[i] == otk::Property::atoms.wm_delete_window) {
- _decorations |= Decor_Close;
- _functions |= Func_Close;
- if (frame)
- frame->adjustSize(); // update the decorations
+ // this means we can request the window to close
+ _delete_window = true;
} else if (proto[i] == otk::Property::atoms.wm_take_focus)
// if this protocol is requested, then the window will be notified
// by the window manager whenever it receives focus
}
-void Client::updateWMHints()
+void Client::updateWMHints(bool initstate)
{
XWMHints *hints;
if (hints->flags & InputHint)
_can_focus = hints->input;
+ // only do this when initstate is true!
+ if (initstate && (hints->flags & StateHint))
+ _iconic = hints->initial_state == IconicState;
+
if (hints->flags & XUrgencyHint)
_urgent = true;
updateIconTitle();
else if (e.atom == otk::Property::atoms.wm_class)
updateClass();
- else if (e.atom == otk::Property::atoms.wm_protocols)
+ else if (e.atom == otk::Property::atoms.wm_protocols) {
updateProtocols();
+ setupDecorAndFunctions();
+ frame->adjustSize(); // update the decorations
+ }
else if (e.atom == otk::Property::atoms.net_wm_strut)
updateStrut();
}
{
if (state == _wmstate) return; // no change
- _wmstate = state;
- switch (_wmstate) {
+ switch (state) {
case IconicState:
- // XXX: cause it to iconify
+ setDesktop(ICONIC_DESKTOP);
break;
case NormalState:
- // XXX: cause it to uniconify
+ setDesktop(openbox->screen(_screen)->desktop());
break;
}
}
printf("Setting desktop %ld\n", target);
- if (!(target >= 0 || target == (signed)0xffffffff)) return;
+ if (!(target >= 0 || target == (signed)0xffffffff ||
+ target == ICONIC_DESKTOP))
+ return;
_desktop = target;
- otk::Property::set(_window, otk::Property::atoms.net_wm_desktop,
- otk::Property::atoms.cardinal, (unsigned)_desktop);
+ // set the desktop hint, but not if we're iconifying
+ if (_desktop != ICONIC_DESKTOP)
+ otk::Property::set(_window, otk::Property::atoms.net_wm_desktop,
+ otk::Property::atoms.cardinal, (unsigned)_desktop);
// 'move' the window to the new desktop
if (_desktop == openbox->screen(_screen)->desktop() ||
frame->show();
else
frame->hide();
+
+ // Handle Iconic state. Iconic state is maintained by the client being a
+ // member of the ICONIC_DESKTOP, so this is where we make iconifying and
+ // uniconifying happen.
+ bool i = _desktop == ICONIC_DESKTOP;
+ if (i != _iconic) { // has the state changed?
+ _iconic = i;
+ if (_iconic) {
+ _wmstate = IconicState;
+ ignore_unmaps++;
+ // we unmap the client itself so that we can get MapRequest events, and
+ // because the ICCCM tells us to!
+ XUnmapWindow(**otk::display, _window);
+ } else {
+ _wmstate = NormalState;
+ XMapWindow(**otk::display, _window);
+ }
+ changeState();
+ }
+
+ frame->adjustState();
}
#ifdef DEBUG
printf("net_active_window for 0x%lx\n", _window);
#endif
+ if (_iconic)
+ setDesktop(openbox->screen(_screen)->desktop());
if (_shaded)
shade(false);
// XXX: deiconify
otk::Property::atoms.atom, netstate, num);
calcLayer();
+
+ if (frame)
+ frame->adjustState();
}
void Client::applyStartupState()
{
// these are in a carefully crafted order..
-
+
+ if (_iconic) {
+ printf("MAP ICONIC\n");
+ _iconic = false;
+ setDesktop(ICONIC_DESKTOP);
+ }
if (_fullscreen) {
_fullscreen = false;
fullscreen(true);
if (!(_functions & Func_Shade) || // can't
_shaded == shade) return; // already done
- _wmstate = shade ? IconicState : NormalState;
+ // when we're iconic, don't change the wmstate
+ if (!_iconic)
+ _wmstate = shade ? IconicState : NormalState;
_shaded = shade;
changeState();
frame->adjustSize();
}
-bool Client::focus() const
+bool Client::focus()
{
// won't try focus if the client doesn't want it, or if the window isn't
// visible on the screen
if (_focused) return true;
// do a check to see if the window has already been unmapped or destroyed
+ // do this intelligently while watching out for unmaps we've generated
+ // (ignore_unmaps > 0)
XEvent ev;
- if (XCheckTypedWindowEvent(**otk::display, _window, UnmapNotify, &ev) ||
- XCheckTypedWindowEvent(**otk::display, _window, DestroyNotify, &ev)) {
+ if (XCheckTypedWindowEvent(**otk::display, _window, DestroyNotify, &ev)) {
XPutBackEvent(**otk::display, &ev);
return false;
}
+ while (XCheckTypedWindowEvent(**otk::display, _window, UnmapNotify, &ev)) {
+ if (ignore_unmaps) {
+ --ignore_unmaps;
+ } else {
+ XPutBackEvent(**otk::display, &ev);
+ return false;
+ }
+ }
if (_can_focus)
XSetInputFocus(**otk::display, _window,
otk::EventHandler::configureRequestHandler(e);
- // XXX: if we are iconic (or shaded? (fvwm does that)) ignore the event
+ // if we are iconic (or shaded (fvwm does this)) ignore the event
+ if (_iconic || _shaded) return;
if (e.value_mask & CWBorderWidth)
_border_width = e.border_width;
{
if (ignore_unmaps) {
#ifdef DEBUG
- printf("Ignored UnmapNotify for 0x%lx (event 0x%lx)\n", e.window, e.event);
+// printf("Ignored UnmapNotify for 0x%lx (event 0x%lx)\n", e.window, e.event);
#endif // DEBUG
ignore_unmaps--;
return;
openbox->screen(_screen)->unmanageWindow(this);
}
+void Client::mapRequestHandler(const XMapRequestEvent &e)
+{
+#ifdef DEBUG
+ printf("MapRequest for already managed 0x%lx\n", e.window);
+#endif // DEBUG
+
+ assert(_iconic); // we shouldn't be able to get this unless we're iconic
+
+ // move to the current desktop (uniconify)
+ setDesktop(openbox->screen(_screen)->desktop());
+ // XXX: should we focus/raise the window? (basically a net_wm_active_window)
+}
+
}