Ubuntu
qtubuntu package

Browser does not respond to Qt.openUrlExternally(link)

Bug #1186556 reported by Jono Bacon
28
This bug affects 4 people
Affects Status Importance Assigned to Milestone
qtubuntu (Ubuntu)
Fix Released
Critical
Albert Astals Cid

Bug Description

I am not sure if this bug is part of the web browser or part of the SDK, but I have an app that has the following line of code:

   onClicked: Qt.openUrlExternally(link)

When running on the desktop, the app correctly loads the link in the default system web browser. This does not happen on the phone: when the item is clicked (tapped) there is no response (other than the ListItem depressing).

Related branches

lp:~aacid/qtubuntu/qtubunturl
Loïc Minier: Approve
Thomas Voß (community): Abstain
PS Jenkins bot: Approve (continuous-integration)
Diff: 151 lines (+87/-1)
5 files modified
src/platforms/base/base.pro (+2/-0)
src/platforms/base/integration.cc (+3/-1)
src/platforms/base/integration.h (+2/-0)
src/platforms/base/platformservices.cc (+50/-0)
src/platforms/base/platformservices.h (+30/-0)
lp:ubuntu/saucy-proposed/qtubuntu
Revision history for this message
Olivier Tilloy (osomon) wrote :

Qt.openUrlExternally(…) relies on QDesktopServices::openUrl(…), which in turn relies on the QPlatformServices instance exposed by the platform plugin (in that case qtubuntu).
As far as I can tell, qtubuntu doesn’t expose any QPlatformServices related functions, so this would have to be implemented there.

affects: webbrowser-app → qtubuntu
Changed in qtubuntu:
status: New → Confirmed
Olivier Tilloy (osomon)
summary: - Browser does not respond to Qt.resolvedUrl
+ Browser does not respond to Qt.openUrlExternally(link)
Revision history for this message
Loïc Molinari (loic.molinari) wrote :

Before being implemented in the QtUbuntu QPA plugin, we'd need support for it in the platform API, then we'll just have to wrap it.

Revision history for this message
Jono Bacon (jonobacon) wrote :

Is there any scheduled time when this bug will be resolved? My app is currently blocking on it, and I suspect many other apps will want to load pages in the browser too.

Florian Boucault (fboucault)
Changed in qtubuntu:
importance: Undecided → Critical
Revision history for this message
Florian Boucault (fboucault) wrote :

We could rely on the information provided by /usr/share/applications/defaults.list

Revision history for this message
Florian Boucault (fboucault) wrote :

See http://www.freedesktop.org/wiki/Specifications/mime-actions-spec/

Revision history for this message
Florian Boucault (fboucault) wrote :

http://qt-project.org/doc/qt-5.0/qtcore/qsettings.html should parse that easily

Revision history for this message
Florian Boucault (fboucault) wrote :

On the desktop, Qt actually uses xdg-open to select the browser. See: qtbase/src/platformsupport/services/genericunix/qgenericunixservices.cpp

I suppose we could do just the same.

Revision history for this message
Florian Boucault (fboucault) wrote :

xdg-open uses xdg-mime to essentially retrieve the default application from /usr/share/applications/defaults.list
However these 2 apps are perl scripts so I would go back to the initial recommendation: parse /usr/share/applications/defaults.list using QSettings.

Michał Sawicz (saviq)
Changed in qtubuntu:
status: Confirmed → In Progress
assignee: nobody → Albert Astals Cid (aacid)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:qtubuntu at revision 177, scheduled for release in qtubuntu, milestone ubuntu-13.05

Changed in qtubuntu:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtubuntu - 0.52+13.10.20130919-0ubuntu1

---------------
qtubuntu (0.52+13.10.20130919-0ubuntu1) saucy; urgency=low

  [ Albert Astals ]
  * Implement the backend for QDesktopServices::openUrl . (LP: #1186556)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 177
 -- Ubuntu daily release <email address hidden> Thu, 19 Sep 2013 14:09:32 +0000

Changed in qtubuntu (Ubuntu):
status: New → Fix Released
Albert Astals Cid (aacid)
Changed in qtubuntu:
status: Fix Committed → Fix Released
Michał Sawicz (saviq)
Changed in qtubuntu (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
importance: Undecided → Critical
no longer affects: qtubuntu
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.