music.html: add multilingual content and translate YouTube section by Copilot · Pull Request #176 · DavidKRK/DavidKRK.github.io

Copilot · 2026-04-15T15:03:56Z

Add translated description paragraphs for each language section (fr, en, es, eu, ar, uk)
Add id="youtube-title-text" to YouTube section heading <span> and id="youtube-section-title" support for auto-injected headings
Add youtube_title key to each language in the langContent object
Update switchLanguage() to also update the YouTube section heading (youtube-title-text and youtube-section-title)
Fix Arabic RTL: switchLanguage() now sets document.documentElement.lang and document.documentElement.dir
Fix language switcher not working: main.js was using classList.toggle() + inline style.display which set display:none inline on all .lang-content elements, overriding CSS class-based visibility on every page. Fixed to use proper classList.add/remove with no inline styles.
Resolve merge conflicts to unblock PR merge:
- index.html: incorporated responsive.css link and owl image div alongside the language selector
- .github/workflows/social-media-post.yml: aligned with base branch version to resolve add/add conflict
- music.html: aligned YouTube span ID (youtube-title-text) and key (youtube_title) with changes already on gh-pages (PR fix: translate YouTube section heading on language switch in music.html #175)

…ions and YouTube heading Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/2ed874a1-db1e-42ff-b88e-b11e778937f0 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

bolt-new-by-stackblitz · 2026-04-15T15:04:00Z

Run & review this pull request in StackBlitz Codeflow.

Copilot

Pull request overview

Improves the usability of the music.html language switcher by adding real per-language page descriptions and making the YouTube section heading translatable via switchLanguage().

Changes:

Added descriptive paragraphs to each .lang-content block for fr/en/es/eu/ar/uk.
Wrapped the YouTube section heading text in a dedicated element so it can be updated dynamically.
Extended the langContent translation map with a youtube_heading key for all languages and updated switchLanguage() accordingly.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

DavidKRK

@copilot @openai-code-agent @anthropic-code-agent regardez si tout est OK, si des erreurs veuillez les corriger

…ty support Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/147b3a61-c8ad-4cce-8967-446c6397d9f8 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

Copilot · 2026-04-15T15:21:30Z

@copilot @openai-code-agent @anthropic-code-agent regardez si tout est OK, si des erreurs veuillez les corriger

Tout a été vérifié. Un bug a été corrigé (commit 4d7c86f) : lors du passage en arabe, le titre et le bouton YouTube (qui sont en dehors des blocs .lang-content) restaient en LTR. switchLanguage() met maintenant aussi à jour document.documentElement.lang et document.documentElement.dir (rtl pour l'arabe, ltr pour les autres langues), ce qui corrige l'affichage et améliore l'accessibilité pour les lecteurs d'écran.

DavidKRK

J approuve

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

#178) * fix: set html lang and dir attributes on language switch for correct RTL rendering Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/045fb7ca-e251-419b-bd87-0c85b16cfc2e Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com> * Update music.html Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

DavidKRK · 2026-04-15T15:36:43Z

@copilot vérifie pourquoi ça marche pas

… inline styles Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/1392f06a-3278-4510-9e84-5aba038fd1a6 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

Copilot · 2026-04-15T15:41:14Z

@copilot vérifie pourquoi ça marche pas

Le bug était dans assets/js/main.js (commit a75f2ec). Ce fichier avait son propre gestionnaire de langue qui utilisait classList.toggle() et des styles inline (element.style.display = 'none'). Les styles inline ont une priorité plus haute que les classes CSS, donc ils écrasaient les règles .lang-content { display: none } / .lang-content.active { display: block } sur toutes les pages.

Résultat : quand on cliquait sur une langue, main.js posait display:none en inline sur tous les blocs, empêchant le switchLanguage() de la page d'en afficher le bon.

Correction : main.js utilise maintenant classList.add/remove (pas de styles inline), cohérent avec le switchLanguage() de chaque page.

DavidKRK

Je suis OK

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

DavidKRK

@copilot je n arrive pas à Merge Peux-tu m'aider

…ia-post.yml Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/dcbd6b39-5873-47cc-9a33-cb1d146df047 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

Copilot · 2026-04-15T16:02:24Z

@copilot je n arrive pas à Merge Peux-tu m'aider

Les conflits qui bloquaient le merge ont été résolus (commit 606214c) :

index.html : conflit entre notre sélecteur de langue et les changements de master (ajout de responsive.css + image david_krk_owl.jpg). Les deux sont maintenant présents.
.github/workflows/social-media-post.yml : conflit "add/add" (deux versions différentes du même fichier). Aligné avec la version de master.

Le PR devrait maintenant pouvoir être mergé sans conflit.

DavidKRK

@copilot si c'est ok vas y

DavidKRK · 2026-04-15T16:07:49Z

Je peux pas Merger @copilot @anthropic-code-agent @anthropic-code-agent

…e merge conflict Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/4e6c3e68-a8ad-4156-ac12-bdb610e20104 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

Copilot · 2026-04-15T16:12:53Z

@copilot si c'est ok vas y

Le PR est prêt à être mergé. Tous les conflits ont été résolus — vous pouvez maintenant cliquer sur Merge pull request dans GitHub.

Je peux pas Merger @copilot @anthropic-code-agent @anthropic-code-agent

Le dernier conflit a été résolu (commit 904fd1a). Il y avait un conflit dans music.html avec le PR #175 déjà mergé sur gh-pages : l'ID du <span> YouTube était différent (youtube-section-title vs youtube-title-text). J'ai aligné notre branche sur la convention de gh-pages (id="youtube-title-text", clé youtube_title).

Le merge devrait maintenant fonctionner depuis l'interface GitHub (Merge pull request).

DavidKRK

Ok

feat: add multilingual content to music.html with translated descript…

b32c9b7

…ions and YouTube heading Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/2ed874a1-db1e-42ff-b88e-b11e778937f0 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

Copilot AI assigned Copilot and DavidKRK Apr 15, 2026

Copilot created this pull request from a session on behalf of DavidKRK April 15, 2026 15:03 View session

DavidKRK marked this pull request as ready for review April 15, 2026 15:04

Copilot AI review requested due to automatic review settings April 15, 2026 15:04

Copilot started reviewing on behalf of DavidKRK April 15, 2026 15:04 View session

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Comment thread music.html Outdated

DavidKRK requested a review from Copilot April 15, 2026 15:10

Copilot started reviewing on behalf of DavidKRK April 15, 2026 15:11 View session

Copilot AI reviewed Apr 15, 2026

View reviewed changes

DavidKRK self-requested a review April 15, 2026 15:18

DavidKRK reviewed Apr 15, 2026

View reviewed changes

Copilot started work on behalf of DavidKRK April 15, 2026 15:20 View session

fix: set document lang and dir on language switch for RTL/accessibili…

4d7c86f

…ty support Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/147b3a61-c8ad-4cce-8967-446c6397d9f8 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

Copilot finished work on behalf of DavidKRK April 15, 2026 15:22

Copilot AI requested a review from DavidKRK April 15, 2026 15:22

DavidKRK approved these changes Apr 15, 2026

View reviewed changes

DavidKRK requested a review from Copilot April 15, 2026 15:24

Copilot started reviewing on behalf of DavidKRK April 15, 2026 15:25 View session

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Copilot started work on behalf of DavidKRK April 15, 2026 15:36 View session

fix: correct language switcher in main.js to use classList instead of…

a75f2ec

… inline styles Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/1392f06a-3278-4510-9e84-5aba038fd1a6 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

Copilot finished work on behalf of DavidKRK April 15, 2026 15:42

Copilot AI requested a review from DavidKRK April 15, 2026 15:42

DavidKRK approved these changes Apr 15, 2026

View reviewed changes

DavidKRK requested a review from Copilot April 15, 2026 15:45

Copilot started reviewing on behalf of DavidKRK April 15, 2026 15:46 View session

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Comment thread assets/js/main.js

DavidKRK self-requested a review April 15, 2026 15:51

DavidKRK reviewed Apr 15, 2026

View reviewed changes

Copilot started work on behalf of DavidKRK April 15, 2026 15:53 View session

fix: resolve merge conflicts with master in index.html and social-med…

606214c

…ia-post.yml Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/dcbd6b39-5873-47cc-9a33-cb1d146df047 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

Copilot finished work on behalf of DavidKRK April 15, 2026 16:03

Copilot AI requested a review from DavidKRK April 15, 2026 16:03

DavidKRK approved these changes Apr 15, 2026

View reviewed changes

Copilot started work on behalf of DavidKRK April 15, 2026 16:07 View session

fix: align music.html youtube span ID and key with gh-pages to resolv…

904fd1a

…e merge conflict Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/4e6c3e68-a8ad-4156-ac12-bdb610e20104 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>

Merge branch 'gh-pages' into copilot/add-languages-to-music-page

5cbef11

Copilot finished work on behalf of DavidKRK April 15, 2026 16:14

Copilot AI requested a review from DavidKRK April 15, 2026 16:14

DavidKRK assigned Claude and Codex Apr 15, 2026

DavidKRK approved these changes Apr 15, 2026

View reviewed changes

DavidKRK merged commit ffb6a40 into gh-pages Apr 15, 2026
6 of 7 checks passed

Conversation

Copilot AI commented Apr 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bolt-new-by-stackblitz bot commented Apr 15, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

DavidKRK left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Apr 15, 2026

Uh oh!

DavidKRK left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

DavidKRK commented Apr 15, 2026

Uh oh!

Copilot AI commented Apr 15, 2026

Uh oh!

DavidKRK left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

DavidKRK left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Apr 15, 2026

Uh oh!

DavidKRK left a comment

Choose a reason for hiding this comment

Uh oh!

DavidKRK commented Apr 15, 2026

Uh oh!

Copilot AI commented Apr 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

DavidKRK left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Copilot AI commented Apr 15, 2026 •

edited

Loading

Copilot AI commented Apr 15, 2026 •

edited

Loading