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> -> <figure>, <img> -> <img> 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'."
Labels |
Added:
?
|
Category | ⇒ | Language & Strings |
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?
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.
I would suggest to replace <figure> -> <figure>, <img> -> <img> etc.for not to prevent other HTML tags (if any) to serve as HTML tags instead of displaying them as text.
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?
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/
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
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-07-08 09:32:05 |
Closed_By | ⇒ | Bakual |
They are displayed correctly