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

Reply via email to