Closed Bug 102650 Opened 23 years ago Closed 23 years ago

venkman skin/contents.rdf sez 'chrome:displayName="modern"'; should be Modern

Categories

(Other Applications Graveyard :: Venkman JS Debugger, defect)

Product:
Other Applications Graveyard
Component:
Venkman JS Debugger
Platform:
x86
Windows 2000
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: jrgmorrison, Assigned: rginda)

References

(
URL
)

Details

Attachments

(1 file)

patch
584 bytes, patch
jrgmorrison
: review+
jrgmorrison
: superreview+
Details | Diff | Splinter Review 
Along the lines of bug 102588 (and leaving aside the question of supplemental 
packages overwriting the author and skin name), I noticed that I had a lower
case value for "modern" in the UI. I believe it comes from this line in 
venkman's contents.rdf.
http://lxr.mozilla.org/seamonkey/source/extensions/venkman/resources/skin/conte
nts.rdf#48
here comes a patch, shaver/hyatt, r/sr?
Status: NEW → ASSIGNED
Attached patch patch — — Splinter Review 

    
    

    
  
Comment on attachment 51761 [details] [diff] [review]
patch

going out on a limb on this one, but r=jrgm, and take an rs=mozbot, sez me.

        Attachment #51761 -
        Flags: superreview+

        Attachment #51761 -
        Flags: review+

    
    

    
  
fixed.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Product: Core → Other Applications

    
  
Product: Other Applications → Other Applications Graveyard

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: