Ubuntu Calculator App

Merge lp:~mihirsoni/ubuntu-calculator-app/RemoveEqualSign-1203267 into lp:~ubuntu-calculator-dev/ubuntu-calculator-app/old_trunk

Proposed by Mihir Soni on 2013-08-06

Status:

Merged

Approved by:

Gustavo Pichorim Boiko on 2013-08-06

Approved revision:

122

Merged at revision:

122

Proposed branch:

Merge into:

Diff against target:

1 file modified

Simple/Screen.qml (+1/-1)

To merge this branch:

bzr merge lp:~mihirsoni/ubuntu-calculator-app/RemoveEqualSign-1203267

Medium

Fix Released

Link a bug report

Reviewer	Review Type	Status
Gustavo Pichorim Boiko (community)		Approve on 2013-08-06
Ubuntu Phone Apps Jenkins Bot	continuous-integration	Approve on 2013-08-06
Mihir Soni		Pending
Review via email: mp+178673@code.launchpad.net

Removed (=) equal sign from the result

Removed (=) equal sign from result

Revision history for this message

Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote on 2013-08-06:

review: Approve (continuous-integration)

Revision history for this message

Gustavo Pichorim Boiko (boiko) wrote on 2013-08-06:

Looks good.

review: Approve

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

People subscribed via source and target branches

to all changes:

1	=== modified file 'Simple/Screen.qml'
2	--- Simple/Screen.qml 2013-08-02 02:51:30 +0000
3	+++ Simple/Screen.qml 2013-08-06 07:46:27 +0000
4	@@ -235,7 +235,7 @@
5	id: formulaLabel
6	labelText: _text
7	numbers: _number
8	- operation: _operation
9	+ operation: _operation == "=" ? "" : _operation
10	numbersHeight: units.gu(5)
11	isLast: (isLastItem && index === repeater.model.count-1)
12