> Specifically my question is, does the following make sense? > > - aRepeatHeaderCB.Check(aInsOpts.mnRowsToRepeat > 0); > + aRepeatHeaderCB.Check((!bHTMLMode) && (aInsOpts.mnRowsToRepeat > 0)); Assuming bHTMLMode means what it seems to mean, looks fine to me? Pushed to master. --tml