-
Notifications
You must be signed in to change notification settings - Fork 1.9k
feat : add new language picker component #2040
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
bjohansebas
merged 14 commits into
expressjs:gh-pages
from
ShubhamOulkar:refactor-lang-picker
Sep 14, 2025
Merged
Changes from all commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
def57d6
remove mobile language picker
ShubhamOulkar 7f02d3f
add click event on btn
ShubhamOulkar 6c9d8d3
add position
ShubhamOulkar 63a14f1
add style
ShubhamOulkar d85fa0e
refactor
ShubhamOulkar 29413a7
close language list on overlay click event
ShubhamOulkar 743f9cf
close on outside click except for lang btn
ShubhamOulkar bd4ccc8
Merge branch 'refactor-lang-picker' of https://github.com/ShubhamOulk…
ShubhamOulkar c553628
Merge branch 'gh-pages' into refactor-lang-picker
ShubhamOulkar 4333c86
Merge branch 'gh-pages' into refactor-lang-picker
ShubhamOulkar 2814d64
CI (lighthouse): check correct header response
ShubhamOulkar 035d48c
add current language highlighting
ShubhamOulkar 25275d9
remove home link from translation
ShubhamOulkar 815495c
Merge branch 'gh-pages' into refactor-lang-picker
bjohansebas File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,3 @@ | ||
# Home | ||
home: Home | ||
|
||
# Getting started | ||
getting_started: Getting started | ||
installing: Installing | ||
|
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,11 +12,6 @@ | |
<nav id="navbar" aria-label="primary"> | ||
<input id="q" placeholder="🔎 search"> | ||
<ul id="navmenu"> | ||
<li> | ||
<a href="/" id="home-menu" {% if page.menu=='home' %} class="active" {% endif %}> | ||
{{ site.data[page.lang].menu.home }} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Translations for this is removed. This link is redundant because express logo also points to homepage. |
||
</a> | ||
</li> | ||
<li id="getting-started-menu" class="submenu"> | ||
<a href="/{{ page.lang }}/starter/installing.html" {% if page.menu=='starter' %} class="active" {% endif%}> | ||
{{ site.data[page.lang].menu.getting_started }} | ||
|
@@ -252,6 +247,5 @@ | |
<span id="icon-sun">{% include icons/sun.svg %}</span> | ||
</button> | ||
{% include language-picker.html %} | ||
{% include language-picker-mobile.html %} | ||
</div> | ||
</header> |
This file was deleted.
Oops, something went wrong.
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,22 +1,21 @@ | ||
<div class="desktop-lang-switcher"> | ||
{% assign url_parts = page.url | split: '/' %} | ||
{% assign url_remainder = url_parts | slice: 2, url_parts.size | join: '/' %} | ||
{% assign current_lang = page.lang %} | ||
<button class="lang-btn" type="button" aria-haspopup="menu" aria-label="Change language"> | ||
<span id="current-lang"></span> | ||
</button> | ||
<ul class="lang-list submenu-content" aria-labelledby="current-lang" > | ||
{% for lang in site.data.languages %} | ||
<li> | ||
<a href="/{{ lang.code }}/{{ url_remainder }}"> | ||
{% if lang.code == current_lang %} | ||
<div class="desktop-lang-switcher"> | ||
<button id="langBtn" class="lang-btn" type="button" aria-expanded="false" aria-haspopup="listbox" aria-label="Change language"> | ||
{% include icons/i18n.svg %} | ||
</button> | ||
<ul id="langList" class="lang-list" role="listbox"> | ||
{% for lang in site.data.languages %} | ||
<li> | ||
<a href="/{{ lang.code }}/{{ url_remainder }}"> | ||
{% if lang.code == current_lang %} | ||
<strong>{{ lang.name }}</strong> | ||
{% else %} | ||
{% else %} | ||
{{ lang.name }} | ||
{% endif %} | ||
</a> | ||
</li> | ||
{% endfor %} | ||
</ul> | ||
<div id="languageData" data-languages='{{ site.data.languages | jsonify }}' style="display:none;"></div> | ||
</div> | ||
{% endif %} | ||
</a> | ||
</li> | ||
{% endfor %} | ||
</ul> | ||
</div> |
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
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correcting header status string. Now lighthouse audit is not running because of wrong status checking.