Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make it possible to enable/disable main menu in Ubuntu Touch #375

Closed
rinigus opened this issue Mar 30, 2021 · 14 comments
Closed

Make it possible to enable/disable main menu in Ubuntu Touch #375

rinigus opened this issue Mar 30, 2021 · 14 comments

Comments

@rinigus
Copy link
Owner

rinigus commented Mar 30, 2021

Right now, enabled is just a dummy property. Fix it properly. Corresponding log from earlier issue:

Published using Logviewer for Ubuntu Touch
From file /home/phablet/.cache/upstart/application-click-osmscout-server.jonnius_osmscout-server_2.0.0.log:
Loading module: 'libubuntu_application_api_touch_mirclient.so.3.0.0'
library "libgui.so" wasn't loaded and RTLD_NOLOAD prevented it
Creating a QMirClientScreen now
qml: Error while creating component
qml: qrc:/qml/MainMenu.qml:29 "MenuDrawerPL.enabled" is not available due to component versioning.

qrc:/qml/platform/ApplicationWindowPL.qml:60: TypeError: Value is null and could not be converted to an object
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Cannot call method 'open' of null
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Cannot call method 'open' of null
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Cannot call method 'open' of null
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Type error
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Type error
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Type error
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Type error
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Type error
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Type error
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Type error
qt.qpa.mirclient: Attempted to deliver an event to a non-existent window, ignoring.
Loading module: 'libubuntu_application_api_touch_mirclient.so.3.0.0'
library "libgui.so" wasn't loaded and RTLD_NOLOAD prevented it
Creating a QMirClientScreen now
qml: Error while creating component
qml: qrc:/qml/MainMenu.qml:29 "MenuDrawerPL.enabled" is not available due to component versioning.

qrc:/qml/platform/ApplicationWindowPL.qml:60: TypeError: Value is null and could not be converted to an object
[PERFORMANCE]: Last frame took 88 ms to render.
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Cannot call method 'open' of null
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Cannot call method 'open' of null
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Cannot call method 'open' of null
qrc:/qml/platform/ApplicationWindowPL.qml:82: TypeError: Type error
@rinigus
Copy link
Owner Author

rinigus commented Apr 21, 2021

As it is using Qt Controls2 Drawer, we just need to wait for Qt 5.12 update of Ubuntu Touch. See https://doc.qt.io/qt-5/qml-qtquick-controls2-popup.html#enabled-prop

@jonnius
Copy link
Contributor

jonnius commented Apr 21, 2021

We already have Qt 5.12 in Ubuntu Touch.

@rinigus
Copy link
Owner Author

rinigus commented Apr 21, 2021

The issue above should not have happened after Qt 5.10 - so I wonder whether update to 5.12 happened in meantime?

@jonnius
Copy link
Contributor

jonnius commented Apr 21, 2021

We were on Qt 5.12 when we hit the issue. I just tested removing 6dc09e3 and the issue persists. It seems we have QtQuickControls 2.5 (qtquickcontrols2-5-dev) in UT.

@rinigus
Copy link
Owner Author

rinigus commented Apr 21, 2021

Strange, there is no such version. See https://doc.qt.io/qt-5/qtquickcontrols-index.html#versions . I wonder why removing that commit doesn't fix it. The error

"MenuDrawerPL.enabled" is not available due to component versioning.

indicates that we are limiting it by versioning, but 2.12 should have it available. Puzzling and cannot really test it over here either.

@jonnius
Copy link
Contributor

jonnius commented Apr 21, 2021

Trying to get help in UT app dev group...

In fact you can reproduce the issue easily in Clickable Desktop Mode as explained in https://github.com/rinigus/osmscout-server/tree/master/packaging/click#readme.

@rinigus
Copy link
Owner Author

rinigus commented Apr 21, 2021

Couldn't get clickable desktop to run last night when I tried and then, after some struggling, had to stop with it. Let's see if we will get help.

@jonnius
Copy link
Contributor

jonnius commented Apr 21, 2021

I would be interested in understanding your struggle in order to improve Clickable.

@jonnius
Copy link
Contributor

jonnius commented Apr 21, 2021

Some comment from the group:

if you have other QMLs in the project importing a different version, it can cause problems. had this issue with like addressbook and dialer iirc, when updating to 5.12

@jonnius
Copy link
Contributor

jonnius commented Apr 21, 2021

And by the way, we have QtQuick and Controls 2.12 in UT, not 2.5. That number is something else.

@rinigus
Copy link
Owner Author

rinigus commented Apr 21, 2021

We do have a mix of versions, as we have support from Qt version of SFOS (5.6). So, the base is imported with the lower version. Maybe that is what we are hitting here. Let's keep it open and see when we can bump the versions for Qt.

As such, this issue is nothing major and may prevent some corner case when server process crashes. However, it will just be impossible to apply any changes anyway, so it should be OK for now.

@rinigus
Copy link
Owner Author

rinigus commented Jan 27, 2023

Is it resolved now with UT 20.04?

@jonnius
Copy link
Contributor

jonnius commented Jan 28, 2023

We already have reverted this workaround. See #402.

@rinigus
Copy link
Owner Author

rinigus commented Jan 28, 2023

Excellent. closing it here then.

@rinigus rinigus closed this as completed Jan 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants