[prev in list] [next in list] [prev in thread] [next in thread]
List: kfm-devel
Subject: Re: Popup menu MIME types
From: David Faure <faure () kde ! org>
Date: 2005-01-28 10:30:45
Message-ID: 200501281130.45910.faure () kde ! org
[Download RAW message or body]
On Friday 28 January 2005 08:20, Jonathan Marten wrote:
> The patch to konq_mainwindow.cc looks extensive, but it is mostly just moving
> code around. Building the "Preview" menu is delayed until the popup menu has
> been built, so that the same MIME type can be used to select appropriate
> previewers.
Wouldn't it be simpler to determine the mimetype in KHTMLPart, before emitting popupMenu?
After all, this is where args.serviceType = QString::fromLatin1( "text/html" ); is hardcoded,
and where it should be set correctly. Hacking konq to do a better guess sounds wrong to me.
Wouldn't this also solve your "it has to be fixed in two places" problem?
--
David Faure, faure@kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic