54912 - table cell right align is assigned incorrectly

Status: VERIFIED DUPLICATE of bug 102135

(Core :: DOM: Editor, defect, P3)

Description

[rubydoo123]

using comm bits from 2000093020 on win98
1. open composer
2. insert table, with several rows and columns, enter text in cells
3. select a cell, select right alignment
4. view source -- the right align is done through the div element. We are 
exporting 4.0 transitional, the align attribute should be used on the cell.
--------------------------------------------------------------------------------
<!-- TH is for headers, TD for data, but for cells acting as both use TD -->
<!ATTLIST (TH|TD)                  -- header or data cell --
   %attrs;                         -- %coreattrs, %i18n, %events --
   abbr        %Text;    #IMPLIED  -- abbreviation for header cell --
   axis        CDATA     #IMPLIED  -- comma-separated list of related headers--
   headers     IDREFS    #IMPLIED  -- list of id's for header cells --
   scope       %Scope;   #IMPLIED  -- scope covered by header cells --
   rowspan     NUMBER    1         -- number of rows spanned by cell --
   colspan     NUMBER    1         -- number of cols spanned by cell --
   %cellhalign;                    -- horizontal alignment in cells --
   %cellvalign;                    -- vertical alignment in cells --
   nowrap      (nowrap)  #IMPLIED  -- suppress word wrap --
   bgcolor     %Color;   #IMPLIED  -- cell background color --
   width       %Length;  #IMPLIED  -- width for cell --
   height      %Length;  #IMPLIED  -- height for cell --
   >

<!ENTITY % cellhalign
   "align      (left|center|right|justify|char) #IMPLIED
   char       %Character;    #IMPLIED  -- alignment char, e.g. char=':' --
   charoff    %Length;       #IMPLIED  -- offset for alignment char --"
   >

<!ENTITY % cellvalign
   "valign     (top|middle|bottom|baseline) #IMPLIED"
   >

Comment 1

[rubydoo123]

actually, this should go to cmanske, he will be back to work on post rtm bugs, 
this can wait till then

Comment 2

[Charles Manske]

If you set the alignment in the Table Properties dialog, it is correct.
We should make the Alignment commands smart enough to detect when cells are
selected so the attribute is set on TD instead of always using the DIV.
Fairly easy to fix.

Comment 3

[Charles Manske]

This may be a dup. The HTML rules code needs to check for selected cells and
set attribute there instead of a div.

Comment 4

[Joe Francis]

moving a bunch of 0.9 bugs to 0.9.1

Comment 5

[rubydoo123]

moving to 1.0

Comment 6

[Charles Manske]

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

Comment 7

[Joe Francis]

Daniel, is this fixed by your patch?

Comment 8

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

*** This bug has been marked as a duplicate of 102135 ***

Comment 9

[James Baxter]

Verified