On Monday 24 August 2009, Chani wrote: > > > > * way for a widget to advertise that it should receive keyboard focus > > > > when clicked and have PanelView grab focus in that case > > > > > > neglecting for a while how actually give focus to the panel, api-wise > > > on plasma side could work as this: > > > Applet::addFocusWidget(QGraphicsWidget *) > > > it's usedto add widgets that are supposed to request focus when > > > clicked, thinking about it maybe we want > > > addFocusWidgets(QList<QGraphicsWidget *>) > > ok, so then an applet would call addFocusWidget(this) if it needs kbd > focus, much like it calls setHasConfigurationInterface... and it'd also > have the flexibility to call that on subsets of itself (should it take a > graphicswidget or graphicsitem? would it ever make sense to focus a > qgraphicsitem?)
i think it's better we keep plasma as qgraphicswidget oriented as possible there are altready some places where i did put qgrahicsitems or qgraphicslayoutitems in the api that now makes me cry every time i look at them :p > and, I'm assuming it can add >1, does that mean that if it > adds a widget that's part of itself that we sent focus to that widget when > that widget is clicked? or we send focus to it when any part of the applet > is clicked? ...probably the former, I suppose. hm, yeah, we can either do a more complex way by adding exactly the applet widgets that are sensible so only when they got clicked the panel will grab focus or a waay simpler api (but more imprecise behaviour) something like Applet::setRequireFocus(bool) or something like that. the panel will force focus only when the user will click anywhere in the applet. > > and then whoever does the actual work will have to read that list of > widgets, and probably keep an eye on them. > > > i don't think we need to provide an addFocusWidgets that takes a list; > > the user of it can call add multiple times. if there was a method that > > added a list, i'd think it should be called "setFocusWidgets"? removal > > would be achieved by deletion of the widget, i suppose? or would it make > > sense to have a removeFocusWidget as well for symmetry? > > probably should have it. extenders? applets that embed other applets? I'm > sure eventually someone will want to remove something without deleting it. yes, makes sense -- Marco Martin _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
