Comment 4 for bug 699933

Revision history for this message
Cris Dywan (kalikiana) wrote :

The menu item needs to say "Customise..." with periods to indicate that it shows a dialogue. Also I think it needs a separator, otherwise I can't tell that it's not another check mark.

Part of the issue is not addressed: it should no longer show up in the tools/ app menu. This may however be slightly confusing to upgrading users, not entirely sure if that's an issue.

In terms of code, please respect the coding style used in the toolbar editor, ie. tabs rather than spaces. Looking at the style of the function is generally good measure if you're unsure.