[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: modeless KFileDialog
From:       Stephan Kulow <coolo () kde ! org>
Date:       2000-02-09 22:22:11
[Download RAW message or body]

Reginald Stadlbauer wrote:

> Well, when thinking about it again, it looks more like a KIconView problem.
> Because this is a single click vs. double click thingy, and QIconView doesn't
> know anything about that. So, I guess you mean clicked() as highlight and
> doubleClicked() as selected. And in single-click mode we connect clicked(),
> with doubleClicked(). That's what you want and that's ok. But we still get
> clicked() of course in single click mode then, which is not what we want. But in
> single click we only want to get that if CTRL is also pressed.
> So I think we need a KIconView::highlighted() signal or something like that
> which is used instead if clicked() and just connected to clicked() in double
> click mode. In single click mode clicked() is connected to a slot which checks
> if CTRL is pressed (unfortunately we don't really know it so we will have to use
> some XQueryKeymap or whatever I fear) and if CTRL is pressed highlighted() is
> emitted. Ok, that's really hacky. Maybe just reimplementing
> contentsMouseRelease() and doing the check for highlighted() there is better.
> But in any way, it has to be done in KIconView (and the other KWhateverView
> classes too)
42
> 
> And what you mean with the "second "I fix that tomorrow"" - I don't know. If
> you mean that I don't spend a lot of time on KDE at the moment - sorry but I'm
> really busy at the moment with many other stuff and I just don't find the time
> to do everything I want to :-((
I understand that. It was just that I asked you about the QIconView
changes and
you said "I do that the next days"

Greetings, Stephan

-- 
It said Windows 95 or better, so in theory Linux should run it
                                                GeorgeH on /.

[prev in list] [next in list] [prev in thread] [next in thread]