[Contribution guidelines] Updates syntax quick reference#5065
[Contribution guidelines] Updates syntax quick reference#5065natasha-moore-elastic wants to merge 3 commits intomainfrom
Conversation
Vale Linting ResultsSummary: 1 suggestion found 💡 Suggestions (1)
The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
🔍 Preview links for changed docs |
Just to explain why I didn't use tabs initially: readers might want to look at the syntax and output at the same time, and tabs make that impossible. (IMHO the tabs in the full syntax guide have this usability problem too.) was trying to follow this guidance! 😁 "don't make users switch tabs to compare content" |
theletterf
left a comment
There was a problem hiding this comment.
Good job! Let's get also a review from @leemthompo and @jmikell821
| **DOs**<br> | ||
| ✅ **Do:** Use different types of diagrams, such as flowcharts, sequence diagrams, state diagrams, and so on<br> | ||
|
|
||
| **DON'Ts**<br> | ||
| ❌ **Don't:** Rely on diagrams for users with JavaScript disabled — they will see raw Mermaid code | ||
|
|
There was a problem hiding this comment.
I would remove this particular DON'T, since we expect users to browse the site using a JS enabled browser (in most cases). LLMs will see the Mermaid diagram source and interpret it all right.
Can we think of do's and don'ts more focused on diagrams? For example: https://developers.google.com/style/images
Makes sense, thanks! I've reverted to the dropdowns 👍 |
|
Changes look good to me, thanks @natasha-moore-elastic. Thinking about the UX of this page, I wonder if we might:
I suspect 90% of folks will just want to copy/paste the syntax as fast as possible, so feels like the secondary information is very front and centre at moment (i.e. the dos and don'ts are kinda hogging real estate and they are visually very loud) That said, we might even consider putting the do's and don'ts in a dropdowns too, if we agree that action > learning for most consumers of this page, and allow folks who want to drill down into best practices to open those dropdowns when needed. Just a few ideas, and TBC I would 100% get an LLM to do this if we think any of these suggestions make sense, because this would be pure manual drudgery :) |
Just wanted to mention that in the first version of the quick ref, we were seeing it as an "internal" document, so the do's and don'ts seemed more important. There were/are also some elements (like Headings) where both dropdowns might elicit a reaction of "well, obviously"... So maybe there's not a one-size-fits-all solution, but for sure the audience and scope of the page have changed. |
Summary
Resolves https://github.com/elastic/docs-content-internal/issues/527.
Updates the syntax quick reference page:
applies_toguidance for accuracyGenerative AI disclosure