D6215: Introduce aboutToShow() signal

[Marco Martin]

mart added inline comments.

INLINE COMMENTS

> mart wrote in dialog.cpp:1368-1370
> > You've not really said what problem we're trying to solve with this patch.
> 
> having a pointt in which we are 100% sure of the final size of the dialog to correctly position it.
> if visualparent is set, this is done internally and all works fine, but for users that do a manual positioning, there are currently no hooks from client code to do so, this is in part the reason for the big ugly custom notifications positioning code
> 
> > this is already done in updateVisibility which is effectively called from
> 
> i think it should be safe to remove it from updatevisibility
> 
> > If you want a hook between us updating our size and the platform showing the window we've got ShowEvent.
> 
> yes, exactly. The idea is for it to be usable from qml, so can't rely on subclassing. which would be ok for notifications but not for simplemenu

>>   this is already done in updateVisibility which is effectively called from
> 
> i think it should be safe to remove it from updatevisibility

hmm, actually not, seems sometimes it needs to be called from updatevisibility, so both seem to be needed

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D6215

To: mart, #plasma, davidedmundson
Cc: hein, davidedmundson, plasma-devel, #frameworks, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170614/6b17e6a8/attachment.html>