#227 table - strikethrough interference in 0.12.0

Verified
closed-works-for-me
PVHL (Paul)
Syntax (35)
9
2011-03-12
2010-08-24
gkrautmann
No

Hi all,
first of all compliments for the great work !!!

I just upgraded to 0.12.0 from 0.11.9 and found a problem with the new table code.
I am using strikthrough in a table cell and display with Firefox 3.0 and 3.6.
At the end of the report is the relevant code.
At least on my browser the strikethrough interferes with the cell ending ||
This does not happen when a non-blank character follows the strikethrough.
This also did not happen in 0.11.9 .

Cheers and thanks again,
Gerd

{|
| IFM # || Webb # || WMO # || ArgosID# || ||
| [[IFM02]] || #36 job #211 || || 06847 || [[Ballasting IFM02]] || [[Deployments IFM02]]
|-
| [[IFM03]] || #68 job #1132 || 18904 || 06841 || [[Ballasting IFM03]] || [[Deployments IFM03]]
|-
| [[IFM04]] || #74 job #919 || <s>test</s> <s>68905</s> || 06843 || [[Ballasting IFM04]] || [[Deployments IFM04]]
|-
| [[IFM05]] || #87 job #1291 || 18905 || 03535 || [[Ballasting IFM05]] || [[Deployments IFM05]]
|-
| [[IFM06]] || #161 job #1621 || 18907 || 96040 || [[Ballasting IFM06]] || [[Deployments IFM06]]
|-
| [[IFM07]] || #162 job #1621 || 18908 || 96041 || [[Ballasting IFM07]] || [[Deployments IFM07]]
|-
| [[IFM08]] || #163 job #1621 || 18909 || 96042 || [[Ballasting IFM08]] || [[Deployments IFM08]]
|-
| [[IFM09]] || #164 job #1621 || 18910 || 96043 || [[Ballasting IFM09]] || [[Deployments IFM09]]
|-
| [[IFM10]] || #165 job #1621 || 18911 || 96044 || [[Ballasting IFM10]] || [[Deployments IFM10]]
|-
| [[IFM11]] || #168 job #1621 || 18912 || 96045 || [[Ballasting IFM11]] || [[Deployments IFM11]]
|-
|}

Discussion

  • milton calnek
    milton calnek
    2011-01-18

    I'd just like to echo the compliments. This is a great piece of software.
    Thanks.

    And, I too have this issue.
    I wanted to "strike-out" text in a table and this did not work as expected.

    {|
    | Month || Retire || Aquire
    | Jan || <s>caster02</s> ||
    | || caster03 || caster06

    There is no cell after caster02 (which is struck out), but the || is now in the 2nd cell.

     
  • milton calnek
    milton calnek
    2011-01-18

    Sorry, my versions are:
    Linux (centos 5.5, Firefox 3.6.13, woas 0.12.)

     
  • PVHL (Paul)
    PVHL (Paul)
    2011-02-25

    What a lovely bug! It only happens with </s>, not </i> or </b>, and it had nothing to do with the table code!

    Not a fix, but if you go to the Special::Options page (from Special::Advanced for instance) and deselect "Enhanced tables syntax" you won't have this issue (but you will lose all the great new features of the enhanced tables!).

    I just fixed this in my private copy, but a new bug-fix version is coming out in the next week or two and I'd prefer to put it in there as some of the code is changing quite a bit. If you would like to fix your own copy then find the code below (it is in parser.js of you are not using the single-file version) :

    // put away big enough HTML tags sequences (with attributes)
    P.body = P.body.replace(/(<\/?\w+[^>]*>[ \t]*)+/g, function (tag) {
    r = woas.parser.place_holder(snippets.length);
    snippets.push(tag);
    return r;
    })

    and replace it with:

    P.body = P.body.replace(/([ \t]*)((?:[ \t]*<\/?\w+.*?>)+)/g, function (str, ws, tags) {
    if (typeof ws === 'undefined') {
    ws = '';
    }
    r = ws + woas.parser.place_holder(snippets.length);
    snippets.push(tags);
    return r;
    })

    This should fix the problem until the new version is released. (There are other issues this doesn't fix, but the new bug release will fix them).

    Cheers, Paul.

     
  • PVHL (Paul)
    PVHL (Paul)
    2011-02-25

    • labels: --> Syntax
    • priority: 5 --> 9
    • assigned_to: nobody --> pvhl
    • milestone: --> Verified
    • status: open --> open-works-for-me
     
  • PVHL (Paul)
    PVHL (Paul)
    2011-02-26

    • status: open-works-for-me --> pending-works-for-me
     
  • This Tracker item was closed automatically by the system. It was
    previously set to a Pending status, and the original submitter
    did not respond within 14 days (the time period specified by
    the administrator of this Tracker).

     
    • status: pending-works-for-me --> closed-works-for-me