Bug 115490 - annoying messagebox in kuickshow at unsupported image-formats
Summary: annoying messagebox in kuickshow at unsupported image-formats
Status: RESOLVED FIXED
Alias: None
Product: kuickshow
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Debian stable Linux
: NOR wishlist
Target Milestone: ---
Assignee: Carsten Pfeiffer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-11-01 21:13 UTC by Judit Foglszinger
Modified: 2006-01-08 15:38 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Judit Foglszinger 2005-11-01 21:13:06 UTC
Version:            (using KDE KDE 3.4.1)
Installed from:    Debian stable Packages
OS:                Linux

Whenever you try to open an image-format which is not 
supported, like .xcf, .svg or .ico, 
even during a diashow, there is the following messagebox:

"Unable to load the image /directory/blablubb.svg.
Perhaps the file format is unsupported or your Imlib is not installed properly."

Every time you have to click ok or press enter, even if you know 
a long time, that kuickshow is not able to handle .svg .ico or .xcf.

It would be better, to let te user decide, if he wants to see
this messagebox or not.

There should be either a possibilty to change this behavior in a configfile or a checkbox like "do not show this message again"
Comment 1 Judit Foglszinger 2005-11-08 19:47:30 UTC
I have overlooked, that this was already in the TODO-list:
"if image not readable -> no nagscreen, jump to next image"
So I must admit that my report is a bit pointless :-)
Comment 2 Judit Foglszinger 2005-12-22 13:55:24 UTC
Now I've got a quick and dirty solution.
On my computer it worked;
but I cannot code - So use at your own risk.


Search the text below in kuickshow.cpp
and comment out the lines between the {} in this way:

void KuickShow::messageCantLoadImage( const QString& filename )
 {
    /* m_viewer->clearFocus();
     QString tmp = i18n("Unable to load the image %1.\n"
                        "Perhaps the file format is unsupported or "
                        "your Imlib is not installed properly.").arg(filename);
     KMessageBox::sorry( m_viewer, tmp, i18n("Image Error") ); */
 }
Comment 3 Carsten Pfeiffer 2006-01-08 15:38:37 UTC
SVN commit 495619 by pfeiffer:

Add "don't show again" to "Can't load image" messagebox.

BUG: 115490


 M  +4 -4      kuickshow.cpp  


--- branches/KDE/3.5/kdegraphics/kuickshow/src/kuickshow.cpp #495618:495619
@@ -206,7 +206,7 @@
     KActionCollection *coll = fileWidget->actionCollection();
 
     redirectDeleteAndTrashActions(coll);
-    
+
     connect( fileWidget, SIGNAL( fileSelected( const KFileItem * ) ),
              this, SLOT( slotSelected( const KFileItem * ) ));
 
@@ -400,7 +400,7 @@
         action->disconnect(fileWidget);
         connect(action, SIGNAL(activated()), this, SLOT(slotDeleteCurrentImage()));
     }
-    
+
     action = coll->action("trash");
     if (action)
     {
@@ -660,7 +660,7 @@
     // entire directory after a file was deleted/trashed!?
     if (!m_viewer)
         return;
-    
+
     if (next)
         showImage(next, false);
     else
@@ -1160,7 +1160,7 @@
     QString tmp = i18n("Unable to load the image %1.\n"
                        "Perhaps the file format is unsupported or "
                        "your Imlib is not installed properly.").arg(filename);
-    KMessageBox::sorry( m_viewer, tmp, i18n("Image Error") );
+    KMessageBox::information( m_viewer, tmp, i18n("Image Error"), "kuick_cant_load_image" );
 }
 
 void KuickShow::initImlib()