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
Add LocaleSwitcher #89
Conversation
Creating LocaleSwitcher Add CHANGELOG
2078b9f
to
116f6c2
Compare
@@ -6,6 +6,7 @@ | |||
"admin/editor.header.link.url": "admin/editor.header.link.url", | |||
"admin/editor.header.show.searchbar.title": "admin/editor.header.show.searchbar.title", | |||
"admin/editor.header.show.login.title": "admin/editor.header.show.login.title", | |||
"admin/editor.header.show.localeswitcher.title": "admin/editor.header.show.localeswitcher.title", |
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.
Should this appear on /admin
? It's not working for me
CHANGELOG.md
Outdated
@@ -6,6 +6,8 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) | |||
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html). | |||
|
|||
## [Unreleased] | |||
### Added | |||
- Adds support for `localeSwitcher`. |
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.
- Adds support for `localeSwitcher`. | |
- Support for `localeSwitcher`. |
@@ -134,6 +138,12 @@ Header.schema = { | |||
default: true, | |||
isLayout: true, | |||
}, | |||
showLocaleSwitcher: { |
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.
I wasn't able to see this configuration in the storefront, can you take a look into that?
messages/es.json
Outdated
@@ -5,6 +5,7 @@ | |||
"admin/editor.header.logo.image": "Imagen del logotipo", | |||
"admin/editor.header.link.url": "Link del logotipo", | |||
"admin/editor.header.show.searchbar.title": "Mostrar Busca", | |||
"admin/editor.header.show.localeswitcher.title": "Mostrar alternador de localidade", |
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.
"admin/editor.header.show.localeswitcher.title": "Mostrar alternador de localidade", | |
"admin/editor.header.show.localeswitcher.title": "Mostrar selector de idioma", |
messages/pt.json
Outdated
@@ -6,6 +6,7 @@ | |||
"admin/editor.header.link.url": "Link da Logo", | |||
"admin/editor.header.show.searchbar.title": "Mostrar Busca", | |||
"admin/editor.header.show.login.title": "Mostrar Login", | |||
"admin/editor.header.show.localeswitcher.title": "Mostrar alternador de localidade", |
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.
"admin/editor.header.show.localeswitcher.title": "Mostrar alternador de localidade", | |
"admin/editor.header.show.localeswitcher.title": "Mostrar seletor de idioma", |
@@ -47,6 +48,7 @@ | |||
"minicart", | |||
"login", | |||
"search-bar", | |||
"locale-switcher", |
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.
You should create an independent block for locale-switcher
as well.
So it will work with the header-layout
Creating LocaleSwitcher Add CHANGELOG
What is the purpose of this pull request?
Creation of the component
LocaleSwitcher
that allows the user to select the language of the storeThis PR depends of these ones:
vtex-apps/store-components#453
vtex-apps/store-theme#140
How should this be manually tested?
Access this workspace
Screenshots or example usage
Types of changes