Wrap Question Number in span tag

[valiceops]

Steps to Reproduce

Quiz questions are no longer translating using TranslatePress so the request is to wrap the question number within the h2 heading that outputs the question title, in a span tag.

We are using a translation plugin (TranslatePress) which is fairly popular. We ran into an issue which, in the interest of time, we had just resolved by modifying the /woothemes-sensei/plugins/sensei-lms/includes/class-sensei-question.php file to wrap the output of the question number ($title_html .= sprintf( esc_html__( '%d. ', 'sensei-lms' ), sensei_get_the_question_number() );) in a span tag.

The reason for the request is that the translation plugin identifies the entire content, including the number, as a string so it wasn't finding our 100+ question translations because prior to a year or so I go I guess now, Sensei didn't output the question numbers. We also, FYI use the random order feature so it would be very daunting to re-translate every possible combination of numbers and questions in the translation file/plugin.

PHP / WordPress / Sensei LMS version

WP 6.5.4 / Sensei LMS 4.24.1

[valiceops]

Just checking in to see if there is any movement here. It's a pretty small change, and doesn't seem like this request would have any impact on the majority of users but would make it compatible with TranslatePress and randomly ordering quiz questions. Or is there at least some other suggestion besides modifying the core template file?

[donnapep]

@valiceops Hello, and thanks for your inquiry. Would you mind confirming whether Learning Mode is enabled on your site?

Note that there is currently a Quiz question points format setting that can be set to Don't show quiz question points. Unfortunately, that setting doesn't appear to be working. If it was working, would that meet your needs?

Please ignore. I just realized that setting does not control the question number format, but rather the points format.

[valiceops]

Hi @donnapep - Yes, we are using Learner Mode

[donnapep]

@valiceops Thanks. Can you confirm that #7738 will resolve the TranslatePress issue?

[valiceops]

I wasn't able to load the playground but viewing the commit, yes, this is what we have in our override and works for translating the randomly ordered questions. Thank you!