Skip to content

Fix icon for "Paste Reference Image From System Clipboard"

Dmitrii Utkin requested to merge (removed):dmitriiutkin/408044-ref-tool-paste-icon into master

Fixing the BUG:408044.

Changes:

  1. Set image and title to bnPasteReferenceImage programmatically to follow the common code style and make i18n possible.

  2. Edited WdgToolOptions.ui to make button look exactly as it's neighbors.

  3. Replaced d->ui->bnPasteReferenceImage->setEnabled(!QApplication::clipboard()->image().isNull()); with slotCheckClipboardContents(); to avoid code duplication.

Test Plan

Affects UI only.

Reproduce:

  1. Copy an image to clipboard
  2. Activate "References image tool"
  3. Look on the Tool Options dock:

Before patch there is no icon.

After patch, there is an standard "paste" icon.

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.

Merge request reports