All submissions to this site are governed by the Second Life Viewer Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you have read, understood, and agreed to those terms.

Review Board 1.6.11

Welcome to the Second Life Viewer Code Review tool.
See the documentation on our wiki for how to use this site.

STORM-1404 Light viewer: untranslated buttons in the People panel

Review Request #412 - Created July 22, 2011 and submitted

Vadim ProductEngine Reviewers
viewer
STORM-1404
None viewer-development
Bug reason: incorrect reuse of an xml element name.
Fix: provided unique names for the buttons.

 
Ship it!
Posted (July 25, 2011, 7:14 a.m.)
Looks like a duplicate of https://codereview.secondlife.com/r/387 which deals with STORM-1403, the issue caused by the same widget naming problem.
Posted (July 25, 2011, 9:59 a.m.)

   

  
share_btn doesn't occur (yet?/anymore?) in the English original, so I think it should be removed from the translations, too, to be consistent.

(Ditto for languages other than 'da')
  1. It's up to the translation team to remove obsolete translations.
Ship it!
Posted (July 25, 2011, 10:04 a.m.)
> It's up to the translation team to remove obsolete translations.

In that case, no further concerns. Should we file a jira issue to make the translation team aware of the obsolete elements?
  1. Not sure whether it makes sense. I'll find out.
  2. So, their localization tool does detect obsolete translations, but only when the "en" file changes.
    If you want to get rid of them earlier, please feel free to file a jira.