Skip to content

feat: use the compact version of language select #18

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

Conversation

igobranco
Copy link
Member

Always use the compacted version of the language selection on the header. When Richie changes the language cookie to be 'pt' the MFEs show any language on the header, so it is kind of strange. So now showing the language name on the header is better, it hides the problem and give us more space on the header, but still showning an easy way the learner to change it.

image

Always use the compacted version of the language selection on the
header. When Richie changes the language cookie to be 'pt' the MFEs
show any language on the header, so it is kind of strange. So now
showing the language name on the header is better, it hides the problem
and give us more space on the header, but still showning an easy way the
learner to change it.
@igobranco igobranco requested a review from sandroscosta April 30, 2025 13:43
@igobranco igobranco self-assigned this Apr 30, 2025
@sandroscosta
Copy link

I agree with your logic here. Showing the incorrect language choice might cause confusion.

Copy link

@sandroscosta sandroscosta left a comment

Choose a reason for hiding this comment

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

LGTM

@sandroscosta sandroscosta merged commit 69b9d8d into nau/redwood.master May 2, 2025
4 checks passed
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.

2 participants