Re: Fwd: Smart quotes patch - committed.

W00t! That's incredible. Thanks for putting it in, whether it goes
in 2.6.0 or not.

The reason smart quotes has its own tab is twofold:
1) Not too long ago, someone removed the GUI which used to exist to
enable smart quotes and it was too much of a bother to find it.
2) I was hoping to add a few preferences like the abiltiy to choose
the quote characters, turn it off for single quotes (as Word and OOo
do) and possibly set up a way to force an apostrophe.

None of these options mean a lot to me personally, since I usually
only type in English and could probably hack my keyboard layout in
Linux to get forced-apostrophes if I need them.

Martin - From your email, it sounds like you come down on the side of
using the spell-check "Set Language" option to set the smart quote
characters. I have already stated my disagreement, but I have no
problem if the concensus is against me.

If the Germans, et al, who have multiple quoting styles for their
locale have a problem with using the "Set Language" option, now would
be a good time to speak up.

Thanks again,
Bobby

On 3/15/08, Martin Sevior <msevior@gmail.com> wrote:
>
> Committed! Excellent work Bobby. works like a charm with AbiCollab
> including undo/redo. Thank you very much. we've wanted to get this
> feature right for a long time.
>
> It may be worth backporting this to 2.6.
>
> There are a few extra features improvements worth discussing though.
>
> 1. Implement Tomas's suggestion to allow the curly quote to be translated.
> 2. The feature has it's own panel in the Unix options dialog. It looks
> pretty lonely up there.
>
> Does it make sense to move the check box to another, pre-existing panel?
>
> Anyway, congratulations once again. None of us has been able to get
> this to work.
>
> Cheers
>
> Martin
>
> --------------------------------------------
> Thanks for the catch. I forgot to turn off smart quote replacement in
> cmdRedo() as well as cmdUndo. Your test case works now. Please let
> me know if you find any other similar problems.
>
> I zipped the patch this time to make it a little smaller.
>
> Bobby
>
> On Wed, Mar 12, 2008 at 6:02 PM, Tomas Frydrych <tf@o-hand.com> wrote:
> >
> > Bobby Weinmann wrote:
> > > Here is my second attempt at a patch. I have done some preliminary
> > > tests and it seems to work for English. The new thing is that UNDO
> > > WORKS! (at least for me)
> >
> > Thanks for working on this; unfortunately the undo/redo seems to be
> > still broken -- try the following:
> >
> > 1. Type: This' a test.
> > 2. Undo 3 times,
> > 3. Redo 1 time: assert at px_ChangeHistory.cpp:138 (2x)
> >
> > After that, it is not possible to do Redo again, and I am left with
> >
> > This'
> >
> >
> > > 1) A GUI to set quote characters (I believe the rules will work for
> > > most languages, only the characters would need to change)
> >
> > As I said before, this really should JustWork without any GUI -- we have
> > all the input that is need to decide what quotes to use, just need a
> > mechanism to translate sqQUOTEls, sqQUOTErs, sqQUOTEld, and sqQUOTErd to
> > the appropriate character based on the current language setting
> > (obtainable with fl_BlockLayout::getProperty()) before calling
> > cmdCharInsert().
> >
> >
> > > 2) An easy way to add an apostrophe at the beginning of a word ('Tis
> > > or 'em) - perhaps you could use the backtick?
> >
> > Probably not the backtick unmodified, but perhaps something like
> > Ctrl+backtick, although that might become problematic with different kbd
> > layouts.
> >
> > Tomas
> >
> >
>
Received on Sun Mar 16 14:55:16 2008