TextArea does not export TextEdit's textDocument property

Bug #1392276 reported by Michael Zanetti
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu Notes app
Fix Released
High
Michael Zanetti
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Medium
Cris Dywan
Vivid
New
Undecided
Unassigned

Bug Description

This would be needed for advanced RTF formatting like this: http://i.imgur.com/dejnDU7.png

Right now, falling back to plain TextEdit is required.

This is a feature to improve the UX of the Reminders core app, so that it can use the Ubuntu text selection and cursor to be better integrated.

In terms of reusing the code that the Reminders app provides to have a full QML API, while technically possible, would require a non-trivial amount of work to make it generic enough for use for other apps. The features of the Reminders text editor are tightly coupled with the Evernote API.

Related branches

Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Zsombor Egri (zsombi)
Revision history for this message
Cris Dywan (kalikiana) wrote :

I'm not comfortable aliasing this as-is because it's not QML API (see http://qt-project.org/doc/qt-5/qquicktextdocument.html) but merely an object that you need to hand over to C++.

We should rather have a QML API for text formatting. Whatever produced the screenshot can probably make a basis for that?

Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: Confirmed → Opinion
Revision history for this message
Michael Zanetti (mzanetti) wrote :

What's the problem with having a property that needs to be handed over to Qt? If you don't want to do that, noone forces you to use the property. But if someone actually wants to do so I don't think we should forbid that.

Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit:
assignee: Zsombor Egri (zsombi) → Christian Dywan (kalikiana)
no longer affects: ubuntu-ui-toolkit (Ubuntu)
affects: ubuntu-ui-toolkit → ubuntu-ui-toolkit (Ubuntu)
David Planella (dpm)
description: updated
David Planella (dpm)
Changed in reminders-app:
status: New → Triaged
importance: Undecided → High
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Opinion → Confirmed
Revision history for this message
Cris Dywan (kalikiana) wrote :

For the record my concern is with the suspicion that an API that's not fully exported to QML may not live up to the standards and apply changes without emitting the signals we rely on.

I'll provide a branch to expose it, and let's aim to unit test it later based on the use cases in Reminders (not to say it'll be the guinea big here, being the only user of this API for now).

Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → In Progress
Changed in reminders-app:
status: Triaged → In Progress
assignee: nobody → Michael Zanetti (mzanetti)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1517+15.10.20150523-0ubuntu1

---------------
ubuntu-ui-toolkit (1.3.1517+15.10.20150523-0ubuntu1) wily; urgency=medium

  [ Christian Dywan ]
  * Expose textDocument of TextField. Fixes LP: #1392276
  * Unset GRID_UNIT_PX before checking default value. Fixes LP: #1457421
  * No cutting and copying of passwords. Fixes LP: #1438976
  * Implement new API tool based on qmlplugindump producing JSON.
    Fixes LP: #1187010

  [ Leo Arias ]
  * Make explicit the dependency on upstart for the autopilot helpers.

  [ Timo Jyrinki ]
  * uitk_test_plan.sh: Switch to vivid-overlay & wily instead of rtm-14.09 &
    vivid. Fixes LP: #1457015.

 -- CI Train Bot <email address hidden> Sat, 23 May 2015 17:24:43 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

Fix committed into lp:reminders-app at revision 476, scheduled for release in reminders-app, milestone 2014-11-27

Changed in reminders-app:
status: In Progress → Fix Committed
David Planella (dpm)
Changed in reminders-app:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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