I just stripped this down, I noticed it in my own journal where I use normal account names but happened to have a zero in some auto-generated postings. I agree that this is a corner case and very low priority, as I can just modify my journal. Just wanted to let you know.
On Wed, Sep 10, 2014 at 3:51 PM, Craig Earls <[email protected]> wrote: > I will look into this, but I think single character account names and > single digit amounts are not realistic. > > On Wed, Sep 10, 2014 at 6:48 AM, Johann Klähn <[email protected]> wrote: > > Nice work Craig! > > One thing I noticed, though: In the following transaction the account > name > > in the second line (B) is highlighted using > > `ledger-font-posting-amount-face`. > > And the 0 in the first posting is highlighted as if it was part of the > > account name. > > > > 2014/03/11 Whatever > > A 0 > > B 6,75€ > > C -6,75€ > > > > This seems to happen for every posting that follows one ending with a > zero > > (i.e. for permutations of the example above). > > > > On Tue, Sep 9, 2014 at 6:43 AM, Craig Earls <[email protected]> wrote: > >> > >> Indentation is totally independent of font-lock. There is a bug noted > >> already. I just haven't gotten to it. > >> > >> > >> On Monday, September 8, 2014, Andrew Schwartzmeyer > >> <[email protected]> wrote: > >>> > >>> > >>> On Sep 8, 2014, at 8:49 PM, Craig Earls <[email protected]> wrote: > >>> > >>> > I have spent the last month overhauling ledger fontification. I gave > >>> > up on using the standard REGEX based methods that most font lockers > >>> > use. Instead I wrote a few functions that actually understand the > >>> > semantics of a ledger xact then had jit-lock and font-lock call > those. > >>> > Overall there is much more flexibility and it seems to be plenty > >>> > fast. The regexes I do use are much simpler. I tested with a file > >>> > that had 2,613,072 postings and it was lighting fast. Comments after > >>> > amounts are now properly recognized and you can fontify an entire > xact > >>> > based on its overall status (my favorite part, that font-lock just > >>> > really couldn't do). > >>> > >>> Awesome! Thanks so much for your work man. > >>> > >>> > For those of you who use lots of different directives there are now > >>> > separate faces possible for all valid directives (or just use one > face > >>> > for all directives) > >>> > > >>> > Please check it out and report any bugs you find. I have banged on > it > >>> > for a while and it has been stable and well behaved. Of course you > >>> > will find problems. > >>> > >>> Gotta say for the first thirty seconds I though it was totally broken > >>> when everything was white, made sense once I scrolled down (white for > >>> cleared transactions, green for budget, normal for uncleared, at least > with > >>> my theme). > >>> > >>> So far it's recognizing (highlighting properly): assertions, lot value > / > >>> posting cost / amount expressions, and posting notes. Sweet! > >>> > >>> However, it unfortunately doesn't smartly indent any of the above > except > >>> plain currency values. What would go into fixing that? I can take a > look. > >>> > >>> > -- > >>> > Craig, Corona De Tucson, AZ > >>> > enderw88.wordpress.com > >>> > > >>> > -- > >>> > > >>> > --- > >>> > You received this message because you are subscribed to the Google > >>> > Groups "Ledger" group. > >>> > To unsubscribe from this group and stop receiving emails from it, > send > >>> > an email to [email protected]. > >>> > For more options, visit https://groups.google.com/d/optout. > >>> > >>> -- > >>> > >>> --- > >>> You received this message because you are subscribed to the Google > Groups > >>> "Ledger" group. > >>> To unsubscribe from this group and stop receiving emails from it, send > an > >>> email to [email protected]. > >>> For more options, visit https://groups.google.com/d/optout. > >> > >> > >> > >> -- > >> Craig, Corona De Tucson, AZ > >> enderw88.wordpress.com > >> > >> -- > >> > >> --- > >> You received this message because you are subscribed to the Google > Groups > >> "Ledger" group. > >> To unsubscribe from this group and stop receiving emails from it, send > an > >> email to [email protected]. > >> For more options, visit https://groups.google.com/d/optout. > > > > > > -- > > > > --- > > You received this message because you are subscribed to the Google Groups > > "Ledger" group. > > To unsubscribe from this group and stop receiving emails from it, send an > > email to [email protected]. > > For more options, visit https://groups.google.com/d/optout. > > > > -- > Craig, Corona De Tucson, AZ > enderw88.wordpress.com > > -- > > --- > You received this message because you are subscribed to the Google Groups > "Ledger" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > For more options, visit https://groups.google.com/d/optout. > -- --- You received this message because you are subscribed to the Google Groups "Ledger" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
