Merge pull request #1463 from jsit/feat/default-to-user-primary-lang

feat: Default language dropdowns to user's interface language
This commit is contained in:
Jay Sitter 2023-06-24 16:30:24 -04:00 committed by GitHub
commit dd43827d67
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 34 additions and 6 deletions

View file

@ -1,4 +1,5 @@
import { myAuthRequired } from "@utils/app"; import { myAuthRequired } from "@utils/app";
import getUserInterfaceLangId from "@utils/app/user-interface-language";
import { capitalizeFirstLetter } from "@utils/helpers"; import { capitalizeFirstLetter } from "@utils/helpers";
import { Component } from "inferno"; import { Component } from "inferno";
import { T } from "inferno-i18next-dess"; import { T } from "inferno-i18next-dess";
@ -40,6 +41,8 @@ export class CommentForm extends Component<CommentFormProps, any> {
: undefined : undefined
: undefined; : undefined;
const userInterfaceLangId = getUserInterfaceLangId(this.props.allLanguages);
return ( return (
<div <div
className={["comment-form", "mb-3", this.props.containerClass].join( className={["comment-form", "mb-3", this.props.containerClass].join(
@ -49,6 +52,7 @@ export class CommentForm extends Component<CommentFormProps, any> {
{UserService.Instance.myUserInfo ? ( {UserService.Instance.myUserInfo ? (
<MarkdownTextArea <MarkdownTextArea
initialContent={initialContent} initialContent={initialContent}
initialLanguageId={userInterfaceLangId}
showLanguage showLanguage
buttonTitle={this.buttonTitle} buttonTitle={this.buttonTitle}
finished={this.props.finished} finished={this.props.finished}

View file

@ -49,7 +49,7 @@ export class LanguageSelect extends Component<LanguageSelectProps, any> {
return this.props.iconVersion ? ( return this.props.iconVersion ? (
this.selectBtn this.selectBtn
) : ( ) : (
<div className="language-select mb-3"> <div className="language-select row mb-3">
<label <label
className={classNames( className={classNames(
"col-form-label", "col-form-label",

View file

@ -260,8 +260,12 @@ export class MarkdownTextArea extends Component<
<LanguageSelect <LanguageSelect
iconVersion iconVersion
allLanguages={this.props.allLanguages} allLanguages={this.props.allLanguages}
// Only set the selected language ID if it exists as an option
// in the dropdown; otherwise, set it to 0 (Undetermined)
selectedLanguageIds={ selectedLanguageIds={
languageId ? Array.of(languageId) : undefined languageId && this.props.siteLanguages.includes(languageId)
? [languageId]
: [0]
} }
siteLanguages={this.props.siteLanguages} siteLanguages={this.props.siteLanguages}
onChange={this.handleLanguageChange} onChange={this.handleLanguageChange}

View file

@ -4,6 +4,7 @@ import {
myAuth, myAuth,
myAuthRequired, myAuthRequired,
} from "@utils/app"; } from "@utils/app";
import getUserInterfaceLangId from "@utils/app/user-interface-language";
import { import {
capitalizeFirstLetter, capitalizeFirstLetter,
debounce, debounce,
@ -323,11 +324,10 @@ export class PostForm extends Component<PostFormProps, PostFormState> {
} }
render() { render() {
const firstLang = this.state.form.language_id;
const selectedLangs = firstLang ? Array.of(firstLang) : undefined;
const url = this.state.form.url; const url = this.state.form.url;
const userInterfaceLangId = getUserInterfaceLangId(this.props.allLanguages);
return ( return (
<form className="post-form" onSubmit={linkEvent(this, handlePostSubmit)}> <form className="post-form" onSubmit={linkEvent(this, handlePostSubmit)}>
<NavigationPrompt <NavigationPrompt
@ -494,8 +494,8 @@ export class PostForm extends Component<PostFormProps, PostFormState> {
</div> </div>
<LanguageSelect <LanguageSelect
allLanguages={this.props.allLanguages} allLanguages={this.props.allLanguages}
selectedLanguageIds={[userInterfaceLangId]}
siteLanguages={this.props.siteLanguages} siteLanguages={this.props.siteLanguages}
selectedLanguageIds={selectedLangs}
multiple={false} multiple={false}
onChange={this.handleLanguageChange} onChange={this.handleLanguageChange}
/> />

View file

@ -53,6 +53,7 @@ import showScores from "./show-scores";
import siteBannerCss from "./site-banner-css"; import siteBannerCss from "./site-banner-css";
import updateCommunityBlock from "./update-community-block"; import updateCommunityBlock from "./update-community-block";
import updatePersonBlock from "./update-person-block"; import updatePersonBlock from "./update-person-block";
import getUserInterfaceLangId from "./user-interface-language";
export { export {
buildCommentsTree, buildCommentsTree,
@ -87,6 +88,7 @@ export {
getIdFromProps, getIdFromProps,
getRecipientIdFromProps, getRecipientIdFromProps,
getUpdatedSearchId, getUpdatedSearchId,
getUserInterfaceLangId,
initializeSite, initializeSite,
insertCommentIntoTree, insertCommentIntoTree,
isAuthPath, isAuthPath,

View file

@ -0,0 +1,18 @@
import { Language } from "lemmy-js-client";
import { I18NextService } from "../../services/I18NextService";
export default function getUserInterfaceLangId(
allLanguages: Language[]
): number {
// Get the string of the browser- or user-defined language, like en-US
const i18nLang = I18NextService.i18n.language;
// Find the Language object with a code that matches the initial characters of
// this string
const userLang = allLanguages.find(lang => {
return i18nLang.indexOf(lang.code) === 0;
});
// Return the ID of that language object, or "0" for Undetermined
return userLang?.id || 0;
}