author | alwin <alwin> | 2005-03-07 23:12:51 (UTC) |
---|---|---|
committer | alwin <alwin> | 2005-03-07 23:12:51 (UTC) |
commit | 2ba21de3fce9ca9fe9a70474a1927f0106d1d7ce (patch) (unidiff) | |
tree | d0ff8c035e34a83fbc935c8f3f6ab926ed432559 /noncore/net/mail/viewmail.cpp | |
parent | a1fab1e1abd009a6a64c7ea3957bdb5587f4be81 (diff) | |
download | opie-2ba21de3fce9ca9fe9a70474a1927f0106d1d7ce.zip opie-2ba21de3fce9ca9fe9a70474a1927f0106d1d7ce.tar.gz opie-2ba21de3fce9ca9fe9a70474a1927f0106d1d7ce.tar.bz2 |
beauty
hopefull some speedups
obsolete stuff removed
fixed a crasher
-rw-r--r-- | noncore/net/mail/viewmail.cpp | 37 |
1 files changed, 31 insertions, 6 deletions
diff --git a/noncore/net/mail/viewmail.cpp b/noncore/net/mail/viewmail.cpp index a64a168..5206220 100644 --- a/noncore/net/mail/viewmail.cpp +++ b/noncore/net/mail/viewmail.cpp | |||
@@ -196,7 +196,7 @@ void ViewMail::setBody(const RecBodyP&body ) | |||
196 | void ViewMail::slotShowHtml( bool state ) | 196 | void ViewMail::slotShowHtml( bool state ) |
197 | { | 197 | { |
198 | m_showHtml = state; | 198 | m_showHtml = state; |
199 | setText(); | 199 | setMainText(); |
200 | } | 200 | } |
201 | 201 | ||
202 | void ViewMail::slotItemClicked( QListViewItem * item , const QPoint & point, int ) | 202 | void ViewMail::slotItemClicked( QListViewItem * item , const QPoint & point, int ) |
@@ -206,7 +206,7 @@ void ViewMail::slotItemClicked( QListViewItem * item , const QPoint & point, int | |||
206 | 206 | ||
207 | if ( ( ( AttachItem* )item )->Partnumber() == -1 ) | 207 | if ( ( ( AttachItem* )item )->Partnumber() == -1 ) |
208 | { | 208 | { |
209 | setText(); | 209 | setMainText(); |
210 | return; | 210 | return; |
211 | } | 211 | } |
212 | QPopupMenu *menu = new QPopupMenu(); | 212 | QPopupMenu *menu = new QPopupMenu(); |
@@ -250,6 +250,15 @@ void ViewMail::slotItemClicked( QListViewItem * item , const QPoint & point, int | |||
250 | 250 | ||
251 | case 2: | 251 | case 2: |
252 | { | 252 | { |
253 | if (m_lastimagepart== (( AttachItem* )item )->Partnumber()) { | ||
254 | if (m_lastdlg) { | ||
255 | setUpdatesEnabled(false); | ||
256 | browser->hide(); | ||
257 | m_lastdlg->show(); | ||
258 | setUpdatesEnabled(true); | ||
259 | return; | ||
260 | } | ||
261 | } | ||
253 | QString tmpfile = QString("/tmp/opiemail-image-%1").arg(_icounter++); | 262 | QString tmpfile = QString("/tmp/opiemail-image-%1").arg(_icounter++); |
254 | encodedString*content = m_recMail->Wrapper()->fetchDecodedPart( m_recMail, m_body->Parts()[ ( ( AttachItem* )item )->Partnumber() ] ); | 263 | encodedString*content = m_recMail->Wrapper()->fetchDecodedPart( m_recMail, m_body->Parts()[ ( ( AttachItem* )item )->Partnumber() ] ); |
255 | if (content && content->Length()) { | 264 | if (content && content->Length()) { |
@@ -276,18 +285,29 @@ void ViewMail::slotItemClicked( QListViewItem * item , const QPoint & point, int | |||
276 | } else { | 285 | } else { |
277 | QMessageBox::critical(this, tr("Reading attachment"), tr("Could not read content of attachment")); | 286 | QMessageBox::critical(this, tr("Reading attachment"), tr("Could not read content of attachment")); |
278 | } | 287 | } |
288 | m_lastimagepart=(( AttachItem* )item )->Partnumber(); | ||
279 | } | 289 | } |
280 | break; | 290 | break; |
281 | case 1: | 291 | case 1: |
282 | if ( ( ( AttachItem* )item )->Partnumber() == -1 ) | 292 | if ( ( ( AttachItem* )item )->Partnumber() == -1 ) |
283 | { | 293 | { |
284 | setText(); | 294 | setMainText(); |
285 | } | 295 | } |
286 | else | 296 | else |
287 | { | 297 | { |
288 | if ( m_recMail->Wrapper() != 0l ) | 298 | if ( m_recMail->Wrapper() != 0l ) |
289 | { // make sure that there is a wrapper , even after delete or simular actions | 299 | { // make sure that there is a wrapper , even after delete or simular actions |
290 | browser->setText( m_recMail->Wrapper()->fetchTextPart( m_recMail, m_body->Parts()[ ( ( AttachItem* )item )->Partnumber() ] ) ); | 300 | if (m_lastdlg) { |
301 | m_lastdlg->hide(); | ||
302 | browser->show(); | ||
303 | } | ||
304 | if (m_lasttextpart == ( ( AttachItem* )item )->Partnumber() ) return; | ||
305 | m_lasttextpart = ( ( AttachItem* )item )->Partnumber(); | ||
306 | QString s = m_recMail->Wrapper()->fetchTextPart( m_recMail, m_body->Parts()[ ( ( AttachItem* )item )->Partnumber() ] );; | ||
307 | if (item->text(0).right(4)!="html") { | ||
308 | s = QString("<html><body>\n<font face=fixed>%1</font></body></html>").arg(QStyleSheet::convertFromPlainText(s)); | ||
309 | } | ||
310 | browser->setText(s); | ||
291 | } | 311 | } |
292 | } | 312 | } |
293 | break; | 313 | break; |
@@ -311,7 +331,7 @@ void ViewMail::setMail(const RecMailP&mail ) | |||
311 | m_mail2[2] = mail->Bcc(); | 331 | m_mail2[2] = mail->Bcc(); |
312 | 332 | ||
313 | setCaption(tr("E-Mail by %1").arg( m_mail[0] ) ); | 333 | setCaption(tr("E-Mail by %1").arg( m_mail[0] ) ); |
314 | setText(); | 334 | setMainText(); |
315 | } | 335 | } |
316 | 336 | ||
317 | 337 | ||
@@ -332,6 +352,8 @@ ViewMail::ViewMail( QWidget *parent, const char *name, WFlags fl) | |||
332 | connect( attachments, SIGNAL( clicked(QListViewItem*,const QPoint&, int) ), SLOT( slotItemClicked(QListViewItem*,const QPoint&, int) ) ); | 352 | connect( attachments, SIGNAL( clicked(QListViewItem*,const QPoint&, int) ), SLOT( slotItemClicked(QListViewItem*,const QPoint&, int) ) ); |
333 | m_lastdlg = 0; | 353 | m_lastdlg = 0; |
334 | _icounter = 0; | 354 | _icounter = 0; |
355 | m_lastimagepart = -1; | ||
356 | m_lasttextpart = -2; | ||
335 | 357 | ||
336 | readConfig(); | 358 | readConfig(); |
337 | attachments->setSorting(-1); | 359 | attachments->setSorting(-1); |
@@ -348,6 +370,7 @@ void ViewMail::slotImageInline(bool how) | |||
348 | m_lastdlg->reparent(0,QPoint(0,0),false); | 370 | m_lastdlg->reparent(0,QPoint(0,0),false); |
349 | delete m_lastdlg; | 371 | delete m_lastdlg; |
350 | m_lastdlg = 0; | 372 | m_lastdlg = 0; |
373 | m_lastimagepart = -1; | ||
351 | } | 374 | } |
352 | } | 375 | } |
353 | 376 | ||
@@ -361,7 +384,7 @@ void ViewMail::readConfig() | |||
361 | showHtml->setOn( m_showHtml ); | 384 | showHtml->setOn( m_showHtml ); |
362 | } | 385 | } |
363 | 386 | ||
364 | void ViewMail::setText() | 387 | void ViewMail::setMainText() |
365 | { | 388 | { |
366 | QString toString; | 389 | QString toString; |
367 | QString ccString; | 390 | QString ccString; |
@@ -372,6 +395,8 @@ void ViewMail::setText() | |||
372 | m_lastdlg->hide(); | 395 | m_lastdlg->hide(); |
373 | } | 396 | } |
374 | browser->show(); | 397 | browser->show(); |
398 | if (m_lasttextpart == -1) return; | ||
399 | m_lasttextpart = -1; | ||
375 | for ( QStringList::Iterator it = ( m_mail2[0] ).begin(); it != ( m_mail2[0] ).end(); ++it ) | 400 | for ( QStringList::Iterator it = ( m_mail2[0] ).begin(); it != ( m_mail2[0] ).end(); ++it ) |
376 | { | 401 | { |
377 | toString += (*it); | 402 | toString += (*it); |