Merge lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/readOnlyWithStyle into lp:ubuntu-ui-toolkit/staging

Proposed by Cris Dywan
Status: Merged
Approved by: Zsombor Egri
Approved revision: 1283
Merge reported by: Zoltan Balogh
Merged at revision: not available
Proposed branch: lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/readOnlyWithStyle
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 62 lines (+15/-0)
4 files modified
examples/ubuntu-ui-toolkit-gallery/TextInputs.qml (+12/-0)
modules/Ubuntu/Components/TextCursor.qml (+1/-0)
modules/Ubuntu/Components/Themes/Ambiance/TextAreaStyle.qml (+1/-0)
modules/Ubuntu/Components/Themes/Ambiance/TextCursorStyle.qml (+1/-0)
To merge this branch: bzr merge lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/readOnlyWithStyle
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Zsombor Egri Approve
Review via email: mp+237449@code.launchpad.net

Commit message

Read-only text entries don't blink or visibly take focus

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Zsombor Egri (zsombi) wrote :

Ok, looks good.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'examples/ubuntu-ui-toolkit-gallery/TextInputs.qml'
2--- examples/ubuntu-ui-toolkit-gallery/TextInputs.qml 2014-09-17 15:11:08 +0000
3+++ examples/ubuntu-ui-toolkit-gallery/TextInputs.qml 2014-10-09 09:09:50 +0000
4@@ -74,10 +74,22 @@
5 }
6
7 TemplateRow {
8+ title: i18n.tr("Read-only")
9+
10+ TextField {
11+ objectName: "textfield_read_only"
12+ text: i18n.tr("Nobody type here")
13+ readOnly: true
14+ width: parent.width
15+ }
16+ }
17+
18+ TemplateRow {
19 title: i18n.tr("Disabled")
20
21 TextField {
22 objectName: "textfield_disabled"
23+ text: i18n.tr("No interaction allowed")
24 enabled: false
25 width: parent.width
26 }
27
28=== modified file 'modules/Ubuntu/Components/TextCursor.qml'
29--- modules/Ubuntu/Components/TextCursor.qml 2014-09-19 09:07:48 +0000
30+++ modules/Ubuntu/Components/TextCursor.qml 2014-10-09 09:09:50 +0000
31@@ -129,6 +129,7 @@
32 }
33 property bool typing: false
34 property bool contextMenuVisible: false
35+ property bool readOnly: handler.main.readOnly
36 function contextMenuHidden(p) {
37 contextMenuVisible = false
38 }
39
40=== modified file 'modules/Ubuntu/Components/Themes/Ambiance/TextAreaStyle.qml'
41--- modules/Ubuntu/Components/Themes/Ambiance/TextAreaStyle.qml 2014-09-18 15:42:37 +0000
42+++ modules/Ubuntu/Components/Themes/Ambiance/TextAreaStyle.qml 2014-10-09 09:09:50 +0000
43@@ -61,6 +61,7 @@
44 onErrorChanged: (error) ? visuals.errorColor : visuals.backgroundColor;
45 color: visuals.backgroundColor;
46 anchors.fill: parent
47+ visible: !styledItem.readOnly
48 }
49
50 Loader {
51
52=== modified file 'modules/Ubuntu/Components/Themes/Ambiance/TextCursorStyle.qml'
53--- modules/Ubuntu/Components/Themes/Ambiance/TextCursorStyle.qml 2014-09-22 07:19:56 +0000
54+++ modules/Ubuntu/Components/Themes/Ambiance/TextCursorStyle.qml 2014-10-09 09:09:50 +0000
55@@ -63,6 +63,7 @@
56 running: (cursorStyle.cursorVisibleTimeout > 0) &&
57 (cursorStyle.cursorHiddenTimeout > 0) &&
58 styledItem.visible &&
59+ !styledItem.readOnly &&
60 !styledItem.contextMenuVisible &&
61 styledItem.positionProperty === "cursorPosition"
62 repeat: true

Subscribers

People subscribed via source and target branches