Closed Bug 1018263 Opened 10 years ago Closed 10 years ago

Selected column in selection list properties does not work

Categories

(SeaMonkey :: Composer, defect)

Product:
SeaMonkey
Component:
Composer
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(seamonkey2.26 wontfix, seamonkey2.27 wontfix, seamonkey2.28 fixed, seamonkey2.29 fixed, seamonkey2.30 fixed)

Status:
RESOLVED FIXED
Milestone:
seamonkey2.30
Tracking Flags:
Tracking Status
seamonkey2.26 --- wontfix
seamonkey2.27 --- wontfix
seamonkey2.28 --- fixed
seamonkey2.29 --- fixed
seamonkey2.30 --- fixed

People

(Reporter: neil, Assigned: neil)

Details

Attachments

(1 file, 1 obsolete file)

With added columns
4.27 KB, patch
philip.chee
: review+
iannbugzilla
: approval-comm-aurora+
iannbugzilla
: approval-comm-beta+
Details | Diff | Splinter Review 
In the selection list properties dialog (available through the debug QA extension, or by accessing the properties of an existing selection list) the column that displays which option is selected does not display anything.

This is a regression from bug 407956 which wasn't picked up at the time.
Attached patch Proposed patch (obsolete) — — Splinter Review 
Assignee: nobody → neil
Status: NEW → ASSIGNED

        Attachment #8431634 -
        Flags: review?(philip.chee)

    
    

    
  
Sat Jun 07 2014 00:49:41
Error: treeBoxObject.getPrimaryColumn is not a function
Source file: chrome://editor/content/EdSelectProps.js
Line: 683

    
    

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

r=me

(In reply to Philip Chee from comment #2)
> Sat Jun 07 2014 00:49:41
> Error: treeBoxObject.getPrimaryColumn is not a function
> Source file: chrome://editor/content/EdSelectProps.js
> Line: 683
I'm not sure how this dialog ever worked?

        Attachment #8431634 -
        Flags: review?(philip.chee) → review+

    
    

    
  
(In reply to Philip Chee from comment #2)
> Error: treeBoxObject.getPrimaryColumn is not a function
> Source file: chrome://editor/content/EdSelectProps.js
> Line: 683

Whoa, that's a regression introduced by bug 221619... fix coming up.

    
    

    
  


  

        Attachment #8431634 -
        Attachment is obsolete: true

        Attachment #8436012 -
        Flags: review?(philip.chee)

    
    

    
  
Comment on attachment 8436012 [details] [diff] [review]
With added columns

r=me thanks!

        Attachment #8436012 -
        Flags: review?(philip.chee) → review+

    
    

    
  
Pushed comm-central changeset 699551e9deae.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.30

    
    

    
  
Comment on attachment 8436012 [details] [diff] [review]
With added columns

[Approval Request Comment]
Regression caused by (bug #): 407956
User impact if declined: Reduced dialog functionality
Testing completed (on m-c, etc.): Landed on c-c
Risk to taking this patch (and alternatives if risky): Low, dialog rarely used
String changes made by this patch: None

        Attachment #8436012 -
        Flags: approval-comm-beta?

        Attachment #8436012 -
        Flags: approval-comm-aurora?

    
    

    
  
Comment on attachment 8436012 [details] [diff] [review]
With added columns

a=me

        Attachment #8436012 -
        Flags: approval-comm-beta?

        Attachment #8436012 -
        Flags: approval-comm-beta+

        Attachment #8436012 -
        Flags: approval-comm-aurora?

        Attachment #8436012 -
        Flags: approval-comm-aurora+

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: