Opened 9 years ago

Closed 9 years ago

#3215 closed enhancement (fixed)

TA i18n enhancement

Reported by: cjl Owned by: walter
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: Turtleart Version: Unspecified
Severity: Unspecified Keywords:
Cc: Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

plugins/turtle_blocks_extras/turtle_blocks_extras.py:596

Please change

"full screen"

to "fullscreen" or better yet "Fullscreen"

to harmonize with most other activities.

Change History (3)

comment:1 Changed 9 years ago by walter

_('Fullscreen').lower() would mean that we could reuse the 'Fullscreen' string already used by the Button and still have a lowercase name consistent with the other blocks. I believe that .lower() is a NOP when there is no distinction between upper- and lowercase?

I might be able to employ the same trick to a handful of other strings, but it is probably not worthwhile given the scale of the i18n effort for TA.

comment:2 Changed 9 years ago by cjl

I think I would suggest just removing the space in "full screen" to "fullscreen", as the effects of using character operations in code may not have the expected effects in languages that do not make case distinctions. As a practical matter, I think Pootle fuzzy matching from the Terminology will ignore the case distinction.

This raises the issue of whether using upper and lower case distinctions in the UI is a good i18n practice at all. I have no idea whether CJK langs or the Tifinagh alphabet used for Tamazight contain such distinctions at all.

https://secure.wikimedia.org/wikipedia/en/wiki/Tifinagh

comment:3 Changed 9 years ago by walter

  • Resolution set to fixed
  • Status changed from new to closed

Change included in v123

Note: See TracTickets for help on using tickets.