J4 Issue ?
avatar brianteeman
brianteeman
5 Sep 2018

The file \layouts\joomla\toolbar\dropdown.php has several hard coded strings that should be translatable

Note it is not just as simple as creating a new string for the text that is there because the text is wrong

avatar brianteeman brianteeman - open - 5 Sep 2018
avatar joomla-cms-bot joomla-cms-bot - change - 5 Sep 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 5 Sep 2018
avatar brianteeman brianteeman - change - 5 Sep 2018
Labels Added: J4 Issue
avatar brianteeman brianteeman - labeled - 5 Sep 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Sep 2018
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Sep 2018
Category Administration
avatar HLeithner
HLeithner - comment - 8 Sep 2018

I can't find an usage of this layout in core. Can you point me in a direction?

avatar brianteeman
brianteeman - comment - 8 Sep 2018

image

avatar Didldu-Florian
Didldu-Florian - comment - 8 Sep 2018

Oh okay, yes, it's not translated and even not displayed.
I would suggest to remove these whole unnecessary line including associaded CSS

<span class="sr-only">Toggle Dropdown</span>

avatar HLeithner
HLeithner - comment - 8 Sep 2018

some lines above is a aria-label:

<div id="<?php echo $id; ?>" class="btn-group dropdown-<?php echo $name ?? ''; ?>" role="group" aria-label="Button Dropdown">

What would you suggest to have as text brian?

avatar Didldu-Florian
Didldu-Florian - comment - 8 Sep 2018

"Toggle Dropdown" should better be placed as title attribute for the button itself ;)

avatar HLeithner
HLeithner - comment - 8 Sep 2018

I think that "Toggle Dropdown" is pretty useless as aria label

avatar Didldu-Florian
Didldu-Florian - comment - 8 Sep 2018

One question: Do we have a prefered language key prefix for layouts?

avatar HLeithner
HLeithner - comment - 8 Sep 2018

I think this text has to be set by the calling component.

avatar brianteeman brianteeman - change - 14 Oct 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-10-14 18:55:12
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 14 Oct 2018

Closed as resolved by #22429

avatar brianteeman brianteeman - close - 14 Oct 2018

Add a Comment

Login with GitHub to post a comment