ngraham added a comment.
Looks and works great; just one style comment below: INLINE COMMENTS > kfilewidget.cpp:108 > + iconSizeSlider(nullptr), > + furtherAction(nullptr), > + closerAction(nullptr) While we're at it, why don't we rename these to be clearer: I'd prefer `zoomInAction` and `zoomOutAction` REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12590 To: rkflx, #frameworks Cc: ngraham, michaelh, bruns