Skip to content

Improves recent files to respect `maxRecentFileItems`

Sam Linnfer requested to merge lllf/krita:lllf/fix-max-recent-files into master

Allow the number of recent files displayed in the UI to be controlled by maxRecentFileItems in Krita's config file. This matches the existing behaviour in RecentFileManager.cpp.

Cleans up strange whitespace and code style in separate commit.

This is a useful feature available in Photoshop, Evince, etc.

Test Plan

  1. Open Krita
  2. Verify number of recent files displayed
  3. Close Krita
  4. Open ~/.config/kritarc
  5. Verify that under [RecentFiles], there is a new key maxRecentFileItems
  6. Change the new key to any value
  7. Re-open Krita and verify that the maximum number of recent files displayed matches the value specified

Formalities Checklist

  • I confirmed this builds.
  • I confirmed Krita ran and the relevant functions work.
  • I tested the relevant unit tests and can confirm they are not broken. (If not possible, don't hesitate to ask for help!)
  • I made sure my commits build individually and have good descriptions as per KDE guidelines.
  • I made sure my code conforms to the standards set in the HACKING file.
  • I can confirm the code is licensed and attributed appropriately, and that unattributed code is mine, as per KDE Licensing Policy.
Edited by Sam Linnfer

Merge request reports