Closed Bug 251465 Opened 20 years ago Closed 20 years ago

-moz-user-focus: none; makes no sense for XUL

Categories

(Core :: XUL, defect)

Product:
Core
Component:
XUL
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: neil, Assigned: neil)

Details

Attachments

(2 files)

Proposed patch
894 bytes, patch
bryner
: review+
bryner
: superreview+
Details | Diff | Splinter Review
port remaining changes to toolkit (checked in)
592 bytes, patch
mconnor
: review+
Details | Diff | Splinter Review 
XUL widgets should either take the focus or ignore it. Unlike HTML, they should
never cause a blur to the canvas.

If you want it, the test case I was using had a widget in a menulist; xul.css
defines -moz-user-focus: none; for that widget and therefore when it is clicked
the menulist loses focus.
Attached patch Proposed patch — — Splinter Review 
Assignee: nobody → neil.parkwaycc.co.uk
Status: NEW → ASSIGNED

    
    

    
  
Comment on attachment 153228 [details] [diff] [review]
Proposed patch

Since * is set to -moz-user-focus: ignore; I thought it would be simplest to
delete all the none; rules.

Please let me know who else I should ask for review from.

        Attachment #153228 -
        Flags: superreview?(bryner)

        Attachment #153228 -
        Flags: review?(bryner)

        Attachment #153228 -
        Flags: superreview?(bryner)

        Attachment #153228 -
        Flags: superreview+

        Attachment #153228 -
        Flags: review?(bryner)

        Attachment #153228 -
        Flags: review+

    
    

    
  
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED

    
    

    
  


  
I missed one occurence when porting most of this to toolkit in Bug 261790

        Attachment #167152 -
        Flags: review?(mconnor) → review+

    
    

    
  
Comment on attachment 167152 [details] [diff] [review]
port remaining changes to toolkit (checked in)

toolkit port checked in

        Attachment #167152 -
        Attachment description: port remaining changes to toolkit → port remaining changes to toolkit (checked in)

    
  
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: xptoolkit.widgets

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: