Differential D18918 Diff 51456 containmentactions/contextmenu/menu.cpp

Changeset View

Standalone View

containmentactions/contextmenu/menu.cpp

Show First 20 Lines • Show All 59 Lines • ▼ Show 20 Line(s)		59	{
60	m_separator1 = new QAction(this);	60		m_separator1 = new QAction(this);
61	m_separator1->setSeparator(true);	61		m_separator1->setSeparator(true);
62		62
63	m_printAction = new QAction(QIcon::fromTheme("edit"), "Print Message...", this);	63		m_printAction = new QAction(QIcon::fromTheme("edit"), "Print Message...", this);
64	connect(m_printAction, &QAction::triggered, [ = ]() {	64		connect(m_printAction, &QAction::triggered, [ = ]() {
65	qDebug() << "Action Trigerred !!!";	65		qDebug() << "Action Trigerred !!!";
66	});	66		});
67		67
68	m_addWidgetsAction = new QAction(QIcon::fromTheme("add"), i18n("Add Widgets..."), this);	68		m_addWidgetsAction = new QAction(QIcon::fromTheme("add"), i18n("Add Widgets..."), this);
			mvourlakosUnsubmitted Not Done when we add character & in the menus it means that a modifier+&Letter activates that option what is the shortcut that add widgets? mvourlakos: when we add character & in the menus it means that a modifier+&Letter activates that option…
			trmdiAuthorUnsubmitted Not Done For example, "&Add widgets..." shortcut: `Alt+a` the shortcut only works when that menu is showing it does not change anything else trmdi: For example, "&Add widgets..." - shortcut: `Alt+a` - the shortcut only works when that menu is…
69	m_addWidgetsAction->setStatusTip(i18n("Show Plasma Widget Explorer"));	69		m_addWidgetsAction->setStatusTip(i18n("Show Plasma Widget Explorer"));
70	connect(m_addWidgetsAction, &QAction::triggered, [ = ]() {	70		connect(m_addWidgetsAction, &QAction::triggered, [ = ]() {
71	QDBusInterface iface("org.kde.plasmashell", "/PlasmaShell", "", QDBusConnection::sessionBus());	71		QDBusInterface iface("org.kde.plasmashell", "/PlasmaShell", "", QDBusConnection::sessionBus());
72		72
73	if (iface.isValid()) {	73		if (iface.isValid()) {
74	iface.call("toggleWidgetExplorer");	74		iface.call("toggleWidgetExplorer");
75	}	75		}
76	});	76		});
77		77
78	m_configureAction = new QAction(QIcon::fromTheme("configure"), i18nc("view settings window", "View Settings..."), this);	78		m_configureAction = new QAction(QIcon::fromTheme("configure"), i18nc("view settings window", "View Settings..."), this);
		79		setConfigureActionText();
79	m_configureAction->setShortcut(QKeySequence());	80		m_configureAction->setShortcut(QKeySequence());
			mvourlakosUnsubmitted Not Done when we add character & in the menus it means that a modifier+&Letter activates that option what is the shortcut that shows dock settings? it has been chosen to be Meta+A so that change does not respond to something mvourlakos: when we add character & in the menus it means that a modifier+&Letter activates that option…
80	connect(m_configureAction, &QAction::triggered, this, &Menu::requestConfiguration);	81		connect(m_configureAction, &QAction::triggered, this, &Menu::requestConfiguration);
81		82
82	m_switchLayoutsMenu = new QMenu;	83		m_switchLayoutsMenu = new QMenu;
83	m_layoutsAction = m_switchLayoutsMenu->menuAction();	84		m_layoutsAction = m_switchLayoutsMenu->menuAction();
84	m_layoutsAction->setText(i18n("Layouts"));	85		m_layoutsAction->setText(i18n("Layouts"));
			mvourlakosUnsubmitted Not Done when we add character & in the menus it means that a modifier+&Letter activates that option what is the shortcut that shows Latte layouts menu? none, this shouldnt be applied also mvourlakos: when we add character & in the menus it means that a modifier+&Letter activates that option…
85	m_layoutsAction->setIcon(QIcon::fromTheme("user-identity"));	86		m_layoutsAction->setIcon(QIcon::fromTheme("user-identity"));
86	m_layoutsAction->setStatusTip(i18n("Switch to another layout"));	87		m_layoutsAction->setStatusTip(i18n("Switch to another layout"));
87		88
88	connect(m_switchLayoutsMenu, &QMenu::aboutToShow, this, &Menu::populateLayouts);	89		connect(m_switchLayoutsMenu, &QMenu::aboutToShow, this, &Menu::populateLayouts);
89	connect(m_switchLayoutsMenu, &QMenu::triggered, this, &Menu::switchToLayout);	90		connect(m_switchLayoutsMenu, &QMenu::triggered, this, &Menu::switchToLayout);
		91
		92		connect(this->containment(), &Plasma::Containment::userConfiguringChanged, this, &Menu::setConfigureActionText);
			mvourlakosUnsubmitted Not Done is this still needed if we check the value each time we show up the context menu? mvourlakos: is this still needed if we check the value each time we show up the context menu?
			trmdiAuthorUnsubmitted Not Done Yes, it's needed. Because when you're opening the Setting window and you right click on the view, it will close the Setting window, not a nice behavior. So we should hide it when the Setting window is opening. trmdi: Yes, it's needed. Because when you're opening the Setting window and you right click on the…
			mvourlakosUnsubmitted Not Done ok this a behavior change irrelevant with the patch but ok we can keep it mvourlakos: ok this a behavior change irrelevant with the patch but ok we can keep it
90	}	93		}
			mvourlakosUnsubmitted Not Done why this is needed? mvourlakos: why this is needed?
			trmdiAuthorUnsubmitted Not Done when the user opens View Settings, there is a case that he change the viewType, so we should set the text again. trmdi: - when the user opens View Settings, there is a case that he change the viewType, so we should…
			mvourlakosUnsubmitted Not Done in this slot is also needed: m_configureAction->setEnabled(true); there are cases that the Dock Settings is shown when !configuring but it is not enabled mvourlakos: in this slot is also needed: ``` m_configureAction->setEnabled(true); ``` there are cases…
			trmdiAuthorUnsubmitted Not Done I don't understand. my code only change the `visible` property, why does it have to handle the "enable" one? You meant the setting is showing but configuring==false? Sound weird. How to make it happen? trmdi: I don't understand. my code only change the `visible` property, why does it have to handle the…
