Switchboard

Merge lp://staging/~donadigo/switchboard/return-to-previous-plug into lp://staging/~elementary-pantheon/switchboard/switchboard

  1. return-to-previous-plug
  2. Merge into switchboard
Proposed by Adam Bieńkowski
Status: Merged
Approved by: Danielle Foré
Approved revision: 574
Merged at revision: 567
Proposed branch: lp://staging/~donadigo/switchboard/return-to-previous-plug
Merge into: lp://staging/~elementary-pantheon/switchboard/switchboard
Diff against target: 66 lines (+19/-2)
1 file modified
src/Switchboard.vala (+19/-2)
To merge this branch: bzr merge lp://staging/~donadigo/switchboard/return-to-previous-plug
Reviewer Review Type Date Requested Status
Danielle Foré Needs Fixing
Review via email: mp+271963@code.staging.launchpad.net

Commit message

When redirected to an other plug make navigation button return to previous plug.

Description of the change

Fixes bug #1497553. When redirected to an other plug make navigation button return to previous plug. This should support multiple redicretions as previous plugs are stored in the list instead of one variable.

To post a comment you must log in.
lp://staging/~donadigo/switchboard/return-to-previous-plug updated
570. By Adam Bieńkowski

Remove unneded line

571. By Adam Bieńkowski

Fix a code a little bit

572. By Adam Bieńkowski

Missing comment

Revision history for this message
Danielle Foré (danrabbit) wrote :

Works for me :)

Only thing I can see is that it looks like current code style is to put a space before <> so "Gee.ArrayList<Switchboard.Plug>" becomes "Gee.ArrayList <Switchboard.Plug>"

review: Needs Fixing
lp://staging/~donadigo/switchboard/return-to-previous-plug updated
573. By Adam Bieńkowski

Small code-style fix

574. By Adam Bieńkowski

Another code-style fix

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: