Re: [O] More flyspell-overlays removed
Re-dl'ed and works fine. Sorry about that. On Tue, Aug 7, 2012 at 7:18 PM, Bastien b...@gnu.org wrote: Hi Jeffrey, Jeffrey Spencer jeffspenc...@gmail.com writes: Just gave this a go and seems to work well but still forgot to add I believe: TYP_TODO and ATTR_LATEX The two words above are already skipped for me. -- Bastien
Re: [O] More flyspell-overlays removed
Hi Jeffrey, Jeffrey Spencer jeffspenc...@gmail.com writes: Just gave this a go and seems to work well but still forgot to add I believe: TYP_TODO and ATTR_LATEX The two words above are already skipped for me. -- Bastien
Re: [O] More flyspell-overlays removed
Just gave this a go and seems to work well but still forgot to add I believe: TYP_TODO and ATTR_LATEX Cheers, Jeff On Wed, Aug 1, 2012 at 1:32 AM, Bastien b...@gnu.org wrote: Sebastien Vauban wxhgmqzgwmuf-genee64ty+gs+fvcfc7...@public.gmane.org writes: I would allow fly-prog-mode in the listins, as one normally does for plain code, no? This is indeed the case right now. -- Bastien
Re: [O] More flyspell-overlays removed
I don't have any problem with natbib cite commands including \ref, \cite, \citet, \citep, etc Maybe your version of flyspell?? But it should act exactly as it does in auctex or your latex mode of emacs. I would check in there if it works properly because should parse the same way if not maybe debug from there. Cheers, Jeff On Mon, Jul 30, 2012 at 9:44 PM, Bjarte Johansen bjo...@student.uib.nowrote: On 30 Jul, 2012, at 13:03 , Bastien b...@gnu.org wrote: I've pushed a fix which should let flyspell ignore more commonly used Org keywords. Please test it. This works great. You forgot #+LATEX_CLASS_OPTIONS: though. On 30 Jul, 2012, at 13:13 , Jeffrey Spencer jeffspenc...@gmail.com wrote: Will give it a test later in the week and let you know. Also you can add this hook to make it act like the fly-spell mode in auctex (if familiar with that) which skips most tex based commands (trips up though if you have only one $ because assumes another $ sign later so won't check spelling in that block. I would just do \$ if you need a single dollar sign. This is the only really limitation I have found to adding this that I have noticed thus far. (add-hook 'org-mode-hook (lambda () (setq ispell-parser 'tex))) I tried this and it does work for most things, but for some reason it doesn't like the natbib \cite commands.
Re: [O] More flyspell-overlays removed
I was also thinking about this recently but hadn't gotten as far as writing a patch. I was thinking some tags you know you want to remove the fly-spell overlays but for example caption you might or might not want the flyspell overlays removed. If there was a variable that could be set to choose some that might or might not want the overlays to remove. Not sure if this is possible but what I was originally looking at doing but hadn't gotten time. Cheers, Jeff On Sun, Jul 29, 2012 at 8:38 AM, Bjarte Johansen bjo...@student.uib.nowrote: On 28 Jul, 2012, at 11:27 , Bastien b...@gnu.org wrote: I cannot apply it because it adds keywords at the wrong place. For example you cannot add startup: in the first (cond ((...))) because startup: is not a backend specific content. And some other discrepencies. OK, I understand. I should have spent some more time trying to understand the code. I'd welcome a patch for removing more flyspell overlays, but it has to be rewritten. I'm not using flyspell so someone else will have to do roll his sleeves. I'd be willing to put in the effort to make a proper patch, but I need some help to understand what is going on in this function. It is not obvious to me. The places flyspell-overlays needs to be removed are in the startup, options, latex_header. Some of structures that might be used in latex like lstlisting and verbatim where the text also should not have flyspell-overlays. There are also some other places like label, caption and attr.
Re: [O] More flyspell-overlays removed
Hi Bjarte, Bjarte Johansen bjo...@student.uib.no writes: I made a patch to remove some more flyspell-overlays in #-blocks. The reason for no : in the latex_header is that for some reason the : does not get captured in dc1. flyspell is also removed for the full verbatim, lstlisting and src blocks. I hope you guys can use the patch. I cannot apply it because it adds keywords at the wrong place. For example you cannot add startup: in the first (cond ((...))) because startup: is not a backend specific content. And some other discrepencies. I'd welcome a patch for removing more flyspell overlays, but it has to be rewritten. I'm not using flyspell so someone else will have to do roll his sleeves. Thanks anyway for the patch! -- Bastien