From effa5c84861a51e5224f7c431918b65ed93a24bc Mon Sep 17 00:00:00 2001
From: Guinet Nicolas <nguinet.pro@gmail.com>
Date: Tue, 23 Jan 2024 08:01:35 +0100
Subject: [PATCH] Update LanguageSelector.astro: value must be omitted for
 boolean attributes in html tags
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

<select>
  👎<option value="/" selected='true'> English </option>       👈 Change from this
  👍<option value="/" selected> English </option>              👈 to this

According HTML5 spec on boolean attributes
- https://www.w3.org/TR/2008/WD-html5-20080610/semantics.html#boolean
If the attribute is present, its value must either be the empty string or a value that is a case-insensitive match for the attribute's canonical name, with no leading or trailing whitespace.
---
 src/components/LanguageSelector.astro | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/components/LanguageSelector.astro b/src/components/LanguageSelector.astro
index 5210682..57e1bd3 100644
--- a/src/components/LanguageSelector.astro
+++ b/src/components/LanguageSelector.astro
@@ -39,8 +39,12 @@ const { showFlag = false, languageMapping, ...attributes } = Astro.props;
 
       const label = flag + nativeName;
 
+      let selectedAttr = {};
+      if (supportedLanguage === currentLanguage)
+        selectedAttr = { ...selectedAttr, selected: ''};
+
       return (
-        <option value={value} selected={supportedLanguage === currentLanguage}>
+        <option value={value} {...selectedAttr}>
           {label}
         </option>
       );