pidgin/pidgin

Convert PidginStatusBox to a GtkMenuButton

13 months ago, Elliott Sales de Andrade
adf6d358d438
Convert PidginStatusBox to a GtkMenuButton

This moves to a menu system to populate the list of statuses, and the new `PidginStatusDisplay` to show them. I tried a `GtkDropDown` previously, but that was unable to show the separators that we previously had. Consequently, it uses actions a bit more then before.

However, since menus don't support icons+text at the same time, and doesn't have a factory property, we do need to manually create custom widgets a bit, but it's mostly only a button + the `PidginStatusDisplay`.

The main display is now centred, but there seems to be nothing we can do about that, as the box with that alignment is internal to the `GtkMenuButton`. If we don't want that, we could switch to managing a `GtkToggleButton` and `GtkPopoverMenu` ourselves.

Testing Done:
Opened status box and confirmed that all primitives and saved statuses were there as expected.
Modified a status, and confirmed that the menu updated to match.
Added/deleted statuses from the manager and confirmed they menu updated to match.
Chose the New Status/Saved Statuses menu items and confirmed they opened the correct dialogs.
Picked a bunch of statuses from the menu items and confirmed the main display updated, and some updates were sent out to protocols (e.g. Demo disconnecting when offline, etc.)

Reviewed at https://reviews.imfreedom.org/r/2386/
/*
* Pidgin - Internet Messenger
* Copyright (C) Pidgin Developers <devel@pidgin.im>
*
* Pidgin is the legal property of its developers, whose names are too numerous
* to list here. Please refer to the COPYRIGHT file distributed with this
* source distribution.
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation; either version 2 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, see <https://www.gnu.org/licenses/>.
*/
#if !defined(PIDGIN_GLOBAL_HEADER_INSIDE) && !defined(PIDGIN_COMPILATION)
# error "only <pidgin.h> may be included directly"
#endif
#ifndef PIDGIN_KEYPAD_H
#define PIDGIN_KEYPAD_H
#include <gtk/gtk.h>
#include <purple.h>
/**
* PidginKeypad:
*
* #PidginKeypad is a widget that displays a DTMF keypad, with the digits 0-9,
* an asterisk, and a number sign.
*
* Since: 3.0.0
*/
G_BEGIN_DECLS
#define PIDGIN_TYPE_KEYPAD pidgin_keypad_get_type()
G_DECLARE_FINAL_TYPE(PidginKeypad, pidgin_keypad, PIDGIN, KEYPAD, GtkGrid)
/**
* pidgin_keypad_new:
*
* Creates a new #PidginKeypad.
*
* Returns: (transfer full): The new instance.
*
* Since: 3.0.0
*/
GtkWidget *pidgin_keypad_new(void);
/**
* pidgin_keypad_set_key_capture_widget:
* @keypad: The keypad.
* @widget: A widget to capture keys from.
*
* Sets @widget as the widget that @keypad will capture key events from.
*
* If key events are handled by the keypad, the DTMF digits will be captured
* and trigger the pressed signal on @keypad.
*
* Since: 3.0.0
*/
void pidgin_keypad_set_key_capture_widget(PidginKeypad *keypad, GtkWidget *widget);
G_END_DECLS
#endif /* PIDGIN_KEYPAD_H */