Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Cannot open a less file with the following content. #6057

Closed
RaymondLim opened this issue Nov 20, 2013 · 6 comments
Closed

Cannot open a less file with the following content. #6057

RaymondLim opened this issue Nov 20, 2013 · 6 comments
Assignees

Comments

@RaymondLim
Copy link
Contributor

  1. Create a new file and name it test.less.
  2. Paste the following content to the new file and save it.
aside li {
    overflow: hidden;
    margin-bottom: 1px;
}

li.categoria {
    padding: 0 0 5px 0;
    border-bottom: 1px solid #09b7d6;
}

li:nth-last-child(-n+4) {
    float: left;
}

input.toggle {
    display: none;
}

input.toggle + label {
    cursor: pointer;
    display: block;
    text-indent: 28px;
    vertical-align: middle;
    line-height: 42px;
}

input.toggle:checked + label {
    background: #5b6669;
}
  1. Relaunch Brackets and try to select test.less in the project tree.

Result: You can't. If it is already in the working set, it shows up nothing in the editor. And if you start selecting another file in the project tree, either the file content shows up in the editor somewhere in the middle of the page. ie. line number 1 starts somewhere in the middle of the window and leaving some spaces above. And if you try to click test.less in the project tree, the selection highlight goes to the file above test.less.

@RaymondLim
Copy link
Contributor Author

@peterflynn narrow it down to the following minimal css code.

div {
}

And it reproduces in sprint 34, but not in sprint 33. So it should be a show-stopper for EC release.

@ghost ghost assigned njx Nov 20, 2013
@njx
Copy link
Contributor

njx commented Nov 20, 2013

I'll look into this for now.

@njx
Copy link
Contributor

njx commented Nov 20, 2013

Looks like this is a bug in the LESS mode which is fixed in the current upstream master, probably by codemirror/codemirror5@464dd7b and codemirror/codemirror5@8c42f65. It's too late for the current Brackets sprint, but for EC we'll need to decide whether to try to roll less.js backward, roll it forward, or (probably unlikely at this point) take the latest CodeMirror.

@RaymondLim
Copy link
Contributor Author

Confirmed fixed.

@FezVrasta
Copy link
Contributor

I've updated to the S35 experimental and I've faced the problem again. It happens when I write a dot in some .less file...

@FezVrasta
Copy link
Contributor

Tried with the less.js from the Sprint 33 and everything seems to work fine. Will keep you updated.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants