D9130: [widgetexplerer] Destroy download dialog at time of widget

2017-12-06 Thread Anthony Fieroni
This revision was automatically updated to reflect the committed changes. Closed by commit R120:84a29ef86380: [widgetexplorer] Make sure KNS download dialog is destroyed (authored by anthonyfieroni). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE

D9130: [widgetexplerer] Destroy download dialog at time of widget

2017-12-06 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D9130 To: anthonyfieroni, davidedmundson, broulik, mart Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,

D9130: [widgetexplerer] Destroy download dialog at time of widget

2017-12-06 Thread Anthony Fieroni
anthonyfieroni added a reviewer: mart. anthonyfieroni added a comment. Anyone against? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D9130 To: anthonyfieroni, davidedmundson, broulik, mart Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,

D9130: [widgetexplerer] Destroy download dialog at time of widget

2017-12-04 Thread Anthony Fieroni
anthonyfieroni updated this revision to Diff 23462. anthonyfieroni added a comment. Ability to detect delete on close. REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9130?vs=23459=23462 REVISION DETAIL https://phabricator.kde.org/D9130

D9130: [widgetexplerer] Destroy download dialog at time of widget

2017-12-04 Thread Anthony Fieroni
anthonyfieroni updated this revision to Diff 23459. anthonyfieroni added a comment. Oh i revert patch, WidgetExplorer is not widget, what DownloadDialog expects as parent. REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9130?vs=23420=23459

D9130: [widgetexplerer] Destroy download dialog at time of widget

2017-12-04 Thread Anthony Fieroni
anthonyfieroni updated this revision to Diff 23420. anthonyfieroni added a comment. Take ownership of DownloadDialog. Remove slot that not present. REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9130?vs=23336=23420 REVISION DETAIL

D9130: [widgetexplerer] Destroy download dialog at time of widget

2017-12-03 Thread Anthony Fieroni
anthonyfieroni created this revision. anthonyfieroni added reviewers: davidedmundson, broulik. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY CCBUG: 355464 TEST PLAN When DownloadDialog is not destroyed KNS3 engine