Reviewing merge request #12: Adding GUI to urlpreview

Adding GUI to urlpreview plugin and some fixes.

Commits that would be merged:

Version 1
  • Version 1
  • Version 1
Showing 3d9cf9c-dcf55f6

Comments

→ State changed from Open to Need refactoring

  1. Use

Config cfg;
cfg.beginGroup(QLatin1String(“urlpreview”));

instead of

Config cfg(“urlpreview”);

  1. You should create SettingsItem at load and delete it at unload

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository