On Wednesday 13 October 2010, Geert Janssens wrote: > On Tuesday 12 October 2010, Cristian Marchi wrote: > > I attach the diff for the "ch_basics.xml" guide file. I've reviewed the > > entire document and I've corrected some typo. The majority of works went > > in updating the "Preferences" chapter. > > I've also added, where applicable, the structure for menu, submenu and > > menuitem (e.g. > > <menuchoice><guimenu>File</guimenu><guisubmenu>Import...</guisubmenu><gui > >me nuitem>Import QIF</guimenuitem></menuchoice>). > > > > My English is not perfect so it's (very) possible that I made some > > mistake. > > > > If I don't get any comments, in the week-end I'll commit the changes. > > > > Thanks > > Cristian > > While reading Cristian's work on the preferences dialog, I found some > issues with the dialog itself that don't have anything to do with the > guide. Before I go fix these issues, I'd like some feedback from others: > > * Summarybar content. This is currently an option in the Accounting Period > tab. I find it very confusing in there. The two options that can be set > here have no relation to the accounting period whatsoever. So I propose to > move this option to another tab. The Miscellaneous options tab already has > some Summary bar related parameters (position). Perhaps it's better to > group them there ? > > * Business -> Open in new window and Accumulate splits > These are valid for both invoices and bills and should be positioned as > such. The current ordering seems to suggest they are only used for > invoices and not bills. I propose to move them before "Invoice", > optionally with another section title, like "General" (which incidentally > exists already in the po file, so that this change wouldn't break string > freeze) > > * Business -> Number of rows and Register Defaults -> Number of rows > These parameters don't seem to have any visual effect. Wouldn't it be > better to remove these options then ? We can still keep the gconf > parameters though as they are read by the code. Or we can also remove the > code regarding these options. > > * General -> Show splash screen > IMO this is an option just to have an option. I can think of no mature > application program that has a preference setting to enable or disable the > splash screen. Either an application has a splash screen and it will be > displayed, or it doesn't have one. I imagine there were some issues with > splash screens in the past and hence it was made optional (I found some > references in the code that there could be issues on OS X, but they have > been worked around programmatically). I vote to remove the option and > always show the splash screen. You can always click it away if it's in > your way. > > I realize Cristian that these changes would mean you'd have to redo some of > your screenshots and move some more text around... > I have just committed r19733 with the proposed changes above (except that the Summary bar content is still in the Accounting Period section, just moved down). For the documentation, this means 4 panels will need new images and some options have been removed/reordered. See the commit message for more details.
Geert _______________________________________________ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
