D14201: Set system default shortcut Ctrl+0 for "Actual Size" action

[Peter Mühlenpfordt]

This revision was automatically updated to reflect the committed changes.
Closed by commit R237:f651a817ab00: Set system default shortcut Ctrl+0 for 
"Actual Size" action (authored by muhlenpfordt).

REPOSITORY
  R237 KConfig

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D14201?vs=38004&id=39450

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

AFFECTED FILES
  src/gui/kstandardshortcut.cpp

To: muhlenpfordt, dfaure, broulik, ngraham, rkflx
Cc: jriddell, rkflx, kde-frameworks-devel, michaelh, ngraham, bruns

D14201: Set system default shortcut Ctrl+0 for "Actual Size" action

[Peter Mühlenpfordt]

muhlenpfordt edited the summary of this revision.

REPOSITORY
  R237 KConfig

BRANCH
  shortcut-actual-size (branched from master)

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

To: muhlenpfordt, dfaure, broulik, ngraham, rkflx
Cc: jriddell, rkflx, kde-frameworks-devel, michaelh, ngraham, bruns

D14201: Set system default shortcut Ctrl+0 for "Actual Size" action

[Peter Mühlenpfordt]

muhlenpfordt added a comment.


  The destination is still KDE Neon with Ubuntu 18.04? Is there any schedule 
for this?

REPOSITORY
  R237 KConfig

BRANCH
  shortcut-actual-size (branched from master)

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

To: muhlenpfordt, dfaure, broulik, ngraham
Cc: rkflx, kde-frameworks-devel, michaelh, ngraham, bruns

D14201: Set system default shortcut Ctrl+0 for "Actual Size" action

[Peter Mühlenpfordt]

muhlenpfordt created this revision.
muhlenpfordt added reviewers: dfaure, broulik, ngraham.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: kde-frameworks-devel.
muhlenpfordt requested review of this revision.

REVISION SUMMARY
  Default shortcuts for zoom in ([Ctrl] + [+]) and out ([Ctrl] + [-])
  already exists but not for going back to 100%.
  The shortcut [Ctrl] + [0] is widely used by different applications
  for zoom to original size. This patch adds [Ctrl] + [0] as system
  default for `KStandardAction::ActualSize`.

TEST PLAN
  Searched lxr for conflicts of `KStandardAction::ActualSize` and
  usage of Ctrl+0 shortcut.
  Checked the following applications with set system default shortcut:
  
  - Gwenview
  - KAlgebra
  - KolourPaint
  - KStars (FITS viewer)
  - Cirkuit
  
  Only Cirkuit uses Ctrl+0 for "Zoom To Fit" action. I'll create a
  patch to solve this conflict.

REPOSITORY
  R237 KConfig

BRANCH
  shortcut-actual-size (branched from master)

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

AFFECTED FILES
  src/gui/kstandardshortcut.cpp

To: muhlenpfordt, dfaure, broulik, ngraham
Cc: rkflx, kde-frameworks-devel, michaelh, ngraham, bruns

D12734: Scale up folder icon before creating preview overlays

[Peter Mühlenpfordt]

This revision was automatically updated to reflect the committed changes.
Closed by commit R320:bafc90d730f2: Scale up folder icon before creating 
preview overlays (authored by muhlenpfordt).

REPOSITORY
  R320 KIO Extras

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D12734?vs=33822&id=34202

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

AFFECTED FILES
  thumbnail/thumbnail.cpp

To: muhlenpfordt, #frameworks, rkflx
Cc: ngraham, rkflx

D12734: Scale up folder icon before creating preview overlays

[Peter Mühlenpfordt]

muhlenpfordt added a comment.


  In D12734#259515 , @rkflx wrote:
  
  > Nevertheless, let's check the impact of this again: 
https://lxr.kde.org/ident?_i=thumbForDirectory
  >  Does not look too critical to me, and we probably tested most of the code 
paths.
  
  
  The normal KDE file dialog (e.g. Kate > File > Open...) also uses the 
upscaled preview without any problems.
  I searched for (`KIO::`)`filePreview` on lxr which gives a couple of results. 
They all use similar methods to handle the icons and I can't spot anything 
unsual related to the returned size.
  Ok to commit?

REPOSITORY
  R320 KIO Extras

BRANCH
  scale-up-folder-icon (branched from Applications/18.04)

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

To: muhlenpfordt, #frameworks, rkflx
Cc: ngraham, rkflx

D12734: Scale up folder icon before creating preview overlays

[Peter Mühlenpfordt]

muhlenpfordt updated this revision to Diff 33822.
muhlenpfordt added a comment.


  Rebase to `Applications/18.04`

REPOSITORY
  R320 KIO Extras

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D12734?vs=33749&id=33822

BRANCH
  scale-up-folder-icon (branched from Applications/18.04)

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

AFFECTED FILES
  thumbnail/thumbnail.cpp

To: muhlenpfordt, #frameworks, rkflx
Cc: ngraham, rkflx

D12734: Scale up folder icon before creating preview overlays

[Peter Mühlenpfordt]

muhlenpfordt added a comment.


  Forgot to mention this...
  There's a nice test application in `kio` for viewing a folder's preview in 
any desired size (created by `KIO::PreviewJob`): `previewtest`. Just enter the 
full path to the folder here.

REPOSITORY
  R320 KIO Extras

BRANCH
  master

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

To: muhlenpfordt, #frameworks, rkflx
Cc: ngraham, rkflx

D12734: Scale up folder icon before creating preview overlays

[Peter Mühlenpfordt]

muhlenpfordt added a comment.


  In D12734#259336 , @rkflx wrote:
  
  > +1, but I'd feel a bit uncomfortable (almost) self-approving my own patch 
(sans the `qMax`) here, so perhaps wait until the end of the week for any other 
opinions before committing.
  
  
  I did nearly the same in my initial search, except for the separate `extent` 
var. So, two ideas leading to the same code can't be wrong.  ;)  But sure, I'll 
wait until next week.
  
  > BTW, any reason you are targetting `master` only?
  
  I wanted to be careful here since I'm a bit afraid of changing library 
functions. If you think it's ok for `Applications/18.04` I'll rebase it.

REPOSITORY
  R320 KIO Extras

BRANCH
  master

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

To: muhlenpfordt, #frameworks, rkflx
Cc: ngraham, rkflx

D12734: Scale up folder icon before creating preview overlays

[Peter Mühlenpfordt]

muhlenpfordt added a reviewer: Frameworks.
muhlenpfordt added subscribers: rkflx, ngraham.
muhlenpfordt added a comment.


  Also see D12509  for Gwenview effects on 
this issue.

REPOSITORY
  R320 KIO Extras

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

To: muhlenpfordt, #frameworks
Cc: ngraham, rkflx