misspelling-reporter icon indicating copy to clipboard operation
misspelling-reporter copied to clipboard

Add Context to the email

Open meyerweb opened this issue 11 years ago • 7 comments

It would be really useful to show the words that surround the reported word. So if I report "claer", the email would look something like this:

Post: http://example.com/post-with-error/

Misspelling: claer

Context: has eyes, it should be claer that this issue affects us

meyerweb avatar May 31 '13 03:05 meyerweb

Love the idea - just thinking out loud, would it be more or less useful to have something like the following? :

Context: Paragraph 3, Line 4

My thought there is that sometimes in blog posts, you'll repeat a thought multiple times for different emphatic reasons, so just having the immediately surrounding context could sometimes be less than helpful.

JustinSainton avatar May 31 '13 20:05 JustinSainton

Agreed! I mean, the best solution of all would be a link straight to the fragment of text in question, but at this stage I think showing contextual text and the paragraph/sentence numbers would be a great addition. Or even just the paragraph number.

meyerweb avatar May 31 '13 21:05 meyerweb

Should this also ensure the entire word was selected just in case the reporter missed a letter on one end? Is that feasible?

blobaugh avatar May 31 '13 21:05 blobaugh

I can imagine the URL in the email linking to http://domain.tld/post-with-error/#mispelled%20werds and that causing any case of mispelled werds to be highlighted on the page, with scroll oriented around the first instance.

jeremyfelt avatar May 31 '13 21:05 jeremyfelt

Oh yeah! Go Jeremy! That is a sick idea. The misspelled words are already stored in the db with the post tagged. I bet that could be pulled off easily enough.

Now that I think about it i should have used post_parent instead of a meta field for that....

blobaugh avatar May 31 '13 21:05 blobaugh

No DB access needed. JS can probably hunt well enough for the matching content based on the URL.

jeremyfelt avatar May 31 '13 22:05 jeremyfelt

Potentially. I updated the DB anyways. So now if you wanted something like http://domain.tld/post-with-error/?missr=1 you could do it :D

blobaugh avatar May 31 '13 22:05 blobaugh