duydao / Text-Pastry

Extend the power of multiple selections in Sublime Text. Modify selections, insert numeric sequences, incremental numbers, generate uuids, date ranges, insert continuously from a word list and more.
MIT License
832 stars 46 forks source link

Number sequence does not work #19

Closed rahiyer closed 10 years ago

rahiyer commented 10 years ago

Inserting number sequence does not work giving the following error when checked in the sublime console:

Traceback (most recent call last):
  File "/Applications/Sublime Text.app/Contents/MacOS/sublime_plugin.py", line 543, in run_
    return self.run(edit, **args)
TypeError: run() got an unexpected keyword argument 'padding'

I am using Sublime Text 3, Build 3059, with Text pastry installed using Package Control.

Other commands like Paste work fine.

duydao commented 10 years ago

Could you describe the steps you did to reproduce this error?

rahiyer commented 10 years ago
  1. Select 4 lines in a new file
  2. From command palette, select "Text Pastry: From 1 to X" (same error for '0 to X').
  3. Check console using "ctrl + `"
duydao commented 10 years ago

Unfortunately, I'm unable to reproduce this error. It works on mac/windows as expected (without error). What version of TP are you using? You can find the Version under command palette -> list packages -> text pastry.

Any chance that you've installed "insert nums"?

rahiyer commented 10 years ago

I am using TP v1.3.7. I did have "Insert nums" installed and the error goes away after uninstalling that package. Closing this issue since no fix is required. Thanks for looking into this and thanks for your effort on TP.

rahiyer commented 10 years ago

BTW, I am getting different results between \i(1,10) and \i(1, 10) (added space between 1 and 10). In fact the command is not tolerant of any space inside the parenthesis. Is that expected behavior or a bug? Do you want me to open another ticket for it?

duydao commented 10 years ago

Thanks for reporting this! The next version of TP won't depend on insert nums anymore, so users will be safe to re-install insert nums if they want to.

About the new issue, this sounds rather like a bug. I'll check it out and will create a ticket if necessary, thanks.