-
Notifications
You must be signed in to change notification settings - Fork 40
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
Views config files do not have any _config_translatables
specified to allow for things like the views label and description to be translated
#6608
Comments
Caution! Views does "its own thing" with translations. There's a views setting to enable translation of certain strings (on by default) admin/structure/views/settings/advanced So, things like views titles are always translatable out of the box. This has nothing to do with config translation. |
Yup 👍🏼 ...I made sure to check that on a vanilla Backdrop site, and it is set to "Core" by default.
Nope 👎🏼 ...I've tested that in great detail over in #6610, and that is not the case at all. Here's a simpler way to reproduce this:
@indigoxela if what you say about views strings always being translatable, then shouldn't the labels of the |
...if I'm doing this whole thing wrong, then I'm happy to be shown how it should be done properly. The goal is to be able to translate views labels/descriptions for starters. |
@klonos multilingual can be a bit tricky for starters. There's multiple ways, how Locale picks up strings. The one that applies to views is visiting the view via UI. It's also necessary to have more than one language enabled on a site. Otherwise views_plugin_localization_core::save_string won't add the string and Locale won't be aware of it. |
Thank you for providing all that insight @indigoxela ...it saves me a lot of time 🙏🏼
That's very interesting 🤔 @laryn mentioned the same in the Zulip thread:
So I tested that, and it still doesn't work for me. I visited the Views listing page, the configuration forms for the
Yes 👍🏼 ...my initial testing was done with only English, but I have also enabled other languages after that and tested again and again. I currently have for example English, Catalan and Greek enabled + imported their most recent .po files from https://localize.backdropcms.org. Still, the views labels are not showing up for me 🤷🏼 Specifically about the fact that you need more than one language, I agree, but we are not making that clear. We are showing elements in the UI that we should not be showing when only one language is enabled, but I am leaving all that for #6610 - I don't want to derail this issue here with all that. |
FTR, I just jumped in here for a short hint, how to trigger string addition in Locale. Neither do I want to go further into details, nor do I want to start (or get involved in) any issue re rewriting our multilingual system. 😆 Just a short hint... and gone again. 👋 |
Thanks again @indigoxela 🙏🏼 👋🏼 FTR (for others that come to this thread here): all I'm trying to do is to translate the name of even one single view. At the moment, I cannot do that. I have explained the steps that I am following. If I am doing something wrong, then please let me know what that is. I think that I have followed the right steps, which is now driving me crazy. So some sanity check please. |
@klonos I've checked the issue, and indeed I wasn't able to translate the view name or a views display name. Other strings are however translatable after adding them to a view. For instance:
|
Thanks for taking the time to test and confirm @olafgrabienski 🙏🏼 ...can you please clarify what you mean by this?:
...I'm assuming that by one of them you mean the "Human-readable name" field, but what is the other thing that is not translated? |
@klonos Sure, as soon you have more displays than the Default display, the displays get a name, e.g. Block, Feed etc. See screenshot below. Display name (editing in the modal): ![]() |
Thanks for clarifying @olafgrabienski 🙏🏼 ...one of the bullet points in your earlier reply though is "Display title", which you mention as being translatable. What is that then? |
The discussion around this was brought up in #4137, which is to discuss whether a specific translation-related commit in Views 7.x needs to be crossposted or not.
This was also discussed during this week's dev meeting, as part of the discussion around other issues (like #6511 and #6512). It was 40:13 into the recording. Here: https://youtu.be/CTPd63wSpAg?t=2413
During the meeting, we decided that it would be better if we created a separate issue and PR for the following:
_config_translatables
with the minimum being the strings for the views label, the display labels etc.The text was updated successfully, but these errors were encountered: