Experiemental slider improvement

src/mumble/ListenerVolumeSlider.cpp

@@ -40,6 +40,7 @@ void ListenerVolumeSlider::on_VolumeSlider_changeCompleted() {
 	}
 
 	VolumeAdjustment adjustment = VolumeAdjustment::fromDBAdjustment(m_volumeSlider->value());
+	updateLabelValue();
 
 	if (handler->m_version >= Mumble::Protocol::PROTOBUF_INTRODUCTION_VERSION) {
 		// With the new audio protocol, volume adjustments for listeners are handled on the server and thus we want

src/mumble/MainWindow.cpp

@@ -67,6 +67,7 @@
 #endif
 
 #include <QAccessible>
+#include <QResizeEvent>
 #include <QtCore/QStandardPaths>
 #include <QtCore/QUrlQuery>
 #include <QtGui/QClipboard>

src/mumble/MenuLabel.cpp

@@ -7,13 +7,50 @@
 
 #include <QLabel>
 
-MenuLabel::MenuLabel(const QString &text, QObject *parent) : QWidgetAction(parent), m_text(text) {
+MenuLabel::MenuLabel(const QString &text, QObject *parent)
+	: QWidgetAction(parent), m_text(text), m_accessibleName(""), m_accessibleDescription("") {
 	setMenuRole(QAction::NoRole);
 }
 
 QWidget *MenuLabel::createWidget(QWidget *parent) {
 	QLabel *label = new QLabel(m_text, parent);
-	label->setFocusPolicy(Qt::TabFocus);
+
+	label->setAccessibleName(m_accessibleName);
+	label->setAccessibleDescription(m_accessibleDescription);
 
 	return label;
 }
+
+QString MenuLabel::text() {
+	return m_text;
+}
+
+void MenuLabel::setText(const QString &text) {
+	m_text = text;
+
+	QLabel *label = qobject_cast< QLabel * >(defaultWidget());
+	if (label) {
+		label->setText(text);
+		label->update();
+	}
+}
+
+void MenuLabel::setAccessibleName(const QString &name) {
+	m_accessibleName = name;
+
+	QLabel *label = qobject_cast< QLabel * >(defaultWidget());
+	if (label) {
+		label->setAccessibleName(m_accessibleName);
+		label->update();
+	}
+}
+
+void MenuLabel::setAccessibleDescription(const QString &description) {
+	m_accessibleDescription = description;
+
+	QLabel *label = qobject_cast< QLabel * >(defaultWidget());
+	if (label) {
+		label->setAccessibleDescription(m_accessibleDescription);
+		label->update();
+	}
+}

src/mumble/MenuLabel.h

@@ -17,11 +17,18 @@ class MenuLabel : public QWidgetAction {
 public:
 	MenuLabel(const QString &text, QObject *parent = nullptr);
 
+	QString text();
+	void setText(const QString &name);
+	void setAccessibleName(const QString &name);
+	void setAccessibleDescription(const QString &description);
+
 protected:
 	QWidget *createWidget(QWidget *parent) override;
 
 private:
-	const QString m_text;
+	QString m_text;
+	QString m_accessibleName;
+	QString m_accessibleDescription;
 };
 
 #endif

src/mumble/UserLocalVolumeSlider.cpp

@@ -43,5 +43,7 @@ void UserLocalVolumeSlider::on_VolumeSlider_changeCompleted() {
 		} else {
 			Global::get().mw->logChangeNotPermanent(QObject::tr("Local Volume Adjustment..."), user);
 		}
+
+		updateLabelValue();
 	}
 }

src/mumble/UserLocalVolumeSlider.h

@@ -23,8 +23,8 @@ class UserLocalVolumeSlider : public VolumeSliderWidgetAction {
 	void setUser(unsigned int sessionId);
 
 private slots:
-	void on_VolumeSlider_valueChanged(int value);
-	void on_VolumeSlider_changeCompleted();
+	void on_VolumeSlider_valueChanged(int value) override;
+	void on_VolumeSlider_changeCompleted() override;
 };
 
 #endif

src/mumble/VolumeSliderWidgetAction.cpp

@@ -4,22 +4,31 @@
 // Mumble source tree or at <https://www.mumble.info/LICENSE>.
 
 #include "VolumeSliderWidgetAction.h"
+#include "MumbleApplication.h"
 #include "InlineWidgets.h"
 #include "VolumeAdjustment.h"
 #include "widgets/EventFilters.h"
 
 #include <QSlider>
 #include <QToolTip>
+#include <QHBoxLayout>
+#include <QLabel>
 
 VolumeSliderWidgetAction::VolumeSliderWidgetAction(QWidget *parent)
-	: QWidgetAction(parent), m_volumeSlider(make_qt_unique< QSlider >(Qt::Horizontal, parent)) {
+	: QWidgetAction(parent),
+	m_widget(make_qt_unique< QWidget >(parent)),
+	m_volumeSlider(new QSlider(Qt::Horizontal, parent)),
+	m_label(new QLabel("0 db", parent)) {
+
 	m_volumeSlider->setMinimum(-30);
 	m_volumeSlider->setMaximum(30);
-	m_volumeSlider->setAccessibleName(tr("Slider for volume adjustment"));
-	m_volumeSlider->setSizePolicy(QSizePolicy::MinimumExpanding, QSizePolicy::MinimumExpanding);
+	m_volumeSlider->setAccessibleName(tr("Local volume adjustment"));
+
+	m_label->setStyleSheet("QLabel { margin-left: 0px; padding: 0px; }");
+	m_volumeSlider->setStyleSheet("QSlider { margin-right: 0px; }");
 
 	KeyEventObserver *keyEventFilter = new KeyEventObserver(this, QEvent::KeyRelease, false,
-															{ Qt::Key_Left, Qt::Key_Right, Qt::Key_Up, Qt::Key_Down });
+															{ Qt::Key_Left, Qt::Key_Right });
 	m_volumeSlider->installEventFilter(keyEventFilter);
 
 	// The list of wheel events observed seems odd at first. We have to check for multiple
@@ -33,27 +42,50 @@ VolumeSliderWidgetAction::VolumeSliderWidgetAction(QWidget *parent)
 		new MouseWheelEventObserver(this, { Qt::ScrollMomentum, Qt::ScrollUpdate, Qt::ScrollEnd }, false);
 	m_volumeSlider->installEventFilter(wheelEventFilter);
 
-	connect(m_volumeSlider.get(), &QSlider::valueChanged, this,
+	connect(m_volumeSlider, &QSlider::valueChanged, this,
 			&VolumeSliderWidgetAction::on_VolumeSlider_valueChanged);
-	connect(m_volumeSlider.get(), &QSlider::sliderReleased, this,
+	connect(m_volumeSlider, &QSlider::sliderReleased, this,
+			[=]() { m_volumeSlider->setFocus(Qt::TabFocusReason); updateLabelValue(false); });
+	connect(m_volumeSlider, &QSlider::sliderReleased, this,
 			&VolumeSliderWidgetAction::on_VolumeSlider_changeCompleted);
 	connect(keyEventFilter, &KeyEventObserver::keyEventObserved, this,
 			&VolumeSliderWidgetAction::on_VolumeSlider_changeCompleted);
 	connect(wheelEventFilter, &MouseWheelEventObserver::wheelEventObserved, this,
 			&VolumeSliderWidgetAction::on_VolumeSlider_changeCompleted);
 
-	setDefaultWidget(m_volumeSlider.get());
-
 	UpDownKeyEventFilter *eventFilter = new UpDownKeyEventFilter(this);
 	m_volumeSlider->installEventFilter(eventFilter);
 
 	m_volumeSlider->setProperty("mouseTracking", true);
+
+	QHBoxLayout *layout = new QHBoxLayout();
+	layout->addWidget(m_volumeSlider);
+	layout->addWidget(m_label);
+	layout->setContentsMargins(0, 0, -1, 0);
+	layout->setSpacing(3);
+	m_widget->setLayout(layout);
+
+	m_widget->setFocusProxy(m_volumeSlider);
+	m_widget->setFocusPolicy(Qt::TabFocus);
+
+	setDefaultWidget(m_widget.get());
+}
+
+void VolumeSliderWidgetAction::updateLabelValue(bool checkMouseButtons) {
+	if (checkMouseButtons && MumbleApplication::instance()->mouseButtons() != Qt::NoButton) {
+		// Do not update the label while the user is dragging the slider.
+		// This will otherwise cause a glitchy experience.
+		return;
+	}
+
+	m_label->setText(QString("%01dB").arg(m_volumeSlider->value()));
 }
 
 void VolumeSliderWidgetAction::updateSliderValue(float value) {
 	int dbShift = VolumeAdjustment::toIntegerDBAdjustment(value);
 	m_volumeSlider->setValue(dbShift);
 	updateTooltip(dbShift);
+	updateLabelValue(false);
 }
 
 void VolumeSliderWidgetAction::updateTooltip(int value) {

src/mumble/VolumeSliderWidgetAction.h

@@ -11,6 +11,8 @@
 #include "QtUtils.h"
 
 class QSlider;
+class QHBoxLayout;
+class QLabel;
 
 class VolumeSliderWidgetAction : public QWidgetAction {
 	Q_OBJECT
@@ -19,8 +21,11 @@ class VolumeSliderWidgetAction : public QWidgetAction {
 	VolumeSliderWidgetAction(QWidget *parent = nullptr);
 
 protected:
-	qt_unique_ptr< QSlider > m_volumeSlider;
+	qt_unique_ptr< QWidget > m_widget;
+	QSlider* m_volumeSlider;
+	QLabel* m_label;
 
+	void updateLabelValue(bool checkMouseButtons = true);
 	void updateSliderValue(float value);
 	void displayTooltip(int value);
 	void updateTooltip(int value);