Closed Bug 462294 Opened 16 years ago Closed 16 years ago

Add "View Video" to context menu for <video> elements

Categories

(Firefox :: Menus, defect)

Product:
Firefox
Component:
Menus
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 3.1b2

People

(Reporter: Dolske, Assigned: Dolske)

References

Details

Attachments

(1 file)

Patch v.1
4.21 KB, patch
Gavin
: review+
jboriss
: ui-review+
Details | Diff | Splinter Review 
This menu item got bumped from bug 449522 because the backend didn't support it. Now it does, so we should add it.
Attached patch Patch v.1 — — Splinter Review 

        Attachment #345451 -
        Flags: ui-review?(jboriss)

        Attachment #345451 -
        Flags: review?(gavin.sharp)

        Attachment #345451 -
        Flags: review?(gavin.sharp) → review+

    
    

    
  
Comment on attachment 345451 [details] [diff] [review]
Patch v.1

Looks good

        Attachment #345451 -
        Flags: ui-review?(jboriss) → ui-review+

    
    

    
  
Pushed changeset bf8ec3b200b9.
Status: NEW → RESOLVED
Closed: 16 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: Firefox 3.1 → Firefox 3.1b2

    
  
Blocks: 461136

    
    

    
  
Maybe the "View Video" item should not be visible when the video is playing as a document?

    
  
Blocks: 462714

    
  
Depends on: 462892

    
    

    
  
Yeah, filed bug 462892.

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: