adding lineEndings UI in statusbar for documents

[core-ai-bot]

Issue by zaggino
Monday Mar 06, 2017 at 22:50 GMT
Originally opened as adobe/brackets#13152

---

continue from adobe/brackets#10346

fixes #10346
fixes #10106
fixes #10594

---

zaggino included the following code: https://github.com/adobe/brackets/pull/13152/commits

[core-ai-bot]

Comment by zaggino
Tuesday Mar 07, 2017 at 00:05 GMT

---

@petetnt :-)

[core-ai-bot]

Comment by zaggino
Tuesday Mar 07, 2017 at 00:06 GMT

---

I was looking at this old PR, tried to polish it up a bit to see what we can do. It works fine I think but undo-redo doesn't work and won't work until we can add custom history events to CodeMirror. I'd have to check CodeMirror's code to see if that would be possible.

[core-ai-bot]

Comment by zaggino
Tuesday Mar 07, 2017 at 00:19 GMT

---

Also, partial solution to the problem is better than nothing at all. With this we can at least change line endings in the UI. In my opinion it's quite questionable how many users would report that this change is not part of the history, and if, we could be solving this as a separate problem (agile way). Let me know what are your opinions on merging something like this without it being part of the undo-redo functionality for now.@ficristo@petetnt@ingorichter@redmunds@swmitra

[core-ai-bot]

Comment by redmunds
Tuesday Mar 07, 2017 at 01:45 GMT

---

@zaggino Nice to have this in the UI! Would be nice to have undo, but I suppose it's not the bad.

My main question: is it common to want to change the newline type for a single file? After changing newline for a file and then I create a new file, the newline type is initially set to type of file currently being edited, then it changes to hard-coded OS default as soon as I make an edit.

It seems that people usually want all files on system or project to use a certain newline. Should a newline preference be set that overrides getPlatformLineEndings()?

[core-ai-bot]

Comment by zaggino
Tuesday Mar 07, 2017 at 01:50 GMT

---

@redmunds Of course it should, thanks for the idea. I'll have a look at it. Also do some tests to this.

[core-ai-bot]

Comment by petetnt
Monday Mar 13, 2017 at 10:41 GMT

---

Re: history. Are the changes visible in the doc even if they don't affect the history? In my persistent history extension I work around similar issue by something like this:

https://github.com/petetnt/brackets-persistent-history/blob/master/main.js#L37

        // Set editor value to cachedDoc before setting history and then set the value back to the modified file
        // so that the history applies to the modified document too.
        if (wasModified) { 
          var currentDoc = codeMirrorDoc.getValue(); // current document
          codeMirrorDoc.setValue(cachedDoc); // the cached document
          codeMirrorDoc.setHistory(history); // history from a JSON file
          codeMirrorDoc.setValue(currentDoc); // apply the current state
        } else {
          codeMirrorDoc.setHistory(history);          
        }

In my case the I work around changes done outside of Brackets while keeping the old history, so the history becomes last state known to Brackets -> outside changes -> current doc. Would this work for old line endings -> new line endings scenario?

[core-ai-bot]

Comment by zaggino
Tuesday Mar 14, 2017 at 22:36 GMT

---

@petetnt The issue with history is that we'd need somehow to push a custom entry there, which CodeMirror would be able to process. The API for this could be quite simple I believe. Something like:

codeMirrorDoc.pushCustomHistoryEvent({
  undo: function() { /* this would revert line ending changes when invoked */ },
  redo: function() { /* this would re-apply line ending changes when invoked */ }
});

but I'm not sure how to achieve this without doing any modifications to CodeMirror itself.
Do you have any ideas how to achieve this? cc@marijnh

[core-ai-bot]

Comment by petetnt
Tuesday Mar 14, 2017 at 23:28 GMT

---

Oh yeah, now I get what you mean. Due to how it's hard to create the undo/redo objects is exactly why I asked/suggested in adobe/brackets#13152 (comment) if it was just possible to setValue so that the whole doc essential changes itself, while retaining the old history if just changing the line-endings doesn't call setValue (and thus creating the undo/redo states). After that one could just add a listener to the CodeMirror#changes event (or similar) to see if the line-endings had changed and adjust the UI etc. accordingly.

pushCustomHistoryEvent API sounds much more saner though.

(Sorry if I am repeating myself here, haven't had the time to test this PR out properly yet myself)

[core-ai-bot]

Comment by saurabh95
Tuesday Jun 13, 2017 at 11:00 GMT

---

I just did some testing around this PR. It is working fine and I agree with@redmunds, would have been better if we had undo/redo as well, but this is also fine. We can work on the history part later. Code also seems ok to me.
It would be really great if we could merge@ficristo's PR adobe/brackets#11614 for 1.10, for which this PR is pre-requisite, because users have been demanding for that feature for quite a long time now :)

[core-ai-bot]

Comment by ErikThiart
Thursday Feb 21, 2019 at 09:08 GMT

---

Any feedback on this guys, I am currently forced to use notepad++ because brackets default to CR LF instead I want the default to be UNIX's line endings LF since the majority of PHP projects built with brackets on a Windows client, will run on a UNIX system.