- 🇮🇳India urvashi_vora Madhya Pradesh, India
Hi,
I have taken a pull of your branch and reviewed the changes.The variable "id" is successfully replaced by "count". The changes look great to be merged.
RTBC++
Thanks for the work.
- Status changed to Closed: works as designed
over 1 year ago 7:21am 23 February 2023 - 🇨🇭Switzerland berdir Switzerland
Well, no, it actually does not represent the count, it really is an ID. we call end() on the ids, which returns the last one in there.
It's a horrible message and should be replaced with the label or something else, but it's technically correct.
- Status changed to Needs work
over 1 year ago 4:35pm 23 February 2023 - 🇭🇺Hungary Balu Ertl Budapest 🇪🇺
Thanks to both of you for your time dealing with this issue. However, I have the feeling that the initial pain source had not been addressed at all:
“[…] Please help translator folks' work by naming variables more meaningfully.”
@berdir could you please elaborate on the direction in more detail of what piece of the code shall we modify to make the UI string of this message less horrible?