Use word boundaries for word matching?

[ryanpcmcquen]

Is there a reason the analyzeNote function doesn't use word boundaries?

  analyzeNote() {
    var s = this.state.note.content.text;
    s = s.replace(/(^\s*)|(\s*$)/gi,"");//exclude  start and end white-space
    s = s.replace(/[ ]{2,}/gi," ");//2 or more space to 1
    s = s.replace(/\n /,"\n"); // exclude newline with a start spacing
    let wordCount = s.split(' ').length;
    this.setState({wordCount: wordCount});
  }

Would become:

  analyzeNote() {
    // Count all word boundaries in the note text and divide by 2
    // to get the word count:
    let wordCount = this.state.note.content.text.match(/\b/gm).length / 2;
    this.setState({wordCount: wordCount});
  }

If this is OK, I'll set up PRs to change it here and in the documentation.

[ryanpcmcquen]

Example: https://repl.it/@ryanpcmcquen/BonyLiquidBase

[moughxyz]

I suppose that could work. However the Action Bar extension is located here: https://github.com/standardnotes/action-bar

This repo is just an example project.

[ryanpcmcquen]

OK, I'll fix it in all 3 places.

[ryanpcmcquen]

@mobitar, it's been PR'ed in all three places!

[ryanpcmcquen]

https://github.com/sn-extensions/react-blank-slate/pull/18 https://github.com/standardnotes/action-bar/pull/19 https://github.com/standardnotes/docs/pull/13