Re: [opensource-dev] Review Request: (STORM-433) Friendship offer shifted up and placed over "Second Life" text
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/138/#review339 --- - Paul On Feb. 4, 2011, 10:49 a.m., Seth ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/138/ > --- > > (Updated Feb. 4, 2011, 10:49 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Fixed reshaping notification panel with friendship offer when it is inserted > into IM chat. > > > This addresses bug STORM-433. > http://jira.secondlife.com/browse/STORM-433 > > > Diffs > - > > indra/newview/llchathistory.cpp 2fe9d48e5881 > indra/newview/lltoastnotifypanel.cpp 2fe9d48e5881 > > Diff: http://codereview.secondlife.com/r/138/diff > > > Testing > --- > > > Thanks, > > Seth > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: (STORM-433) Friendship offer shifted up and placed over "Second Life" text
> On Feb. 8, 2011, 8:42 a.m., Paul ProductEngine wrote: > > Looks good. Ship it! - Paul --- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/138/#review339 --- On Feb. 4, 2011, 10:49 a.m., Seth ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/138/ > --- > > (Updated Feb. 4, 2011, 10:49 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Fixed reshaping notification panel with friendship offer when it is inserted > into IM chat. > > > This addresses bug STORM-433. > http://jira.secondlife.com/browse/STORM-433 > > > Diffs > - > > indra/newview/llchathistory.cpp 2fe9d48e5881 > indra/newview/lltoastnotifypanel.cpp 2fe9d48e5881 > > Diff: http://codereview.secondlife.com/r/138/diff > > > Testing > --- > > > Thanks, > > Seth > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: (STORM-433) Friendship offer shifted up and placed over "Second Life" text
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/138/#review341 --- Ship it! - Paul On Feb. 4, 2011, 10:49 a.m., Seth ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/138/ > --- > > (Updated Feb. 4, 2011, 10:49 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Fixed reshaping notification panel with friendship offer when it is inserted > into IM chat. > > > This addresses bug STORM-433. > http://jira.secondlife.com/browse/STORM-433 > > > Diffs > - > > indra/newview/llchathistory.cpp 2fe9d48e5881 > indra/newview/lltoastnotifypanel.cpp 2fe9d48e5881 > > Diff: http://codereview.secondlife.com/r/138/diff > > > Testing > --- > > > Thanks, > > Seth > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
[opensource-dev] Review Request: STORM-680 (Avaline callers are added to the Recent list)
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/146/ --- Review request for Viewer. Summary --- - When accepting an avaline call, add a caller to the recent list as AvalineListItem - When adding item to the LLRecentPeople, check whether item with the same phone number exists and delete it if exists. This is need to avoid duplication in the Recent list of the panel People. This addresses bug STORM-680. http://jira.secondlife.com/browse/STORM-680 Diffs - indra/newview/llavatarlist.h 33fc9ed99d29 indra/newview/llavatarlist.cpp 33fc9ed99d29 indra/newview/llrecentpeople.h 33fc9ed99d29 indra/newview/llrecentpeople.cpp 33fc9ed99d29 indra/newview/llvoicechannel.h 33fc9ed99d29 indra/newview/llvoicechannel.cpp 33fc9ed99d29 Diff: http://codereview.secondlife.com/r/146/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-974 UI button alignment issues in Landmark panel, About Land window, and Hardware Settings window
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/142/#review365 --- Ship it! - Paul On Feb. 8, 2011, 5:22 a.m., Jonathan Yap wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/142/ > --- > > (Updated Feb. 8, 2011, 5:22 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Please see the jira for the referenced pictures. > > In picture #1, in the preferences > graphics > hardware settings, the line of > "gamma" setting is a little too up and it collides with the "antialiasing" > line. > > In picture #2 in the "places" tab under "my landmarks" tab the "profile" > button gets cut off at the end from the rest of the sidebar's UI. > There are a number of other buttons with alignment issues with no screenshot. > > In picture #3 in the Access tab in the About Land window, the remove buttons > are off to the right a few millimetres. > > Changes made were to align the Hardware Settings and About Land buttons and > to completely go over the buttons in the Landmarks panel. The small > down-arrow at the bottom right corner was using an imbedded UTF-8 so I fixed > that and also made it point up while the menu is open (there is a separate > jira for reversing the direction it points in these two states). > > This patch really needs a test viewer built to make sure all works and that > nothing was missed in the Landmarks panel. There are 3 other related jiras > that I hope are taken in; a test viewer could be made incorporating all 4 > jiras at that point. > > > This addresses bug STORM-974. > http://jira.secondlife.com/browse/STORM-974 > > > Diffs > - > > indra/newview/skins/default/xui/en/panel_places.xml b0bceb572090 > > Diff: http://codereview.secondlife.com/r/142/diff > > > Testing > --- > > > Thanks, > > Jonathan > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
[opensource-dev] Review Request: STORM-951 Accidental Self-Muting/Self-Blocking
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/303/ --- Review request for Viewer. Summary --- - To avoid self blocking, add blocked object by its ID. - Before showing offer from object, check whether this object is blocked, to avoid spamming offers from blocked objects. This addresses bug storm-951. http://jira.secondlife.com/browse/storm-951 Diffs - indra/newview/llnotificationofferhandler.cpp 4a7158d7714c indra/newview/llviewermessage.cpp 4a7158d7714c Diff: http://codereview.secondlife.com/r/303/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
[opensource-dev] Review Request: STORM-1460 (Cursor doesn't go to the input field on the Find Floater due to lack of window focus)
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/386/ --- Review request for Viewer. Summary --- Reason: Focus didn't go to the browser Solution: Set the focus to the browser if a page is loaded and a floater containing a browser has a focus This addresses bug storm-1460. http://jira.secondlife.com/browse/storm-1460 Diffs - indra/newview/llfloatersearch.h 68ad362920c1 indra/newview/llfloatersearch.cpp 68ad362920c1 Diff: http://codereview.secondlife.com/r/386/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
[opensource-dev] Review Request: STORM-1503 (Find floater doesn't set focus to its browser)
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/393/ --- Review request for Viewer. Summary --- Just added tab_stop to the browser. This addresses bug storm-1503. http://jira.secondlife.com/browse/storm-1503 Diffs - indra/newview/skins/default/xui/en/floater_search.xml 68ad362920c1 Diff: http://codereview.secondlife.com/r/393/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
[opensource-dev] Review Request: STORM-519 ( "Delete" is enabled in the context menu for folders which contain worn items)
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/396/ --- Review request for Viewer. Summary --- - Disable "Delete" menu item in case selected folder contains non-removable items. This addresses bug storm-519. http://jira.secondlife.com/browse/storm-519 Diffs - indra/newview/llinventorybridge.cpp 68ad362920c1 Diff: http://codereview.secondlife.com/r/396/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1503 (Find floater doesn't set focus to its browser)
> On July 14, 2011, 9:05 a.m., Boroondas Gupte wrote: > > indra/newview/skins/default/xui/en/floater_search.xml, lines 40-43 > > <http://codereview.secondlife.com/r/393/diff/1/?file=6663#file6663line40> > > > > While we're editing around here, fix indentation of line 42. It's too late to fix this indentation because the repo for this fix has already been published. - Paul --- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/393/#review878 --- On July 13, 2011, 8:48 a.m., Paul ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/393/ > --- > > (Updated July 13, 2011, 8:48 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Just added tab_stop to the browser. > > > This addresses bug storm-1503. > http://jira.secondlife.com/browse/storm-1503 > > > Diffs > - > > indra/newview/skins/default/xui/en/floater_search.xml 68ad362920c1 > > Diff: http://codereview.secondlife.com/r/393/diff > > > Testing > --- > > > Thanks, > > Paul > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-519 ( "Delete" is enabled in the context menu for folders which contain worn items)
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/396/ --- (Updated July 19, 2011, 11:11 a.m.) Review request for Viewer. Summary --- - Disable "Delete" menu item in case selected folder contains non-removable items. This addresses bug storm-519. http://jira.secondlife.com/browse/storm-519 Diffs (updated) - indra/newview/llinventorybridge.cpp 7c7d01be2310 Diff: http://codereview.secondlife.com/r/396/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-519 ( "Delete" is enabled in the context menu for folders which contain worn items)
> On July 14, 2011, 9:03 a.m., Boroondas Gupte wrote: > > indra/newview/llinventorybridge.cpp, line 2502 > > <http://codereview.secondlife.com/r/396/diff/1/?file=6693#file6693line2502> > > > > We could just use > > if (!isItemRemovable()) > > here, couldn't we? You are right. Changed according to your commentaries. - Paul --- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/396/#review877 --- On July 19, 2011, 11:11 a.m., Paul ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/396/ > --- > > (Updated July 19, 2011, 11:11 a.m.) > > > Review request for Viewer. > > > Summary > --- > > - Disable "Delete" menu item in case selected folder contains non-removable > items. > > > This addresses bug storm-519. > http://jira.secondlife.com/browse/storm-519 > > > Diffs > - > > indra/newview/llinventorybridge.cpp 7c7d01be2310 > > Diff: http://codereview.secondlife.com/r/396/diff > > > Testing > --- > > > Thanks, > > Paul > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
[opensource-dev] Review Request: STORM-1474 Implement Show Last Search in viewer chrome
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/402/ --- Review request for Viewer. Summary --- - Removed hiding floater on teleport This addresses bug storm-1474. http://jira.secondlife.com/browse/storm-1474 Diffs - indra/newview/llagent.cpp d8c37b383028 Diff: http://codereview.secondlife.com/r/402/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
[opensource-dev] Review Request: STORM-1458 ([crashhunters] crash at LLParticipantList::LLParticipantListMenu::isGroupModerator() [secondlife-bin llparticipantlist.cpp])
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/404/ --- Review request for Viewer. Summary --- - Added null checking This addresses bug storm-1458. http://jira.secondlife.com/browse/storm-1458 Diffs - indra/newview/llparticipantlist.cpp d8c37b383028 Diff: http://codereview.secondlife.com/r/404/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1458 ([crashhunters] crash at LLParticipantList::LLParticipantListMenu::isGroupModerator() [secondlife-bin llparticipantlist.cpp])
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/404/ --- (Updated July 21, 2011, 3:27 a.m.) Review request for Viewer. Changes --- Updated diff according to Boroondas' commentaries Summary --- - Added null checking This addresses bug storm-1458. http://jira.secondlife.com/browse/storm-1458 Diffs (updated) - indra/newview/llparticipantlist.cpp d8c37b383028 Diff: http://codereview.secondlife.com/r/404/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1458 ([crashhunters] crash at LLParticipantList::LLParticipantListMenu::isGroupModerator() [secondlife-bin llparticipantlist.cpp])
> On July 21, 2011, 4:21 a.m., Boroondas Gupte wrote: > > indra/newview/llparticipantlist.cpp, lines 807-808 > > <http://codereview.secondlife.com/r/404/diff/2/?file=6722#file6722line807> > > > > Hmm ... looking at the comment at > > https://bitbucket.org/lindenlab/viewer-development/src/d8c37b383028/indra/newview/llagent.h#cl-705 > > and the implementation of isInGroup at > > https://bitbucket.org/lindenlab/viewer-development/src/d8c37b383028/indra/newview/llagent.cpp#cl-2414 > > , this test actually checks whether the agent in question is a member of > > the group which has an UUID matching the conversation's session ID. > > > > Because the session IDs of group chats will match the UUID of the > > respective groups (and all other chat session types should have session IDs > > not matching any group UUIDs), this also tells us whether the session is a > > group chat. > > > > So maybe the comment should be yet different to reflect this. (Sorry, > > should have looked up this stuff for the first review already.) Maybe: > > // Is session a group chat? (It is if the session ID is the > > UUID of a group of which the agent is a member.) > > > > I wonder whether there is a better way (as in: making less assumptions > > / not using an agent as pivot) to test for this. I think the comment may be: "Is agent in group call/chat ?" - Paul --- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/404/#review898 --- On July 21, 2011, 3:27 a.m., Paul ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/404/ > --- > > (Updated July 21, 2011, 3:27 a.m.) > > > Review request for Viewer. > > > Summary > --- > > - Added null checking > > > This addresses bug storm-1458. > http://jira.secondlife.com/browse/storm-1458 > > > Diffs > - > > indra/newview/llparticipantlist.cpp d8c37b383028 > > Diff: http://codereview.secondlife.com/r/404/diff > > > Testing > --- > > > Thanks, > > Paul > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1458 ([crashhunters] crash at LLParticipantList::LLParticipantListMenu::isGroupModerator() [secondlife-bin llparticipantlist.cpp])
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/404/ --- (Updated July 21, 2011, 7:01 a.m.) Review request for Viewer. Summary --- - Added null checking This addresses bug storm-1458. http://jira.secondlife.com/browse/storm-1458 Diffs (updated) - indra/newview/llparticipantlist.cpp d8c37b383028 Diff: http://codereview.secondlife.com/r/404/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1474 Implement Show Last Search in viewer chrome
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/402/ --- (Updated July 22, 2011, 5:10 a.m.) Review request for Viewer. Changes --- Updated diff according to the new acceptance criteria of this issue. Now floater should be minimized on teleport. Summary (updated) --- - Minimize the search floater on teleport This addresses bug storm-1474. http://jira.secondlife.com/browse/storm-1474 Diffs (updated) - indra/newview/llagent.cpp 216ebc0442d2 Diff: http://codereview.secondlife.com/r/402/diff Testing --- Thanks, Paul ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1585 Duplicated XUI ID: floater_preferences_proxy.xml
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/467/#review1021 --- Ship it! Looks good to me. - Paul On Sept. 12, 2011, 5:50 p.m., Vadim ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/467/ > --- > > (Updated Sept. 12, 2011, 5:50 p.m.) > > > Review request for Viewer and Log Linden. > > > Summary > --- > > Fixed duplicated/missing XUI IDs in floater_preferences_proxy.xml. > > > This addresses bug STORM-1585. > http://jira.secondlife.com/browse/STORM-1585 > > > Diffs > - > > indra/newview/skins/default/xui/en/floater_preferences_proxy.xml > 25b876619ca6 > > Diff: http://codereview.secondlife.com/r/467/diff > > > Testing > --- > > > Thanks, > > Vadim > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1595 Crash in developer builds when changing selection in detached sidebar inventory panel
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/469/#review1022 --- Ship it! - Paul On Sept. 13, 2011, 7:26 a.m., Vadim ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/469/ > --- > > (Updated Sept. 13, 2011, 7:26 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Removing a wrong debug assertion. > > > This addresses bug STORM-1595. > http://jira.secondlife.com/browse/STORM-1595 > > > Diffs > - > > indra/newview/llsidetray.cpp 25b876619ca6 > > Diff: http://codereview.secondlife.com/r/469/diff > > > Testing > --- > > > Thanks, > > Vadim > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1597 Inventory Window "object profile" warped when Inventory Window is a resized floater
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/470/#review1023 --- Ship it! Looks good! - Paul On Sept. 13, 2011, 8:05 a.m., Vadim ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/470/ > --- > > (Updated Sept. 13, 2011, 8:05 a.m.) > > > Review request for Viewer. > > > Summary > --- > > The Back button int the object profile panel now sticks to the left. > > > This addresses bug STORM-1597. > http://jira.secondlife.com/browse/STORM-1597 > > > Diffs > - > > indra/newview/skins/default/xui/en/sidepanel_task_info.xml 25b876619ca6 > > Diff: http://codereview.secondlife.com/r/470/diff > > > Testing > --- > > > Thanks, > > Vadim > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1599 Unexpected camera zoom occurs if scroll search history drop-down list by mouse wheel
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/471/#review1024 --- Ship it! Good! - Paul On Sept. 13, 2011, 8:44 a.m., Vadim ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/471/ > --- > > (Updated Sept. 13, 2011, 8:44 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Trivial fix: don't pass mouse wheel events to parent views. > > > This addresses bug STORM-1599. > http://jira.secondlife.com/browse/STORM-1599 > > > Diffs > - > > indra/newview/skins/default/xui/en/panel_navigation_bar.xml 25b876619ca6 > > Diff: http://codereview.secondlife.com/r/471/diff > > > Testing > --- > > > Thanks, > > Vadim > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1666 Redundant "Release Notes" text in the About window for non-English locales
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/505/#review1067 --- Ship it! - Paul On Oct. 25, 2011, 10:06 a.m., Vadim ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/505/ > --- > > (Updated Oct. 25, 2011, 10:06 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Ported the XML changes made for STORM-1611 to other locales. > > > This addresses bug STORM-1666. > http://jira.secondlife.com/browse/STORM-1666 > > > Diffs > - > > indra/newview/skins/default/xui/da/floater_about.xml 08e65f3ead3d > indra/newview/skins/default/xui/es/floater_about.xml 08e65f3ead3d > indra/newview/skins/default/xui/ja/floater_about.xml 08e65f3ead3d > indra/newview/skins/default/xui/pl/floater_about.xml 08e65f3ead3d > indra/newview/skins/default/xui/ru/floater_about.xml 08e65f3ead3d > indra/newview/skins/default/xui/tr/floater_about.xml 08e65f3ead3d > indra/newview/skins/default/xui/zh/floater_about.xml 08e65f3ead3d > > Diff: http://codereview.secondlife.com/r/505/diff > > > Testing > --- > > > Thanks, > > Vadim > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1667 Square brackets around of Release Notes link in the About window
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/506/#review1068 --- Ship it! - Paul On Oct. 25, 2011, 10:28 a.m., Vadim ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/506/ > --- > > (Updated Oct. 25, 2011, 10:28 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Adding back code that was accidentally removed in changeset ff333a95d1aa (the > fix of STORM-1611). > > > This addresses bug STORM-1667. > http://jira.secondlife.com/browse/STORM-1667 > > > Diffs > - > > indra/newview/llfloaterabout.cpp 08e65f3ead3d > > Diff: http://codereview.secondlife.com/r/506/diff > > > Testing > --- > > > Thanks, > > Vadim > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
Re: [opensource-dev] Review Request: STORM-1668 Localized strings.xml contains reference to non-existing gesture "bow1"
--- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/507/#review1069 --- Ship it! - Paul On Oct. 25, 2011, 10:37 a.m., Vadim ProductEngine wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/507/ > --- > > (Updated Oct. 25, 2011, 10:37 a.m.) > > > Review request for Viewer. > > > Summary > --- > > Fixed references to nonexistent gesture /bow1 in translations. > > > This addresses bug STORM-1668. > http://jira.secondlife.com/browse/STORM-1668 > > > Diffs > - > > indra/newview/skins/default/xui/es/strings.xml 08e65f3ead3d > indra/newview/skins/default/xui/ja/strings.xml 08e65f3ead3d > indra/newview/skins/default/xui/ru/strings.xml 08e65f3ead3d > indra/newview/skins/default/xui/tr/strings.xml 08e65f3ead3d > > Diff: http://codereview.secondlife.com/r/507/diff > > > Testing > --- > > > Thanks, > > Vadim > > ___ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
