| 2016-08-12 18:27:15 |
Dmitry Shachnev |
bug |
|
|
added bug |
| 2016-08-12 18:28:16 |
Dmitry Shachnev |
bug task added |
|
indicator-appmenu (Ubuntu) |
|
| 2016-08-13 17:53:58 |
Dmitry Shachnev |
description |
Currently we are using appmenu-qt5 as our platform theme on desktop, however it has many disadvantages, which make me want to get rid of it:
1) Its design is a hack: instead of the using normal QPA API for getting the menu, it retrieves the menu bar using QWidget::findChild, and then casts the pointer to QMenu*. The normal methods (which are getting called by Qt) remain empty stubs.
2) It is preventing applications from using GTK+ theme integration, such as dialogs (I reported this as bug 1378935, but there is no easy way to fix that).
3) Finally, it is mostly unmaintained: most patches since 2014 are authored by me, however I see no point continuing to develop that code.
Recently, Shawn Rutledge and I have written a native implementation of what appmenu-qt5 provides (global menu and system tray), which is part of Qt. That code uses the normal API, is well maintained, and works better than appmenu-qt5 (or at least not worse).
So I propose to drop appmenu-qt5 from default Ubuntu installations, and maybe later from archive too.
Unfortunately we use Qt 5.6, and some of the needed patches are only in Qt 5.7, so I would like to backport them to our packaging:
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=f199bb9133fe0446 (will be in 5.6.2 / 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=488cf78e44947eff (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=b6a824d0a3b4fabd (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=e4d79e1fdeb6b26b (will be in 5.7.0)
Timo: if you have no objections, I will commit those patches to the packaging Git.
After we do that, I will remove the appmenu-qt5 recommendation from indicator-appmenu (that is the only package referring to it). |
Currently we are using appmenu-qt5 as our platform theme on desktop, however it has many disadvantages, which make me want to get rid of it:
1) Its design is a hack: instead of the using normal QPA API for getting the menu, it retrieves the menu bar using QWidget::findChild, and then casts the pointer to QMenu*. The normal methods (which are getting called by Qt) remain empty stubs.
2) It is preventing applications from using GTK+ theme integration, such as dialogs (I reported this as bug 1378935, but there is no easy way to fix that).
3) It is not working with Qt Quick applications (because it expects a QtWidgets window; see also bug 1323853). The standard implementation will work with i.e. Qt Quick Controls 2 using apps (implemented in https://codereview.qt-project.org/142733).
4) Finally, it is mostly unmaintained: most patches since 2014 are authored by me, however I see no point continuing to develop that code.
Recently, Shawn Rutledge and I have written a native implementation of what appmenu-qt5 provides (global menu and system tray), which is part of Qt. That code uses the normal API, is well maintained, and works better than appmenu-qt5 (or at least not worse).
So I propose to drop appmenu-qt5 from default Ubuntu installations, and maybe later from archive too.
Unfortunately we use Qt 5.6, and some of the needed patches are only in Qt 5.7, so I would like to backport them to our packaging:
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=f199bb9133fe0446 (will be in 5.6.2 / 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=488cf78e44947eff (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=b6a824d0a3b4fabd (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=e4d79e1fdeb6b26b (will be in 5.7.0)
Timo: if you have no objections, I will commit those patches to the packaging Git.
After we do that, I will remove the appmenu-qt5 recommendation from indicator-appmenu (that is the only package referring to it). |
|
| 2016-08-22 16:49:55 |
Dmitry Shachnev |
description |
Currently we are using appmenu-qt5 as our platform theme on desktop, however it has many disadvantages, which make me want to get rid of it:
1) Its design is a hack: instead of the using normal QPA API for getting the menu, it retrieves the menu bar using QWidget::findChild, and then casts the pointer to QMenu*. The normal methods (which are getting called by Qt) remain empty stubs.
2) It is preventing applications from using GTK+ theme integration, such as dialogs (I reported this as bug 1378935, but there is no easy way to fix that).
3) It is not working with Qt Quick applications (because it expects a QtWidgets window; see also bug 1323853). The standard implementation will work with i.e. Qt Quick Controls 2 using apps (implemented in https://codereview.qt-project.org/142733).
4) Finally, it is mostly unmaintained: most patches since 2014 are authored by me, however I see no point continuing to develop that code.
Recently, Shawn Rutledge and I have written a native implementation of what appmenu-qt5 provides (global menu and system tray), which is part of Qt. That code uses the normal API, is well maintained, and works better than appmenu-qt5 (or at least not worse).
So I propose to drop appmenu-qt5 from default Ubuntu installations, and maybe later from archive too.
Unfortunately we use Qt 5.6, and some of the needed patches are only in Qt 5.7, so I would like to backport them to our packaging:
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=f199bb9133fe0446 (will be in 5.6.2 / 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=488cf78e44947eff (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=b6a824d0a3b4fabd (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=e4d79e1fdeb6b26b (will be in 5.7.0)
Timo: if you have no objections, I will commit those patches to the packaging Git.
After we do that, I will remove the appmenu-qt5 recommendation from indicator-appmenu (that is the only package referring to it). |
Currently we are using appmenu-qt5 as our platform theme on desktop, however it has many disadvantages, which make me want to get rid of it:
1) Its design is a hack: instead of the using normal QPA API for getting the menu, it retrieves the menu bar using QWidget::findChild, and then casts the pointer to QMenu*. The normal methods (which are getting called by Qt) remain empty stubs.
2) It is preventing applications from using GTK+ theme integration, such as dialogs (I reported this as bug 1378935, but there is no easy way to fix that).
3) It is not working with Qt Quick applications (because it expects a QtWidgets window; see also bug 1323853). The standard implementation will work with i.e. Qt Quick Controls 2 using apps (implemented in https://codereview.qt-project.org/142733).
4) It breaks other environments such as Plasma when installed: see bug 1434516.
5) Finally, it is mostly unmaintained: most patches since 2014 are authored by me, however I see no point continuing to develop that code.
Recently, Shawn Rutledge and I have written a native implementation of what appmenu-qt5 provides (global menu and system tray), which is part of Qt. That code uses the normal API, is well maintained, and works better than appmenu-qt5 (or at least not worse).
So I propose to drop appmenu-qt5 from default Ubuntu installations, and maybe later from archive too.
Unfortunately we use Qt 5.6, and some of the needed patches are only in Qt 5.7, so I would like to backport them to our packaging:
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=f199bb9133fe0446 (will be in 5.6.2 / 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=488cf78e44947eff (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=b6a824d0a3b4fabd (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=e4d79e1fdeb6b26b (will be in 5.7.0)
Timo: if you have no objections, I will commit those patches to the packaging Git.
After we do that, I will remove the appmenu-qt5 recommendation from indicator-appmenu (that is the only package referring to it). |
|
| 2016-09-01 06:09:46 |
Dmitry Shachnev |
attachment added |
|
bt_full.txt https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1612767/+attachment/4731962/+files/bt_full.txt |
|
| 2016-09-03 07:44:07 |
Dmitry Shachnev |
description |
Currently we are using appmenu-qt5 as our platform theme on desktop, however it has many disadvantages, which make me want to get rid of it:
1) Its design is a hack: instead of the using normal QPA API for getting the menu, it retrieves the menu bar using QWidget::findChild, and then casts the pointer to QMenu*. The normal methods (which are getting called by Qt) remain empty stubs.
2) It is preventing applications from using GTK+ theme integration, such as dialogs (I reported this as bug 1378935, but there is no easy way to fix that).
3) It is not working with Qt Quick applications (because it expects a QtWidgets window; see also bug 1323853). The standard implementation will work with i.e. Qt Quick Controls 2 using apps (implemented in https://codereview.qt-project.org/142733).
4) It breaks other environments such as Plasma when installed: see bug 1434516.
5) Finally, it is mostly unmaintained: most patches since 2014 are authored by me, however I see no point continuing to develop that code.
Recently, Shawn Rutledge and I have written a native implementation of what appmenu-qt5 provides (global menu and system tray), which is part of Qt. That code uses the normal API, is well maintained, and works better than appmenu-qt5 (or at least not worse).
So I propose to drop appmenu-qt5 from default Ubuntu installations, and maybe later from archive too.
Unfortunately we use Qt 5.6, and some of the needed patches are only in Qt 5.7, so I would like to backport them to our packaging:
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=f199bb9133fe0446 (will be in 5.6.2 / 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=488cf78e44947eff (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=b6a824d0a3b4fabd (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=e4d79e1fdeb6b26b (will be in 5.7.0)
Timo: if you have no objections, I will commit those patches to the packaging Git.
After we do that, I will remove the appmenu-qt5 recommendation from indicator-appmenu (that is the only package referring to it). |
Currently we are using appmenu-qt5 as our platform theme on desktop, however it has many disadvantages, which make me want to get rid of it:
1) Its design is a hack: instead of the using normal QPA API for getting the menu, it retrieves the menu bar using QWidget::findChild, and then casts the pointer to QMenu*. The normal methods (which are getting called by Qt) remain empty stubs.
2) It is preventing applications from using GTK+ theme integration, such as dialogs (I reported this as bug 1378935, but there is no easy way to fix that).
3) It is not working with Qt Quick applications (because it expects a QtWidgets window; see also bug 1323853). The standard implementation will work with i.e. Qt Quick Controls 2 using apps (implemented in https://codereview.qt-project.org/142733).
4) It breaks other environments such as Plasma or GNOME when installed: see bug 1434516, bug 1307619.
5) Finally, it is mostly unmaintained: most patches since 2014 are authored by me, however I see no point continuing to develop that code.
Recently, Shawn Rutledge and I have written a native implementation of what appmenu-qt5 provides (global menu and system tray), which is part of Qt. That code uses the normal API, is well maintained, and works better than appmenu-qt5 (or at least not worse).
So I propose to drop appmenu-qt5 from default Ubuntu installations, and maybe later from archive too.
Unfortunately we use Qt 5.6, and some of the needed patches are only in Qt 5.7, so I would like to backport them to our packaging:
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=f199bb9133fe0446 (will be in 5.6.2 / 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=488cf78e44947eff (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=b6a824d0a3b4fabd (will be in 5.7.0)
https://code.qt.io/cgit/qt/qtbase.git/commit/?id=e4d79e1fdeb6b26b (will be in 5.7.0)
Timo: if you have no objections, I will commit those patches to the packaging Git.
After we do that, I will remove the appmenu-qt5 recommendation from indicator-appmenu (that is the only package referring to it). |
|
| 2016-09-03 08:07:27 |
Rik Mills |
bug |
|
|
added subscriber Rik Mills |
| 2016-09-06 04:09:05 |
Launchpad Janitor |
qtbase-opensource-src (Ubuntu): status |
New |
Fix Released |
|
| 2016-09-27 11:23:40 |
Launchpad Janitor |
indicator-appmenu (Ubuntu): status |
New |
Confirmed |
|
| 2016-09-27 11:27:27 |
Albert Astals Cid |
bug |
|
|
added subscriber Albert Astals Cid |
| 2016-09-27 12:14:07 |
Launchpad Janitor |
branch linked |
|
lp:~mitya57/indicator-appmenu/drop-appmenu-qt5 |
|
| 2016-09-28 12:51:56 |
Launchpad Janitor |
indicator-appmenu (Ubuntu): status |
Confirmed |
Fix Released |
|
| 2017-01-27 13:57:10 |
eris23 |
bug |
|
|
added subscriber eris23 |
