[PATCH] Recent documents: Hide hidden files

Pascal Létourneau pletourn at globetrotter.net
Thu Feb 17 19:36:23 GMT 2005


On February 17, 2005 14:09, Bram Schoenmakers wrote:
> Hi,
>
> I came across something strange in the code of the Recent Documents. The
> reason to look there was due to a bugreport I created yesterday (#99562).
>
> I saw today in CVS's log that pletourn added QDir::Hidden to the selection,
> allowing hidden files in the menu. This contradicts his commit message,
> which states: "Allow hidden files to be expunged from the recent documents
> list".
There is no contradiction. I added QDir::Hidden to the list of entries that 
can be REMOVED from recent documents list.
If you want hidden files not to be shown, you have to remove this QDir::Hidden


QStringList KRecentDocument::recentDocuments()
{
    QDir d(recentDocumentDirectory(), "*.desktop", QDir::Time,
           QDir::Files | QDir::Readable | QDir::Hidden);
...

I don't think it's a good idea. If you explicitly open an hidden file, I think 
it should appear in the Recent Document menu

Pascal Létourneau




More information about the kde-core-devel mailing list