[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice
Subject:    kspread does not load column size changes
From:       "Byron C. Servies" <bservies () pacang ! com>
Date:       1999-12-23 21:16:22
[Download RAW message or body]

Hi there!

kspread has not been reloading changes to column widths for me in the past several
weeks.  I had some time this morning, and a (very) little investigation showed
that it was saving the information, but was using the attribute name "col", while
the KSpreadTable::loadXML routine was looking for the attribute name "column".

Since "column" is listed in the DTD, I changed kspread_layout.cc to use "column"
instead of "col" and seem to have corrected the problem.

Context diff attached from cvsup of yesterday (12/22).

Byron

["kspread_layout.cc.patch" (application/octet-stream)]

*** kspread_layout.cc.orig	Thu Dec 23 13:03:52 1999
--- kspread_layout.cc	Thu Dec 23 13:04:12 1999
***************
*** 247,255 ****
  
  QDomElement ColumnLayout::save( QDomDocument& doc )
  {
!   QDomElement col = doc.createElement( "col" );
    col.setAttribute( "width", m_fWidth );
!   col.setAttribute( "col", m_iColumn );
  
    return col;
  }
--- 247,255 ----
  
  QDomElement ColumnLayout::save( QDomDocument& doc )
  {
!   QDomElement col = doc.createElement( "column" );
    col.setAttribute( "width", m_fWidth );
!   col.setAttribute( "column", m_iColumn );
  
    return col;
  }
***************
*** 263,269 ****
      if ( !ok ) return false;
    }
  
!   m_iColumn = col.attribute( "col" ).toInt( &ok );
    if ( !ok ) return false;
  
    // Validation
--- 263,269 ----
      if ( !ok ) return false;
    }
  
!   m_iColumn = col.attribute( "column" ).toInt( &ok );
    if ( !ok ) return false;
  
    // Validation


[prev in list] [next in list] [prev in thread] [next in thread]