91		94
			mvourlakosUnsubmitted Not Done style issue. even for single lines ifs if (..) { } mvourlakos: style issue. even for single lines ifs ``` if (..) { } ```
92		95
93	void Menu::requestConfiguration()	96		void Menu::requestConfiguration()
94	{	97		{
95	if (this->containment()) {	98		if (this->containment()) {
96	emit this->containment()->configureRequested(containment());	99		emit this->containment()->configureRequested(containment());
97	}	100		}
98	}	101		}
99		102
Show All 20 Lines		122	if (m_layoutsData.size() > 3) {
120	m_layoutsAction->setEnabled(true);	123		m_layoutsAction->setEnabled(true);
121	m_layoutsAction->setVisible(true);	124		m_layoutsAction->setVisible(true);
122	} else {	125		} else {
123	m_layoutsAction->setVisible(false);	126		m_layoutsAction->setVisible(false);
124	}	127		}
125		128
126	return actions;	129		return actions;
127	}	130		}
128		131
			mvourlakosUnsubmitted Not Done same thing for these & characters, do not respond to any shortcut so they should be removed mvourlakos: same thing for these & characters, do not respond to any shortcut so they should be removed
129	QAction *Menu::action(const QString &name)	132		QAction *Menu::action(const QString &name)
130	{	133		{
131	if (name == "add widgets") {	134		if (name == "add widgets") {
132	return m_addWidgetsAction;	135		return m_addWidgetsAction;
133	} else if (name == "configure") {	136		} else if (name == "configure") {
134	return m_configureAction;	137		return m_configureAction;
135	} else if (name == "layouts") {	138		} else if (name == "layouts") {
136	return m_layoutsAction;	139		return m_layoutsAction;
▲ Show 20 Lines • Show All 67 Lines • ▼ Show 20 Line(s)		205	QTimer::singleShot(400, [this, layout]() {
204		207
205	if (iface.isValid()) {	208		if (iface.isValid()) {
206	iface.call("switchToLayout", layout);	209		iface.call("switchToLayout", layout);
207	}	210		}
208	});	211		});
209	}	212		}
210	}	213		}
211		214
		215		void Menu::setConfigureActionText()
		216		{
		217		Latte::Types::ViewType viewType = static_cast<Latte::Types::ViewType>(this->containment()->config()
		218		.readEntry("viewType", static_cast<int>(Latte::Types::DockView)));
		219		m_configureAction->setText(i18nc("view settings window", viewType == Latte::Types::DockView ? "Dock Settings..." : "Panel Settings..."));
		220		}
		221
212	K_EXPORT_PLASMA_CONTAINMENTACTIONS_WITH_JSON(lattecontextmenu, Menu, "plasma-containmentactions-lattecontextmenu.json")	222		K_EXPORT_PLASMA_CONTAINMENTACTIONS_WITH_JSON(lattecontextmenu, Menu, "plasma-containmentactions-lattecontextmenu.json")
213		223
214	#include "menu.moc"	224		#include "menu.moc"