Example:

---start---

This is a typical header comment block

# class MyClass(object): pass ---end--

Selecting the lines between start and end, and then attempting to comment will end up yielding:

---start---

This is a typical header comment block

class MyClass(object): pass ---end--

when I would have expected it to have commented the entire block:

---start---

This is a typical header comment block

#

class MyClass(object):

pass

---end--

I would suggest that for single line comment styles (like #, or // in c), only the first character of the line be considered. Additionally, if any line in the selected block is not commented the entire block should be considered not commented.

Do you think its worth it to try and be "smart" about commenting/uncommenting things? Or should I just have Ctrl+/ be for comments and Ctrl+Shift+/ be to uncomment it. Or is it really this tweak would be enough?

Personally I'd prefer the two different key-combos, but that's mainly because I prefer predictabity even if it makes use slighlty more complex. Usage patterns can become second nature, but having to check to see if a system guessed right this time never goes away. Given that preference, and the feeling that distinct key-combos would be easier than a smart system to implement, I'd lean that direction. I think those tweaks would fix the issues with python though. I'm not sure about other languages.

I agree. I am going to change it to be the two key-combos then. It should not take long to do...I will post back when it is updated.

Ok. I have the plugin updated and available as a download. I decided to use the Ctrl+/ to comment the text and Ctrl+Backslash to uncomment it.

The reason I choose the backslash key was because on a US querty keyboard the question mark key is on the same key as the slash and Ctrl+Shift+? is just strange.

When I was updating the docs I realized I could have used Alt+/ and the uncomment. Which way do you like better? Or do you have a better suggestion?

I think the the ctrl+/ and ctrl+\ shortcuts match what eclipse uses. That's probably a good choice since it will help ease converts.

Great! It is ready then.