Hi,
I’m on the team that deals with WordPress translations to Swedish. This theme isn’t yet translated to Swedish.
I can see that you’ve suggested one string. However, in order for the language pack to be generated, at least 95% of the strings in the project need to be translated (and the translations need to be approved by me or another translation editor for Swedish.
We’ve got a style guide for WordPress translations to Swedish. There you’ll also find some advice on how to avoid various possible problems in the process. https://wpsv.se/engagera-dig/stilguide-for-oversattningar/
I noticed the theme uses TGMA. I’ve imported translations from another theme that also uses it, so now some 60% are done. I’ll try to fix the rest by tomorrow-ish.
Notes to theme developer:
1. Please change
echo wp_kses_post( $before . esc_html__('Posts by ', 'minimal-portfolio') . $userdata->display_name . $after );
to use printf() and change the string to “Posts by %s”.
2. Google Plus is closed.
3. In https://themes.trac.ww.wp.xz.cn/browser/minimal-portfolio/1.0.7/inc/template-tags.php?marks=48#L48
$comments = $num_comments . __( ' Comments', 'minimal-portfolio' );
You should use printf() and _n()
Check in one of the Twenty-something themes how it’s better done.
Yeah, I see that it’s an /inc/ path, but perhaps you can report it upstream?
Thread Starter
kebbon
(@kebbon)
Thanks Tor-Bjorn, you are a hero! 🙂
Now it seems like all the errors are gone, except two:
– The “Details” button in the portfolio section isn’t translated.
– The “Related Projects” headline on the porfolio pages isn’t translated.
My guess is that this problem derives from the Filterable Portfolio plugin, rather than from the theme. There are fields in the portfolio settings for changing/translating these phrases, but they don’t seem to work.
What is shown on the public side of your site (in addition to your actual content) comes from WordPress core (to a minimal extent), your theme, but also any plugin, if it adds any visible content to the front-end.
Your guess is probably correct, but without any link to your site, I can’t really double-check in any way.
Thread Starter
kebbon
(@kebbon)
I see, the link to the site is:
http://textlisa.se
Yes, based on the CSS class appointed to those buttons, I’m quite sure that you’re correct. I’ve started looking into the Swedish translation of that plugin, but there still is some work left there before a language pack can be generated.
Looking at the translation strings for the plugin, it seems as if you’re able to do your own customization of the button text in the settings for that plugin. Even if the plugin itself gets translated, the default values in English may still remain from when you activated the plugin.
Thread Starter
kebbon
(@kebbon)
Yes, that’s correct. Unfortunately, all changes made in the settings doesn’t apply on the public site. Most of them do, but not the “Details” button and not the “Related Projects” headline.
Thread Starter
kebbon
(@kebbon)
Now I saw there was an update available with a new translation for the plugin, so I installed it – and now the mentioned phrases are translated correctly. I’m guessing you are the man behind this, Tor-Bjorn – so thank you!
However, another problem appeared. Suddenly, the portfolio images aren’t clickable on my mobile device. It works fine on a desktop, but not on my iPhone. I have no idea whether this has anything to do with the update, but it appeared after updating.
Any clues on what might be the problem?
(Works fine in Chrome on my Android.)
We’ve now left the initial topic of your post.
I suggest you open a new post.
Possibly in https://ww.wp.xz.cn/support/plugin/filterable-portfolio/ but probably better to slant it more towards troubleshooting this, since that plugin hasn’t been updated for 4 months. Then I’d suggest https://ww.wp.xz.cn/support/forum/how-to-and-troubleshooting/ instead.
Hi,
@tobifjellner
1. Please change
echo wp_kses_post( $before . esc_html__(‘Posts by ‘, ‘minimal-portfolio’) . $userdata->display_name . $after );
to use printf() and change the string to “Posts by %s”. —> Updated in new version
2. Google Plus is closed –> Removed in new version
3. In https://themes.trac.ww.wp.xz.cn/browser/minimal-portfolio/1.0.7/inc/template-tags.php?marks=48#L48
$comments = $num_comments . __( ‘ Comments’, ‘minimal-portfolio’ );
You should use printf() and _n() –> Updated in new version
@kebbon : Fixed all issues and relased new version.
Thanks,
Abileweb
