----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/65/ -----------------------------------------------------------
Review request for Viewer. Summary ------- This change enables the "+" menu in Inventory/Recent It grays out "New Folder" in this menu It enables identical menu entries when you right click on an inventory item. Question: Is graying out "New Folder" best done where I am doing it now -- in llpanelmaininventory.cpp / LLPanelMainInventory::onAddButtonClick() This addresses bug storm-737. http://jira.secondlife.com/browse/storm-737 Diffs ----- doc/contributions.txt e843e274fa58 indra/newview/llinventorybridge.cpp e843e274fa58 indra/newview/llpanelmaininventory.cpp e843e274fa58 Diff: http://codereview.secondlife.com/r/65/diff Testing ------- I opened up Inventory/My Inventory and used all the "New xxx" options for both right clicking on an inventory item and also from the "+" menu. I then changed to the Recent tab and performed the same steps. New items were created as expected, except "New Folder" was not an option via either method when the Recent tab was active. 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