This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110629/

shells/newshell/package/contents/ui/mediacenter.qml (Diff revision 1) 

    140
    
            Keys.onEscapePressed: {mediaBrowser.visible=true

  


 





there should be a newline after '{'














 
  

   
    shells/newshell/package/contents/ui/mediacenter.qml
    

     (Diff revision 1)

    
   
  

 



 
 

 

  

    
    

    141
    
            mediaImageViewer.dimImage();

  


 





missing indentation









- Shantanu






On May 24th, 2013, 7:46 a.m. UTC, Akshay Ratan wrote:











 

  


Review request for Plasma, Shantanu Tushar and Sinny Kumari.


By Akshay Ratan.




Updated May 24, 2013, 7:46 a.m.








Description 

 

 

  
   
Dimming the image when it is not viewed directly or more simply when it shows up in background while navigating in the Plasma Media Center.

The existing minor problem was that when user views a picture in a slideshow and then navigate to the main screen or one level up, the image is shown in the same opacity as the original in the background which causes a little discomfort in seeing the gallery and other written texts on screen.

  		
 






Testing 



 

  
   
Works as intended.

  		
 








Diffs 


    

 
  • shells/newshell/package/contents/ui/mediacenter.qml (ef71938)
    • 





View Diff