Merge lp:~daker/ubuntu-ui-toolkit/fix.1658121 into lp:ubuntu-ui-toolkit/staging

Proposed by Adnane Belmadiaf
Status: Merged
Approved by: Cris Dywan
Approved revision: 2162
Merged at revision: 2162
Proposed branch: lp:~daker/ubuntu-ui-toolkit/fix.1658121
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 32 lines (+11/-0)
2 files modified
components.api (+1/-0)
src/imports/Components/1.3/TextArea.qml (+10/-0)
To merge this branch: bzr merge lp:~daker/ubuntu-ui-toolkit/fix.1658121
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve
Cris Dywan Approve
Review via email: mp+315241@code.launchpad.net

Commit message

Expose append method in TextArea

Description of the change

Expose append method in TextArea

To post a comment you must log in.
Revision history for this message
Cris Dywan (kalikiana) wrote :

Looks good. Thanks for adding this!

review: Approve
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :

FAILED: Autolanding.
No commit message was specified in the merge proposal. Hit 'Add commit message' on the merge proposal web page or follow the link below. You can approve the merge proposal yourself to rerun.
https://code.launchpad.net/~daker/ubuntu-ui-toolkit/fix.1658121/+merge/315241/+edit-commit-message

review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Adnane Belmadiaf (daker) wrote :

What do i need to do to fix the api version issue ?

Revision history for this message
Cris Dywan (kalikiana) wrote :

> What do i need to do to fix the api version issue ?

You need to update components.api, either as part of a local 'make check' or ./tests/qmlapicheck.sh, and replacing it with the generated components.api.new. The diff between the files is what you see in the failed build log.

2162. By Adnane Belmadiaf

Update to components.api

Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'components.api'
--- components.api 2016-10-07 10:54:38 +0000
+++ components.api 2017-01-25 12:54:49 +0000
@@ -1416,6 +1416,7 @@
1416 function var cut()1416 function var cut()
1417 function var deselect()1417 function var deselect()
1418 function var insert(var position, var text)1418 function var insert(var position, var text)
1419 function var append(var text)
1419 function var positionAt(var x, var y)1420 function var positionAt(var x, var y)
1420 function var isRightToLeft(var start, var end)1421 function var isRightToLeft(var start, var end)
1421 function var moveCursorSelection(var position, var mode)1422 function var moveCursorSelection(var position, var mode)
14221423
=== modified file 'src/imports/Components/1.3/TextArea.qml'
--- src/imports/Components/1.3/TextArea.qml 2016-11-10 15:36:49 +0000
+++ src/imports/Components/1.3/TextArea.qml 2017-01-25 12:54:49 +0000
@@ -581,6 +581,16 @@
581 }581 }
582582
583 /*!583 /*!
584 Appends a new paragraph with text to the end of the TextArea.
585
586 In order to append without inserting a new paragraph, call TextArea.insert(TextArea.length, text) instead.
587 */
588 function append(text)
589 {
590 editor.append(text);
591 }
592
593 /*!
584 Returns the text position closest to pixel position (x, y).594 Returns the text position closest to pixel position (x, y).
585595
586 Position 0 is before the first character, position 1 is after the first596 Position 0 is before the first character, position 1 is after the first

Subscribers

People subscribed via source and target branches