GNOME Bugzilla – Bug 413452
Wrong example for Preferred Application capplet usage
Last modified: 2009-02-12 08:08:10 UTC
Documentation Section: <sect2 id="prefs-preferredapps"> in goscustdesk.xml For example, you can specify Xterm as your preferred terminal application. When you open the System menu then choose Open Terminal, Xterm starts. Correct version: Better something like: "For example, you can specify the web browser application (Epiphany, Mozilla Firefox, Opera...) to launch when you enter a web address in the Run Application dialog or when you clic on a web link in other applications like email clients or document viewer" Other information: Note that there is no System -> Open Terminal command in default GNOME Desktop (a Sun customization?). Also the web browser example is better IMHO for a larger audience. In case, make sure to use the proper name for Firefox (FireFox? Mozilla Firefox? Firefox?)
*** Bug 452868 has been marked as a duplicate of this bug. ***
A possible fix is just removing the Xterm example and replacing with the example from the 8.1.5.1 section of goscustdesk.xml, which is similar to the correction above or using something like: "For example, you can specify the web browser Firefox to launch when you click on a web link in email clients or when you enter a web address in the Run Application dialog." Example from section 8.1.5.1: "For example, the preferred web browser opens when you select a URL in an application, or when you select a URL launcher on the desktop" Then remove the example from the Web Browser section because examples would be redundant. Also the other sections for email and terminal do not have examples.
Created attachment 108671 [details] [review] Preferred applications rewritten Here is a rewrite proposal. Basically, I removed the specific Web browser, Mail reader and Terminal sections to replace them with a more general "Custom command options" section.
Created attachment 108672 [details] Screenshot of new section Here's the corresponding Yelp screenshot for easier review.
A few comments: * "Run Application" should be inside <application>. Better still, just remove that bit. The Run Application dialog isn't in the Applications menu anymore, so it's kind of an insider-feature. * In that same sentence, change "web link" to "link" and "like" to "such as". I know, I know, just "link" is vague, because it could be a mailto link or something else. But it's not worth the verbage. * Change "applications effectively installed" to simply "applications installed". * Capitalize each word in "Custom command options" for the title. * In "<command>"%s"</command>", drop the quotes. I'd probably change command to literal or userinput as well, since "%s" isn't really a command. * "Enter the <command>exec</command> option to use with the command." That doesn't seem right. "exec" certainly isn't a command, and it's not really any literal value that should get special treatment. This probably needs to be a few sentences that explain what the hell it is. "Most terminal applications have an option that cause them to treat the remaining command line options as commands to run. This is used when blah blah blah..." Not sure exactly, but this is certainly non-obvious to normal people, so should get a decent explanation. Note that I haven't applied the patch and looked at it in Yelp. Nor have I compared the content to make sure it's actually in like with 2.24. All I did was look over the patch file for things that stood out to me. Thanks Claude.
Created attachment 120405 [details] [review] Revised patch for preferred applications Here's the revised patch. Check particularly the wording of the "Exec Flag" section.
FYI I've checked on current Ubuntu Intrepid beta and I don't see any change related to this section.
Patch looks good to me.
Committed in trunk.