Skip to content

Add art assets for chatbot and security workshops with accessible alt text#629

Merged
ozhang22 merged 7 commits into
NuevoFoundation:masterfrom
beagandica:fix/art-assets-alt-text
Apr 29, 2026
Merged

Add art assets for chatbot and security workshops with accessible alt text#629
ozhang22 merged 7 commits into
NuevoFoundation:masterfrom
beagandica:fix/art-assets-alt-text

Conversation

@beagandica
Copy link
Copy Markdown
Member

Summary

Applies @YuliMasi's art assets from #594 with accessibility fixes that couldn't be pushed to the original fork (maintainerCanModify was disabled).

What's included

Chatbot workshop (11 new illustration images):
introduction.png, surprised.png, elderly.png, chatbot1.png, chat_Alice.png, ideas.png, idea.png, learn.png, chat_persons_robot.png, sentence_response.png, party.png

Security Fundamentals workshop (5 new illustration images):
hacker.png, question.png, secret-notes.png, security.png, teacher.png

Fixes applied on top of #594

  • Added descriptive alt text to all 10 chatbot images that had empty alt=""
  • Fixed 2 malformed self-closing img tags (missing closing bracket) in activity-3 and activity-5
  • Added missing alt attribute to hacker.png img tag
  • Replaced placeholder "Cannot load image" alt text with proper description
  • Added alt parameter to 3 Hugo figure shortcodes in security-fundamentals
  • Removed trailing ? from security.png image path
  • Cleaned up unnecessary empty style="" attributes
  • Dropped unrelated German adafruit change

Credits

All art assets created by @YuliMasi - thank you! This PR supersedes #594 to apply the accessibility fixes.

Closes #594

YuliMasi and others added 6 commits April 28, 2026 18:57
… text

Apply YuliMasi's art assets from NuevoFoundation#594 with accessibility fixes:

Chatbot workshop (11 images):
- Add descriptive alt text to all 10 images that had empty alt=""
- Fix 2 malformed self-closing <img> tags (missing closing >)
- Remove unnecessary empty style attributes

Security Fundamentals workshop (5 images):
- Add alt text to 3 Hugo figure shortcodes
- Add missing alt attribute to hacker.png <img> tag
- Replace placeholder 'Cannot load image' alt text with description
- Remove trailing ? from security.png image path

Supersedes NuevoFoundation#594. Drops unrelated German adafruit change.

Co-authored-by: YuliMasi <[email protected]>
Co-authored-by: Copilot <[email protected]>
…creenshot sizing

- Fix PT-BR alt text: add proper diacritics (segurança, três, cenários, possíveis)
- Convert security figure widths from pixels to percentages for mobile
  responsiveness (900px→60%, 700px→50%, 600px→40%, 300px→20%)
- Restore chatbot customer_service_chatbot.png from 20% to 35% (UI screenshot
  needs to be legible, not decorative-art sized)

Co-authored-by: Copilot <[email protected]>
Copy 5 art images (hacker, security, question, secret-notes, teacher)
to brazilian-portuguese and korean security-fundamentals directories.
Add figure/img tags with properly localized alt text to all 8 translated
markdown files across both languages.

This ensures Portuguese and Korean learners see the same engaging
illustrations as English learners.

Co-authored-by: Copilot <[email protected]>
@beagandica beagandica force-pushed the fix/art-assets-alt-text branch from b5032b9 to 251f895 Compare April 29, 2026 02:11
@ozhang22 ozhang22 merged commit a63c13f into NuevoFoundation:master Apr 29, 2026
4 of 5 checks passed
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.

3 participants