Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Expanding multiple elements #8

Open
dburles opened this Issue Jul 1, 2010 · 3 comments

Comments

Projects
None yet
3 participants

dburles commented Jul 1, 2010

The files i'm working with the newlines are CR LF (windows) and i'm using tabs for indentation

when expanding any element over multiple lines it's adding extra newlines and losing the indentation after the first element (as the tabs are before the extra newline) i.e:

(tab)(tab)div*3

then generates exactly:

(tab)(tab)div/divCRLF

(tab)(tab)CRLF

div/divCRLF

(tab)(tab)CRLF

div/divCRLF

CRLF

had to modify the div's github doesn't seem to like them when written as proper tags in that format

dburles commented Jul 2, 2010

quick update, in some instances it seems to work correctly and other times not, I can't quite figure out whats causing it to mess up..

Owner

mikecrittenden commented Jul 2, 2010

So it seems like the only problem is that it's adding extra line breaks...the indends are there, they're just not REALLY there because there's a line break between the indents and the code on each line. So if we can get the line breaks problem worked out we should be good.

I'm not sure that I'm going to be much help with this. I don't have a Windows box to test on and I can't reproduce on Linux. With your last update you said that sometimes it works fine, so if you're able to make a scenario where it always doesn't work, then let me know and I'll try and get my hands on a Windows box to test it out.

I can repro in Linux, Ubuntu 10.10 Maverick

I tried the following abbreviation:
ul.column-holder.clearfix>li.column_3>h4+img_3

This adds the line breaks, and if I'm expanding this a few indents in, it will add the line break after the parent indent level.

Now if I break these into two different abbreviations:
ul.column-holder.clearfix
li.column_3>h4+img_3
and expand the second abbreviation inside the expanded first abbreviation, it works as expected.

Let me know if you need a guinea pig, or if I ever get around to learning some Python, I can fork and see what I can do. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment