?
avatar aaleksanyants
aaleksanyants
7 Jul 2015

This report needs to be checked by more skilled user who knows if where are the following strings displayed. Might be the strings are processed to get displayed as text, but i feel there should be implemented an escape to show them properly, ie. <figure> -> &lt;figure&gt;, <img> -> &lt;img&gt; etc. Here are the strings:

admin/en-GB.com_media.ini COM_MEDIA_ALIGN_DESC "This will apply the classes 'pull-left', 'pull-center' or 'pull-right' to the '<figure>' or '<img>' element."
admin/en-GB.com_media.ini COM_MEDIA_CAPTION_CLASS_DESC "This will apply the entered class to the '<figcaption>' element. For example: 'text-left', 'text-right', 'text-center'."
site/en-GB.com_media.ini COM_MEDIA_ALIGN_DESC "This will apply the classes 'pull-left', 'pull-center' or 'pull-right' to the '<figure>' or '<img>' element."
site/en-GB.com_media.ini COM_MEDIA_CAPTION_CLASS_DESC "This will apply the entered class to the '<figcaption>' element. For example: 'text-left', 'text-right', 'text-center'."

avatar aaleksanyants aaleksanyants - open - 7 Jul 2015
avatar brianteeman
brianteeman - comment - 7 Jul 2015

They are displayed correctly

vge0

avatar zero-24 zero-24 - change - 7 Jul 2015
The description was changed
Labels Added: ?
avatar zero-24 zero-24 - change - 7 Jul 2015
Category Language & Strings
avatar aaleksanyants
aaleksanyants - comment - 8 Jul 2015

I tried to replace with the appropriate HTML entities and they still do. Because there are too many places in language strings with HTML tags inserted just to change the appearance, how correct is to restrict their direct usage in a single module only because to display them as text?

avatar infograf768
infograf768 - comment - 8 Jul 2015

how correct is to restrict their direct usage in a single module only because to display them as text?

Can you explain better what you suggest? The specific ones above are not html tags but pure text.

avatar aaleksanyants
aaleksanyants - comment - 8 Jul 2015

I would suggest to replace <figure> -> &lt;figure&gt;, <img> -> &lt;img&gt; etc.for not to prevent other HTML tags (if any) to serve as HTML tags instead of displaying them as text.

avatar infograf768
infograf768 - comment - 8 Jul 2015

TBH, all has been working fine until now the way we have been doing with the few we have.
Have you remarked other occurences than these few ones?

avatar brianteeman
brianteeman - comment - 8 Jul 2015

If they work (and they do) then we should leave it as it is - it is much
more readable and makes the life of translators easier

On 8 July 2015 at 09:57, infograf768 notifications@github.com wrote:

TBH, all has been working fine until now the way we have been doing with
the few we have.
Have you remarked other occurences than these few ones?


Reply to this email directly or view it on GitHub
#7367 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar Bakual
Bakual - comment - 8 Jul 2015

I would leave them unescaped in the language files as we have currently. It makes translations much easier.
If we need to escape them, we can (and already do) escape them when we need it. PHP has a command for that :smile:

avatar Bakual Bakual - change - 8 Jul 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-07-08 09:32:05
Closed_By Bakual
avatar Bakual Bakual - close - 8 Jul 2015

Add a Comment

Login with GitHub to post a comment