diff options
author | Yulong Bai <yulong.bai@qt.io> | 2017-12-18 12:55:15 +0100 |
---|---|---|
committer | Yulong Bai <yulong.bai@qt.io> | 2018-01-21 17:54:47 +0000 |
commit | 4d898a73e63e69e63bcb2c26ca15da595d7efb7a (patch) | |
tree | 62bddc4e70aad7f542fb8340dd2ccd34dc8b440a /src/gui | |
parent | b3e7e9b182cb5a5e2b5e3592352ff1ea0e264c2d (diff) |
QMenuBar: Fix repetitive emission of triggered() when using addAction(QString)
The action which added by QMenuBar::addAction(const QString &text) already
connected relevant signals and slots implicitly, however, while
QMenuBarPrivate::updateGeometry -ing, it reconnects them
if there's a extension button associated with a hidden popup menu.
In that case the QMenuBar::triggered would be fired twice.
Since the QAction's ownership may be changed or added dynamically,
there are still very rare cases like several widgets share the same
QAction object to result in this problem.
[ChangeLog][QtWidgets][QMenu] Fixed a bug in QMenu that caused QMenuBar::triggered
to be fired multiple times.
Task-number: QTBUG-25669
Change-Id: I4d52e82a2136a992e0b37118e41237d96a2c5d22
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@qt.io>
Diffstat (limited to 'src/gui')
0 files changed, 0 insertions, 0 deletions