Konqueror Context Specific Popup Menu

[David Faure]

On Wednesday 17 September 2003 16:36, Sashmit Bhaduri wrote:
> The old one was probably more logical, as actions with a link are used more 
> widely on the web than actions with images. See attached khtml patch to 
> revert that.

I think you attached the wrong patch - this is libkonq again.

> > Can you explain the KonqPopupMenu one?
> 
> No :) I was foot's'ing around for a while trying to find a way to detect this, 
> and this was the closest way I could find. Do you have any better 
> suggestions?

What about comparing the URL of the view with the URL of what we're clicking
upon? I think KonqPopupMenu does this already... Yes, that's the currentDir bool
(should be named "click on background", aka the view URL itself, don't mind the "dir" in the name).
Or am I missing the purpose here?

> > [That patch is Binary Incompatible, too, you'd need a new KonqPopupMenu
> > constructor]
> 
> That constructor which was changed was added after 3.1, that's fine right?

Oh - then indeed it's fine.

-- 
David Faure -- faure at kde.org, dfaure at klaralvdalens-datakonsult.se
Qt/KDE/KOffice developer
Klarälvdalens Datakonsult AB, Platform-independent software solutions