We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
in core_renderer.php on function render_custom_menu_item. the class is set to be the custom_menu_item title Line: 277 $class = $menunode->get_title();
$class = $menunode->get_title();
that title might be in a foreign language
The text was updated successfully, but these errors were encountered:
that since it takes the class name from the title, the class name might be in a foreign lang as well, that's the part that creates problems.
On Wed, Mar 9, 2016 at 6:51 PM, Gareth J Barnard [email protected] wrote:
It might be, but as your screen shot clearly shows, the 'title' attribute is set correctly to the different language. So, what's the problem? — Reply to this email directly or view it on GitHub #410 (comment) .
It might be, but as your screen shot clearly shows, the 'title' attribute is set correctly to the different language. So, what's the problem?
— Reply to this email directly or view it on GitHub #410 (comment) .
Sorry, something went wrong.
So what do you suggest is the solution?
Trying to add more informative classes to Moodle's menus (and tree elements) so I can compose needed UI user tours, using: https://moodle.org/plugins/local_usertours I created https://tracker.moodle.org/browse/MDL-55250 And suggested patch: moodle/moodle@master...nadavkav:MDL-55250_master Please see if it helps with this issue and also any feedback would be very much appriciated
No branches or pull requests
in core_renderer.php on function render_custom_menu_item.
the class is set to be the custom_menu_item title
Line: 277
$class = $menunode->get_title();
that title might be in a foreign language
The text was updated successfully, but these errors were encountered: