Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use null char to start markdown cell placeholder #729

Merged
merged 1 commit into from Aug 25, 2011

Conversation

minrk
Copy link
Member

@minrk minrk commented Aug 25, 2011

'&#x1D106' was added to the front of the placeholder text, to allow 'T' to be typed, but that simply prevented '&' from being typed. A null character should actually be safe until the codemirror bug is worked out.

@fperez
Copy link
Member

fperez commented Aug 25, 2011

search for that 1d106 string in the whole code, I'm pretty sure it was in more than one place in the previous PR. If it works, that's fine... Alternatively, if \u can serve as an escape instead of &, it could just be a matter of replacing the escapes.

Though the null has the advantage of being non-printing, which is nice (the musical one shows up). As long as it doesn't introduce other weirdnes..

@minrk
Copy link
Member Author

minrk commented Aug 25, 2011

The '&' is the HTML escape for unicode, but in javascript it's just the ascii string exactly as it appears, so it isn't treated any differently by codemirror than 'T' or '.' was before.

I haven't found any issues, even copying and pasting null-char-including strings into the cell (I think they were stripped out from the clipboard, though).

I just pushed again, after searching for the 1D106, and replacing everything I found.

@ellisonbg ellisonbg merged commit 5ae42c2 into ipython:master Aug 25, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants