Skip to content
This repository was archived by the owner on Aug 14, 2024. It is now read-only.

Conversation

bkmgit
Copy link
Contributor

@bkmgit bkmgit commented Apr 18, 2021

Use collapsible sections for connecting, follows https://gist.github.com/pierrejoubert73/902cc94d79424356a8d20be2b382e1ab though can possible add a template for this. Addresses #40

Use collapsible sections for connecting, follows https://gist.github.com/pierrejoubert73/902cc94d79424356a8d20be2b382e1ab though can possible add a template for this. Addresses #40
@tkphd
Copy link
Member

tkphd commented Apr 18, 2021

I like the result, but have concerns over the implementation.

  • All of the options are drop-downs, leaving no default.
  • It is preferable to use Markdown instead of raw HTML.

Please present the recommended practice (EdDSA) in full, then create a {: .callout} box mentioning that other crypto-systems or authentication methods may be employed. Inside, use {: .solution} blocks to create drop-down choices.

@bkmgit
Copy link
Contributor Author

bkmgit commented May 7, 2021

@tkphd Maybe we need something other than {: .solution} to create a dropdown. Made an issue for this in the styles repository carpentries/styles#595

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants