Don't onebox in the middle of my code tags!


  • Discourse touched me in a no-no place

    Furthermore, pre/code seems to treat CRLF as two line breaks. Which is fucking dumb.

    ====Executing command 'npm install'====

    npm WARN package.json nodebb-plugin-soundpack-default@0.1.6 No README data
    npm WARN package.json passport@0.3.2 No README data
    npm WARN git config --get remote.origin.url returned wrong result (git://github.com/wzrdtales/simple-recaptcha)
    npm ERR! git clone git://github.com/wzrdtales/simple-recaptcha
    npm ERR! Error: spawn ENOENT
    npm ERR! at errnoException (child_process.js:1011:11)
    npm ERR! at Process.ChildProcess._handle.onexit (child_process.js:802:34)
    npm ERR! If you need help, you may report this entire log,
    npm ERR! including the npm and node versions, at:
    npm ERR! http://github.com/npm/npm/issues
    npm ERR! System Windows_NT 6.1.7601
    npm ERR! command "C:\Program Files\nodejs\\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install"
    npm ERR! cwd C:\Users\Weng\Documents\GitHub\NodeBB-SQL
    npm ERR! node -v v0.10.35
    npm ERR! npm -v 1.4.28
    npm ERR! syscall spawn
    npm ERR! code ENOENT
    npm ERR! errno ENOENT
    npm

    ====npm command completed with exit code 1====

    or

    ====Executing command 'npm install'====
    
    npm WARN package.json nodebb-plugin-soundpack-default@0.1.6 No README data
    npm WARN package.json passport@0.3.2 No README data
    npm WARN `git config --get remote.origin.url` returned wrong result (git://github.com/wzrdtales/simple-recaptcha) 
    npm ERR! git clone git://github.com/wzrdtales/simple-recaptcha 
    npm ERR! Error: spawn ENOENT
    npm ERR!     at errnoException (child_process.js:1011:11)
    npm ERR!     at Process.ChildProcess._handle.onexit (child_process.js:802:34)
    npm ERR! If you need help, you may report this *entire* log,
    npm ERR! including the npm and node versions, at:
    npm ERR!     <http://github.com/npm/npm/issues>
    npm ERR! System Windows_NT 6.1.7601
    npm ERR! command "C:\\Program Files\\nodejs\\\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install"
    npm ERR! cwd C:\Users\Weng\Documents\GitHub\NodeBB-SQL
    npm ERR! node -v v0.10.35
    npm ERR! npm -v 1.4.28
    npm ERR! syscall spawn
    npm ERR! code ENOENT
    npm ERR! errno ENOENT
    npm
    
    ====npm command completed with exit code 1====
    
    


  • Does it onebox inside markdown code blocks?

    ====Executing command 'npm install'====
    
    npm WARN package.json nodebb-plugin-soundpack-default@0.1.6 No README data
    npm WARN package.json passport@0.3.2 No README data
    npm WARN `git config --get remote.origin.url` returned wrong result (git://github.com/wzrdtales/simple-recaptcha) 
    npm ERR! git clone git://github.com/wzrdtales/simple-recaptcha 
    npm ERR! Error: spawn ENOENT
    npm ERR!     at errnoException (child_process.js:1011:11)
    npm ERR!     at Process.ChildProcess._handle.onexit (child_process.js:802:34)
    npm ERR! If you need help, you may report this *entire* log,
    npm ERR! including the npm and node versions, at:
    npm ERR!     <http://github.com/npm/npm/issues>
    npm ERR! System Windows_NT 6.1.7601
    npm ERR! command "C:\\Program Files\\nodejs\\\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install"
    npm ERR! cwd C:\Users\Weng\Documents\GitHub\NodeBB-SQL
    npm ERR! node -v v0.10.35
    npm ERR! npm -v 1.4.28
    npm ERR! syscall spawn
    npm ERR! code ENOENT
    npm ERR! errno ENOENT
    npm
    
    ====npm command completed with exit code 1====
    

    Edit: Looks like it doesn't.


  • Discourse touched me in a no-no place

    pre also oneboxing..

    npm ERR! including the npm and node versions, at:
    npm ERR!     <http://github.com/npm/npm/issues>
    npm ERR! System Windows_NT 6.1.7601
    


  • <pre> doesn't stop parsing of its contents. Not a bug.


  • Discourse touched me in a no-no place

    @ben_lubar said in Don't onebox in the middle of my code tags!:

    <pre> doesn't stop parsing of its contents. Not a bug.

    With all due respect, that's bollocks. The whole point of pre is to not interpret its contents. Things like not interpreting <. Or not collapsing multiple spaces. Or not inserting linefeeds. Or - in this instance - not expanding oneboxes.

    Totally a bug. Or, perhaps, less aggressively - not the expected behaviour. Perhaps we could have a poll on the matter?

    Talking of which, having to re-add the backticks into your quote, to stop <pre> not appearing in the quote, is another bug.



  • @PJH said in Don't onebox in the middle of my code tags!:

    Things like not interpreting <

    Except that it does interpret <. pre is not CDATA.


  • Discourse touched me in a no-no place

    @ben_lubar said in Don't onebox in the middle of my code tags!:

    Except that it does interpret <. pre is not CDATA.

    Oops. I done fucked up....



  • @Weng This one box plugin is broken buggy shit.

    Why did "we" (by which I mean "Ben L, unilaterally") turn it back on? Until it's about 40,000 times better than it is today, trying to deal with this shit is a waste of time.



  • @blakeyrat said in Don't onebox in the middle of my code tags!:

    @Weng This one box plugin is broken buggy shit.
    Why did "we" (by which I mean "Ben L, unilaterally") turn it back on? Until it's about 40,000 times better than it is today, trying to deal with this shit is a waste of time.

    Hmm, slight convenience for me vs DDOS that's not really my problem.

    Keep it on, ben.


  • Discourse touched me in a no-no place

    @ben_lubar ... Can we have CDATA? With syntax highlighting?



  • @Weng you can either put four spaces before each line:

    def foo():
        return "https://www.google.com/"
    

    or put three or more backticks (```) on the lines immediately before and after the code block:

    def foo():
        return "https://www.google.com/"
    

  • Fake News

    @ben_lubar

    Is your final answer "you're doing it wrong"?


  • Discourse touched me in a no-no place

    I thought "No markupdownupdownleftrightleftrighbabastart markup soup" was one of our key functional requirements. Or was that requirement discarded as nothing on the market provides such functionality anymore?



  • Can we just fix the onebox plugin to only onebox url patterns that match the whole line?



  • This should be easy to settle, let's all vote here

    [poll]

    • Keep onebox plugin enabled
    • Disable it until it gets better
      [/poll]

    Oh, right, no polls anymore...



  • @PleegWat To be on the safe side, I'd make it require some explicit tag or syntax, i.e. just write something like [embed]http://myurl.invalid/thing.html[/embed] or {http://myurl.invalid/thing.html}.


Log in to reply
 

Looks like your connection to What the Daily WTF? was lost, please wait while we try to reconnect.