Déjà Dup

Merge lp://staging/~jbicha/deja-dup/fix-help-link into lp://staging/deja-dup/28

fix-help-link
Merge into 28

Proposed by Jeremy Bícha on 2013-07-11

Status:	Merged
Merged at revision:	1457
Proposed branch:	lp://staging/~jbicha/deja-dup/fix-help-link
Merge into:	lp://staging/deja-dup/28
Diff against target:	22 lines (+9/-0) 1 file modified preferences/PreferencesPanel.c (+9/-0)
To merge this branch:	bzr merge lp://staging/~jbicha/deja-dup/fix-help-link
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Déjà Dup Developers		2013-07-11	Pending
Review via email: mp+174287@code.staging.launchpad.net

Commit message

fix control-center help link

Description of the change

This fixes System Settings' built-in help link in the System Settings menu in the top menu bar. Also, F1 will now work correctly.

Revision history for this message

Michael Terry (mterry) wrote on 2013-07-15:

This looks good, works as advertised. I wonder if we shouldn't also remove the Help button and add a similar global app menu with Help/Close to deja-dup-preferences?

Revision history for this message

Jeremy Bícha (jbicha) wrote on 2013-07-15:

Yeah, that should be possible since it's already using GtkApplication.

Revision history for this message

Jim Campbell (jwcampbell) wrote on 2013-07-15:

Tangentially related, but Michael had you seen my proposed merge for the
documentation?

https://code.launchpad.net/~jwcampbell/deja-dup/deja-dup-doc-updates

Jim

On Mon, Jul 15, 2013 at 3:50 PM, Jeremy Bicha <email address hidden> wrote:

> Yeah, that should be possible since it's already using GtkApplication.
> --
> https://code.launchpad.net/~jbicha/deja-dup/fix-help-link/+merge/174287
> You are subscribed to branch lp:deja-dup.
>