search

boakley / brackets-robotframework

Brackets.io extension for editing robotframework pipe-separated plain text files
20 stars 3 forks source link

Different separator? #15

Closed aaltat closed 9 years ago

aaltat commented 10 years ago

Instead of using pipe as a separator, is it possible to use spaces as a separator?

boakley commented 10 years ago

At present, no. It's something only one other person has asked for so far, so it's not a high priority for me right now.

On Mon, Aug 11, 2014 at 4:35 AM, Tatu Aalto notifications@github.com wrote:

Instead of using pipe as a separator, is it possible to use spaces as a separator?

— Reply to this email directly or view it on GitHub https://github.com/boakley/brackets-robotframework/issues/15.

aaltat commented 10 years ago

If I would like to contribute, could you point me to right direction? I am total newbie to javascript, I am more used to Python.

boakley commented 10 years ago

Are you wanting to contribute so that you can implement the space-separated format, or do you just want to contribute anywhere that needs help?

--bryan

On Tue, Aug 12, 2014 at 2:10 AM, Tatu Aalto notifications@github.com wrote:

If I would like to contribute, could you point me to right direction? I am total newbie to javascript, I am more used to Python.

— Reply to this email directly or view it on GitHub https://github.com/boakley/brackets-robotframework/issues/15#issuecomment-51880726 .

aaltat commented 10 years ago

Implement the space separated format would be choice.

boakley commented 10 years ago

The code that does the syntax highlighting is a file named robot.js. Most or all of the separator-specific code is in that file. The bulk of the code is in a function named base_mode, but the actual syntax tokenizer is in the token function (look for token: function(stream, state) {...).

I've written up my thoughts on what it would take to add support for space and tab separators here: Supporting space and tab delimited files

You'll also need to read the codemirror API documentation -- codemirror is the underlying text edit widget. See Writing CodeMirror Modes

aaltat commented 10 years ago

I did read your post and it does not look too hard. Although I need to get my head around the JavaScript. Only thing comes in my mind with space format is that how many spaces is a separator. I would like to force it to be always same amount and not some random number for each line, that is bigger than two.

boakley commented 10 years ago

Every space-separated format file I've ever seen uses different amounts of spaces, usually to line up columns. The changes to support spaces as separators would have to account for that.

On Wed, Aug 13, 2014 at 2:44 PM, Tatu Aalto notifications@github.com wrote:

I did read your post and it does not look too hard. Although I need to get my head around the JavaScript. Only thing comes in my mind with space format is that how many spaces is a separator. I would like to force it to be always same amount and not some random number for each line, that is bigger than two.

— Reply to this email directly or view it on GitHub https://github.com/boakley/brackets-robotframework/issues/15#issuecomment-52100157 .

boakley commented 9 years ago

Support for space-separated format is available starting with version 1.2 of the extension. Implementation is spread over several changesets, starting with 46635d93ce44f1869751e579206a353e80ccdec4