diff options
| author | andreykproductengine <akleshchev@productengine.com> | 2016-12-28 19:58:32 +0200 | 
|---|---|---|
| committer | andreykproductengine <akleshchev@productengine.com> | 2016-12-28 19:58:32 +0200 | 
| commit | 6c023787cdee1ab80a6306fa8ae61e01cba84457 (patch) | |
| tree | d951a76d6786427703a009f2a4d8e9a782e30254 | |
| parent | c79dc296adf4c3fca25b587805739a428f507f81 (diff) | |
MAINT-2450 PRIM_MEDIA_PERMS_CONTROL shouldn't disable mouse wheel scrolling
| -rw-r--r-- | indra/newview/llpanelprimmediacontrols.cpp | 11 | 
1 files changed, 8 insertions, 3 deletions
| diff --git a/indra/newview/llpanelprimmediacontrols.cpp b/indra/newview/llpanelprimmediacontrols.cpp index 0bcd8a9e63..5f413fc3c0 100644 --- a/indra/newview/llpanelprimmediacontrols.cpp +++ b/indra/newview/llpanelprimmediacontrols.cpp @@ -566,11 +566,16 @@ void LLPanelPrimMediaControls::updateShape()  			}  		} -		// MAINT-1392 If this is a HUD always set it visible, but hide each control if user has no perms. -		// When setting it invisible it won't receive any mouse messages anymore +		// Web plugins and HUD may have media controls invisible for user, but still need scroll mouse events. +		// LLView checks for visibleEnabledAndContains() and won't pass events to invisible panel, so instead +		// of hiding whole panel hide each control instead (if user has no perms). +		// Note: It might be beneficial to keep panel visible for all plugins to make behavior consistent, but  +		// for now limiting change to cases that need events. -		if( !is_hud ) +		if (!is_hud && (!media_plugin || media_plugin->pluginSupportsMediaTime())) +		{  			setVisible(enabled); +		}  		else  		{  			if( !hasPermsControl ) | 
