Menu

#104 TextTools: Surrounding white space

open
None
5
2008-11-03
2008-11-02
admirau
No

Please consider adding option if to add space to range comment:
/* comment */
instead:
/*comment*/

It is very useful, for example in CSS mode.

Discussion

  • Ian Lewis

    Ian Lewis - 2008-11-03

    Is there a reason to make this an option and not add a space all the time kind of like how the line comment works?

     
  • Ian Lewis

    Ian Lewis - 2008-11-03
    • assigned_to: nobody --> ian_lewis
     
  • admirau

    admirau - 2008-11-03

    Space might be added by default.
    I think space is ok for all just one line comments (range or line).
    But for example in DocBlock comments, space is not needed.

    And why author didn't make it default?

     
  • Ian Lewis

    Ian Lewis - 2008-11-03

    Is there any situations where having a space like that would be bad? I'm thinking it would be better to just make it have a space in there no matter what.

    As for why it wasn't there to begin with, it's anyone's guess. The important thing is it isn't and what we should do to fix it.

     
  • Robert Schwenn

    Robert Schwenn - 2008-11-03

    The built-in action "range-comment" does insert a space too - it should be good for any cases.

    BTW: Is there a good reason to have built-in comment actions that only do 50% of the job (only insert), so that 100% actions (toggle) are needed in a plugin?
    ;-)

     

Log in to post a comment.