Merge lp://staging/~donadigo/pantheon-calculator/fix-1624777-multiple-dependent into lp://staging/~elementary-apps/pantheon-calculator/trunk
Proposed by
Adam Bieńkowski
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Corentin Noël | ||||
| Approved revision: | 303 | ||||
| Merged at revision: | 303 | ||||
| Proposed branch: | lp://staging/~donadigo/pantheon-calculator/fix-1624777-multiple-dependent | ||||
| Merge into: | lp://staging/~elementary-apps/pantheon-calculator/trunk | ||||
| Diff against target: |
23 lines (+3/-3) 1 file modified
src/MainWindow.vala (+3/-3) |
||||
| To merge this branch: | bzr merge lp://staging/~donadigo/pantheon-calculator/fix-1624777-multiple-dependent | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Zisu Andrei (community) | Approve | ||
| elementary Apps team | Pending | ||
|
Review via email:
|
|||
Commit message
* Fix multiple calculators being always displaying the same content as others
Description of the change
This branch fixes bug #1624777: "Multiple calculators are not independent".
Instead of binding the properties to the UI which was causing same effects on all opened windows, save them on the window delete event.
To post a comment you must log in.
Revision history for this message
| Corentin Noël (tintou) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
I can confirm this branch fixes the issue.