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

Docs: Update readme #273

Merged
merged 3 commits into from
Apr 16, 2024
Merged

Docs: Update readme #273

merged 3 commits into from
Apr 16, 2024

Conversation

phil-s
Copy link

@phil-s phil-s commented Apr 5, 2024

Per (info "(org)Escape Character") I've introduced a zero-width space before the closing paren in "(while writing in minibuffer: ~C-c '~​)" in order to achieve the correct formatting. This can be visualised using (set-face-background 'glyphless-char "red").

I've substituted S-<return> everywhere that S-RET appeared, as the latter isn't a valid representation.

@phil-s
Copy link
Author

phil-s commented Apr 5, 2024

Darn, Github doesn't like what I did with the C-c ' formatting!

https://github.com/phil-s/ement.el/tree/phil-s/docs?tab=readme-ov-file#room-buffers

I'll see if I can figure out a fix for that...

Along similar lines to (info "(org)Escape Character") I've introduced
a ZERO WIDTH JOINER character after the apostrophe in ~C-c '‍~ in order
to achieve the correct appearance in both Emacs and Github.  This can
be visualised using (set-face-background 'glyphless-char "red").

I've substituted S-<return> everywhere that S-RET appeared, as the
latter isn't a valid representation.
@phil-s
Copy link
Author

phil-s commented Apr 5, 2024

That was a pain -- I think I now understand why the text had previously been written as C-c ') !

Through trial-and-error I've found that a single ZERO WIDTH JOINER after the apostrophe in C-c '‍ works for both Emacs and Github, so I've pushed that fix.

The reply functionality has changed since that was written, and an
active region no longer has any bearing on the quoted text.
I prefer the Oxford comma. :)
@alphapapa alphapapa merged commit 7645f75 into alphapapa:master Apr 16, 2024
0 of 4 checks passed
@alphapapa
Copy link
Owner

Thank you.

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.

2 participants