Merge lp:~flscogna/ubuntu-terminal-app/fix-1365602 into lp:ubuntu-terminal-app

Proposed by Filippo Scognamiglio
Status: Merged
Approved by: Alan Pope 🍺🐧🐱 πŸ¦„
Approved revision: 144
Merged at revision: 144
Proposed branch: lp:~flscogna/ubuntu-terminal-app/fix-1365602
Merge into: lp:ubuntu-terminal-app
Diff against target: 24 lines (+5/-3)
1 file modified
src/app/qml/Terminal.qml (+5/-3)
To merge this branch: bzr merge lp:~flscogna/ubuntu-terminal-app/fix-1365602
Reviewer Review Type Date Requested Status
Alan Pope 🍺🐧🐱 πŸ¦„ (community) Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+234557@code.launchpad.net

Commit message

Fix: terminal starts in application directory instead of home.

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote :

Brilliant, thanks Filippo! Tested on my Nexus 4.

http://popey.mooo.com/screenshots/device-2014-09-13-123036.png

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/app/qml/Terminal.qml'
2--- src/app/qml/Terminal.qml 2014-09-04 03:54:20 +0000
3+++ src/app/qml/Terminal.qml 2014-09-12 22:36:22 +0000
4@@ -80,9 +80,6 @@
5 pgConf.colorSchemes = kterm.availableColorSchemes();
6
7 mview.actionManager.addLocalContext(actionContext);
8-
9- // Change directory to $HOME
10- ksession.sendText("cd $HOME;clear;\n")
11 }
12
13 Connections {
14@@ -251,6 +248,11 @@
15 Qt.quit()
16 }
17 }
18+
19+ Component.onCompleted: {
20+ // Change directory to $HOME
21+ ksession.sendText("cd $HOME;clear;\n");
22+ }
23 }
24 }
25

Subscribers

People subscribed via source and target branches