-
Notifications
You must be signed in to change notification settings - Fork 4
Define next iteration of training content #649
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
Define next iteration of training content #649
Conversation
sc-8309/define-training-portal-next-iteration
A PR preview is available: Preview URL |
A PR preview is available: Preview URL |
A PR preview is available: Preview URL |
A PR preview is available: Preview URL |
A PR preview is available: Preview URL |
A PR preview is available: Preview URL |
1 similar comment
A PR preview is available: Preview URL |
A PR preview is available: Preview URL |
A PR preview is available: Preview URL |
1 similar comment
A PR preview is available: Preview URL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Learn with confidence
-
FYI, we have to be careful about using the images in the headings when we have the ToC active:
It works on the main landing page because we don't have the inline ToC active, but not so much in these other pages. Compromise?
-
According to our file naming conventions, the
program/overview.qmd
should actually beprogram/program-overview.qmd
to minimize conflicts with our OTHER main file namedoverview.qmd
...- I think we shouldn't be naming files ANYTHING overview personally and instead just using the page title but we've already done it and it was in the original internal style guide so y'know 💀
-
For someone who always harps on visual clutter, there sure is a lot of it on this overview page. Why are we using the
.highlights
class here randomly in the MIDDLE of the page? I've removed it as that looks better to me. "You have options" is... also oddly ominous. How about: -
Conversely, I think we do want to draw attention to the CTA at the bottom, so how about:
Learning paths
-
The coming soon callouts were a bit intrusive, highlighting information that was a placeholder. How about:
This also has the benefit of showing up in the ToC, giving users a heads-up instead of disappointment:
-
I feel like a lot of attention is being given to the module #s when they shouldn't be the focus. I tried instead:
Where do I find ...?
-
Ellipses should have a space before and after, as they are treated as a word in formatting.
-
The original
button-small
really doesn't work for me as the font is illegible, but I see why you tried them out to reduce the amount of spacing there needs to be between lines. I would rather have more white-space than unreadable buttons and honestly the lack of vertical spacing between the lines of buttons is oof for parsing.I'm thinking the ASCII arrows might not render the same on everyone's machines/browsers either. When we use smallcaps generally we don't capitalize the first letter either, since that looks off.
How about:
FUTURE FOOD FOR THOUGHT: Also.... this layout isn't realllllyyyyy mobile responsive:
The fact that we have to use a separate flexbox for each question is an authoring nightmare. Perhaps a use case for our new filtered tables if the layout is gonna be wonky anwyay?
General notes
- I noticed you removed the BG colour from the sidebar since our topnav is now that colour for training, but now it looks kind of off/blends in too much to me. Also, you removed it for ALL pages in the site AND the ToC section (you thought you could sneak that by me, huh? After we had a month long argument about the sidebar/ToC BG? Nice try!) — I don't agree with this change as I preferred the version with SOME contrast and that was the one we previously agreed upon. Can we compromise, add it back to the rest of the site where training isn't involved?
- For training, let's keep the lack of BG colour, but I've changed the colour of the border just slightly for visual distinction:

- Overall, I like the restructure/outline given by this new content a lot! It introduces what our training has to offer and gives users some better direction. 🎉
A PR preview is available: Preview URL |
Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com>
A PR preview is available: Preview URL |
Guilty as charged but this was actually unintended, with an apology! I had meant to remove the background and switch to the floating sidebar style only for the training pages but forgot I started to update the CSS and _quarto.yml but never quite completed the necessary changes to limit the changes to the training portal ... Thank you for catching that. To be honest, this change is rooted in my API work. The preview of the text is just so much cleaner without all the extra visual items on the left and right and it makes the content feel airy. (I basically was using some other docs sites and the ones that resonated with me had a simple, minimalist design that's similar.) |
This is interesting, at IBM we closed up the ellipsis if it was followed by a punctuation mark. But apparently, that's not what the CMOS says ... I did find this clarification that agrees with you:
|
Sidebar again ... I removed the dark vertical line you added and tried switching back to the However, when I tested this change it doesn't work unless you also change the sidebar for the main site, very much is not in the spirit of compromise. So I undid that but did, separately, simplify the sidebar for the training portal. |
A PR preview is available: Preview URL |
1 similar comment
A PR preview is available: Preview URL |
Agreed! I appreciate the readability improvements you made to the new button but the overall authoring experience is too much of 'I'm hacking Tachyons CSS' and not enough of 'I can focus on the content'. If we can find a way to serve FAQ-style content with less authoring effort, that would be helpful — I would include the main FAQ section that you revised in that effort, where all of the categories are manually added, for example. |
A PR preview is available: Preview URL |
Yes, I think this ties into "we should really start to move the training to its own site project, because right now we're using a bunch of CSS & navbar hacks which is a bit of a mess." All good, I figured if you had actually wanted it to change back you would have actually said something.
I think the ToC doesn't need to be there with the API docs, since it seems to repeat what's in the sidebar... I will the hold the rest of my thoughts on the increasing minimalism of the design until directly asked for input, though. |
... OK, this is clearly what you want, and it's your PR, so I'm gonna bow out of this one I think 😅 I'm not going to win this battle and you're in charge, so I'm just gonna suck it up. |
This isn’t just about what I want, though, it's about tested design principles? Like these:
And, since that list seriously lacks diversity, how about these?
I would qualify all of these principles with: you have taught me that you look at a page differently and that we can make this a part of our design. It has to be an inclusive design, though, and that includes you AND me. |
Yes, I agree, but this is too minimalist for me. Here are some counterpoints that this article sums up nicely:
Some of how we landed on the current theme is you suggesting that we borrow elements from the Platform UI — which I think was valid and lended cohesion to the docs which is definitely not a bad compromise. But we've moved away from that immediately in favour of more minimalism without more discussion, or actual input from design which is also something that has been suggested should happen when I attempt to make visual changes. The messaging around what you want the design to be has been really inconsistent when it comes to my suggestions, and I can't make sense of it. |
Those are very good counterpoints and they reflect the need for inclusive design. Also, there are standards and I ran an accessibility checker for ADA Section 508 against these sections of our docs site:
The results didn't show any discernible difference for the sidebar but I did notice some other things we should discuss. I'll open a story for this — let's discuss there.
We reached a point of saturation, adding things to the the site until it started to affect MY ability to read our site. Seeing as I am deep in GitHub comments rather than getting ready for tomorrow, how about this? Let's defer the design language for our docs site until we have a designer at the table AND can make sure we meet all the accessibility requirements. |
* Adding latest `RawData` notebook to docs (#647) * Pulling in latest RawData notebook * Added notebook to testing overview * Added notebook to FAQ - testing * feat: make lighter version of docs-site target * Fix notebook link (#651) * Part 1 — +"Breaking changes and deprecation" page under releases to support the new processes (#650) * Adding a breaking changes page * Details * Editing headers * Adding product area * Folder * Testing R table * It woooorks * Setup for multiple entires & adding search back * Adjusting test data for clarity * Single-sourcing the functions * Moving functions to scri[pt for cleaner exp * Column widths back in * Cleanup & templates * Templating README * Tweaking * Adding version and type columns * Filters for version & type * Examples and more testing * Intro to README * Moved history into its own folder * Instructions for +year * Add an entry draft * Entry example table * Testing R setup * Switching placement * Forgot a package * Cleanup & caching * Forgot a package again * Adding R env to staging & prod flows * Naming the code cells for output cleanliness * Removed fake 2025 example * Wow, typo * Wording adjustment * Removed blog posts * Fix notebook link (#651) (#654) Co-authored-by: Nik Richers <nik@validmind.ai> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * Define next iteration of training content (#649) * Rough in some topics * WIP content noodling * Fix merge conflict * Content rework * Content edits * Add mapping from questions to courses * Content edits * Updated content, move videos * Update learning paths * Update learning paths * Expand learning paths * Update learning paths, FAQ * Edits * Update course content * Update FAQ * Fix margin callout, shorten sample training plan * Minor edits * Add docs links to FAQ * Update FAQ and learning paths * Training FAQ updates * Clean up temporary CSS, add small button CSS * Hide .attn sections to be able to publish the page * Update FAQ page formatting, add Pandas link * Add some FAQ-elements * Best offer for formatting an shenanigans * Hide more working notes stuff, add .attn to learning path course cards * More learning paths formatting improvements * Simplify language and remove old file * Add callouts to learning paths * Modifying some style stuff * Redoing buttons for readability * Quick tweak to sidebar * Overview rename & cleanup * Reverting sidebar & toc for rest of site * Update site/training/program/sample-training-plan.qmd Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com> * Training sidebar tweaks * Remove extra newline and commented out line * Improve 'coming soon' text --------- Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com> --------- Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Spencer Krum <nibz@validmind.ai> Co-authored-by: Spencer Krum <nibz@spencerkrum.com>
Internal Notes for Reviewers
This PR adds more content to our training portal. LIVE PREVIEW
Landing page changes
New section about the training program
Includes:
Relocated video playlists
Training program
Program overview
Adds a short, public-facing prospectus that accompanies our internal training playbook with new sidebar navigation:
Learning paths
Includes our full curriculum for the different courses in one place.
Note:
Sample training plan
Similar to our internal training playbook, this page provides a sample of what is available through our training program on request:
Where do I find ...?
Training FAQs that map actual customer questions we have been asked to training and docs.
Notes
{.button-small}
that can be used for inline links where margin links are cumbersome.External Release Notes
We’ve added a training overview, learning paths, and a sample training plan to help customers navigate their training journey. We’ve also introduced an FAQ that maps common customer questions to relevant learning paths.
Training portal{.button}