Skip to content

Conversation

@agananya03
Copy link

@agananya03 agananya03 commented Jan 10, 2026

Summary

Fixed broken language navigation and added Portuguese to the documentation language switcher.

Changes Made

  • ✅ Added "Português" link to language navigation in all guide HTML files
  • ✅ Fixed language switching across English, Spanish, Japanese, Chinese, and Portuguese guides
  • ✅ Added missing images to Portuguese guide directory (50+ SVG/PNG files)
  • ✅ Fixed header-icons paths in Portuguese documentation
  • ✅ All language links now use correct relative paths from any language version

Issues Fixed

Fixes #5108 - Portuguese documentation not accessible via language switcher
Fixes #5110 - Broken language links in non-English guides

Testing Done

  • Tested language switching from English → all languages
  • Tested language switching from Spanish → all languages
  • Tested language switching from Portuguese → all languages
  • Verified Portuguese documentation loads correctly
  • Verified all language links work correctly from any language version
  • Checked browser console for errors - resolved 404s for images

Environment

  • Server: Python's http.server (port 8000)
  • Browser: Chrome
  • Operating System: Windows 11

Results

✅ Portuguese Language Added to Switcher

Portuguese (Português) now appears in the language navigation across all guide versions

✅ Portuguese Guide Loading Successfully

The Portuguese guide loads correctly with all language navigation links functional

✅ No Console Errors

Browser console shows no 404 errors - all images and assets load successfully

✅ Language Switching Works from All Guides

Language switching works correctly from English, Spanish, Japanese, Chinese, and Portuguese guides

📝 Changes Made

  • Modified: Docs/guide/index.html, Docs/guide-es/index.html, Docs/guide-ja/index.html, Docs/guide-pt/index.html, Docs/guide-zhCN/index.html

    • Added Portuguese (Português) link to language navigation in all guide HTML files
    • Fixed language switcher paths to work correctly from all language versions
  • Added: 50+ image files to Docs/guide-pt/ directory

    • Copied missing SVG and PNG assets to resolve 404 errors
    • Includes all necessary diagrams, icons, and visual elements
  • Modified: Docs/guide-pt/index.html and other Portuguese guide files

    • Fixed header-icons paths from /Docs/header-icons/ to /header-icons/
    • Removed reference to missing mode-up-down.png sample image

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

Copy link
Contributor

@zealot-zew zealot-zew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

its a good fix, but the commit aint clean(fix that)
also add portuguese guide link in all other language html files
image
so portuguese would be an option here

also this does not solve the issue #5110

- Added Português link to language navigation in all guide HTML files
- Fixed language switching across English, Spanish, Japanese, Chinese, and Portuguese guides
- Added missing images to Portuguese guide directory
- Fixed header-icons paths in Portuguese documentation
- All language links now work correctly from any language version

Portuguese documentation is now fully accessible and all language navigation works properly.

Fixes sugarlabs#5109
Fixes sugarlabs#5110
@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@aryan1752
Copy link
Contributor

its a good fix, but the commit aint clean(fix that) also add portuguese guide link in all other language html files image so portuguese would be an option here

also this does not solve the issue #5110

hey,i already solved this issue go through my
pr once

its a good fix, but the commit aint clean(fix that) also add portuguese guide link in all other language html files image so portuguese would be an option here

also this does not solve the issue #5110

hey,i already solved this go through my pr once

@aryan1752
Copy link
Contributor

aryan1752 commented Jan 10, 2026

its a good fix, but the commit aint clean(fix that) also add portuguese guide link in all other language html files image so portuguese would be an option here

also this does not solve the issue #5110

6d9c577
3b2f080

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.

Broken language links in Music Blocks Guide (non-English guides fail to load) ! Portuguese language option missing in Music Blocks Guide navigation !

3 participants