Closed Bug 309075 Opened 18 years ago Closed 18 years ago

Right-clicking a partially off-screen link shouldn't scroll the page

Categories

(Core :: DOM: UI Events & Focus Handling, defect, P1)

Product:
Core
Component:
DOM: UI Events & Focus Handling
Platform:
x86
Windows XP
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.8.1

People

(Reporter: martijn.martijn, Assigned: masayuki)

References

(
URL
)

Details

(Keywords: fixed1.8.1, testcase, verified1.8.0.1)

Attachments

(1 file)

Patch rv1.0
3.14 KB, patch
roc
: review+
roc
: superreview+
Details | Diff | Splinter Review 
Bug 105894 is fixed, but that bug is still happening when right-clicking a link.
Priority: -- → P1
Target Milestone: --- → mozilla1.9alpha
Attached patch Patch rv1.0 — — Splinter Review 
Assignee: events → masayuki
Status: NEW → ASSIGNED

        Attachment #202138 -
        Flags: superreview?(roc)

        Attachment #202138 -
        Flags: review?(roc)

    
    

    
  
That patch doesn't affect behavior when tabbing to the link?  That is, in that case we scroll?

    
    

    
  
Yes, this affects only when the context menu is opened on the link by mouse.

    
    

    
  
Comment on attachment 202138 [details] [diff] [review]
Patch rv1.0

looks good!

        Attachment #202138 -
        Flags: superreview?(roc)

        Attachment #202138 -
        Flags: superreview+

        Attachment #202138 -
        Flags: review?(roc)

        Attachment #202138 -
        Flags: review+

    
    

    
  
checked-in.

    
    

    
  
Sorry for the spam.

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Target Milestone: mozilla1.9alpha → mozilla1.8.1

    
    

    
  
*** Bug 322755 has been marked as a duplicate of this bug. ***

    
    

    
  
v.fixed on 1.8.0.1 with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.0.1) Gecko/20060111 Firefox/1.5.0.1, no scroll when right-clicking partially off-screen link.
Keywords: fixed1.8.0.1 → verified1.8.0.1
Component: Event Handling → User events and focus handling

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