feat: add ethskills callout section to /developers#17753
Merged
Conversation
Promotes ethskills.com with its ASCII block logo, a short description of the agentic Ethereum stack, an animated terminal cycling through natural-language agent prompts, and a CTA button. Removes the previous inline link from the Quickstart card. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
wackerow
force-pushed
the
feat/ethskills-section
branch
from
52a39ea to
e5610cc
Compare
Move EthSkillsTerminal into a reusable TerminalTypewriter UI component that accepts a messages prop. Add Storybook story with multiple variants. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Add translation keys for the ethskills section heading, description, terminal messages, and CTA button. Use interpolation variable for brand name protection. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Add dir="ltr" to terminal chrome and dir="auto" to text content for proper RTL language rendering. Add RTL Storybook story and fix width display in stories. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
github-actions
Bot
added
content 🖋️
Move dir="ltr" to terminal container so window buttons stay left in RTL contexts. Move Storybook story to Molecules / Display Content. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Add translated ethskills section strings (title, desc, CTA, 5 terminal messages) for all non-English locales. AI-generated -- recommend native speaker review. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Use useTranslations in Storybook story so locale dropdown drives terminal messages. Remove hardcoded RTL and ShortMessages examples. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Add dir="ltr" to ASCII art banner to prevent RTL flip. Replace literal "onchain" in ethskills-title with approved glossary translations for each locale. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Fix component reference in meta to use aliased import. Add required args to satisfy StoryObj type constraint. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Flatten section markup by moving flex classes to Section container. Use gap-8 for 2rem spacing and start-align on mobile with sm:items-center. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Replace Unicode box-drawing ETHSKILLS banner with an optimized SVG (7KB) to fix monospace font rendering on Android. Uses fill=currentColor for dynamic theming. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
- remove custom light-mode styling, embeds brand color in svg - moves svg to public; removes import - adjusts size
- use internal mono font for cli appearance - reusability: add data-label for custom targeting cli styling, e.g., adjusting height when consumed
pettinarip
approved these changes
Mar 13, 2026
Member
pettinarip
left a comment
pettinarip
left a comment
There was a problem hiding this comment.
@wackerow pulling this in.
I've added two small improvements/fixes:
- empty messages guard
- css blink animation instead of the js
setInterval
Other things we could add later:
- pause offscreen with
IntersectionObserverRevealto stop thesetTimeoutchains when not visible - reduced motion
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Preview link
https://deploy-preview-17753.ethereum.it/developers/#ethskills