Make the path/uri argument documentation more clear in theme_image_style

It looks like it should accept either, but probably you can't start your path with /sites (should be just sites without the leading slash). I agree that the documentation should be updated. It also applies to Drupal 8 (it's called 'uri' not 'path' in Drupal 8 but it does the same thing).

Needs a patch... possibly a good Novice project?

Something like this maybe?

Thanks! The text in the patch looks good to me. But the addition is making the line longer than 80 characters, so it needs to be wrapped.

Here we go, this has fixed that :)

Not quite! Just make the wrapped line go all the way to the left (indented the same as the rest of the @param section).

By the way, here are the documentation standards:
http://drupal.org/node/1354

hehe, ok. I think I see what you mean with the indentation. How about this??

Um, not quite. Just leave the existing lines as they were, and indent the new ones the same as the old ones. :)

:) Don't worry, I *WILL* get this....

I think I see the issue with the second patch (which was closer) now that I didn't notice before I posted it!

There you go! :) Don't worry, I'm (hopefully) patient with new contributors. I'll get this committed to 8.x shortly. We will also need a patch for 7.x, which is the same except for the argument name being different (I think?).

Yay! good news. I am definitely new to the core contributions category! been doing all of my work in the contrib space. I will look at a D7 patch shortly...

Yep, I think the only difference is that the key is called 'path' in D7 and 'uri' in D8. Here is a re-rolled patch for D7. Will this fail until issue version tag is changed to D7?

Yep. You can name the patch with "do-not-test" in the name (e.g. 1444650-5_d7-do-not-test.patch) to make testbot skip it.

Ah, clever :) I did not know that.

Committed to 8.x and 7.x. Thanks!