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

Minor fixes #505

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Minor fixes #505

wants to merge 5 commits into from

Conversation

aadcg
Copy link
Collaborator

@aadcg aadcg commented Mar 21, 2022

No description provided.

@joaotavora
Copy link
Owner

Hi @aadcg , this is useful of course. Can you tell me if you've checked that the new https links actually work or may be broken? (if both http and https are broken, of course it doesn't matter).

@aadcg
Copy link
Collaborator Author

aadcg commented Mar 22, 2022

Hi João! I just checked, and we're good.

The following are both broken (http and https):

Is there smth else I can do? Thanks.

@aadcg
Copy link
Collaborator Author

aadcg commented Mar 23, 2022

6c45343 introduces a minor correction in the manual. Another approach would be to add <down> and <up> to the transient keymap. Let me know what you think @joaotavora.

@aadcg
Copy link
Collaborator Author

aadcg commented Apr 6, 2022

Ready when you are @joaotavora :)

@joaotavora
Copy link
Owner

joaotavora commented Apr 6, 2022 via email

@aadcg
Copy link
Collaborator Author

aadcg commented Oct 11, 2022 via email

The <down> and <up> bindings are only bound in the *sly-completions*
buffer, whereas C-n and C-p are bound in the buffer originating
a *sly-completions* buffer.
;;; instructions on http://www.grumblesmurf.org/lisp/corman-sly.
;;; https://web.archive.org/web/20160401182927/https://www.grumblesmurf.org/lisp/corman-patches
;;; ,otherwise SLY will blow up in your face. You should also follow
;;; the instructions on https://www.grumblesmurf.org/lisp/corman-sly.
Copy link
Collaborator Author

@aadcg aadcg Nov 12, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this last link doesn't exist, should we just delete this paragraph? @joaotavora

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

2 participants