This forum has been archived. All content is frozen. Please use KDE Discuss instead.

kreenshot-editor discussion

Tags: None
(comma "," separated)
User avatar
gregormi
Registered Member
Posts
87
Karma
1
OS

kreenshot-editor discussion

Sun Apr 12, 2015 7:41 am
This is the main entry for discussions around the KDE playground project kreenshot-editor.

Coordinates:
ripper17
Registered Member
Posts
46
Karma
0
OS

Re: kreenshot-editor discussion

Mon Apr 20, 2015 10:54 am
Perfect! I just found this, and I really hope this is going to be able to replace Shutter - which in itself is quite cool, but the KDE support of Shutter is not any good (e.g. nfs issues)

From looking at the screenshots on the homepage, I really have great hopes :-)

Would it be possible to also include something like a Sketch circle in a later version? And maybe also some kind of hand-drawn arrow? I'm using screenshots for user manuals, and pseudo-hand-drawn stuff looks a bit more "relaxed" in my opinion, while "perfect" arrows and circles are a bit "strict". Of course, that's a matter of taste...

And while I'm at writing down features that are probably out of scope and wishful-thinking: is this also planned to be a successor of Screenie? There has been no development of that tool and it's missing some features (e.g. you can move the pictures horizontally in Screenie, but not vertically)


openSUSE 13.2 x64, Platform Version 4.14.X, Kubuntu 14.04 (LTS), Platform Version 4.13.X
User avatar
google01103
Manager
Posts
6668
Karma
25

Re: kreenshot-editor discussion

Mon Apr 20, 2015 4:49 pm
would it be possible to add the option to save to a image hosting site?


OpenSuse Leap 42.1 x64, Plasma 5.x

User avatar
gregormi
Registered Member
Posts
87
Karma
1
OS

Re: kreenshot-editor discussion

Sat Apr 25, 2015 1:38 pm
Thanks for the first feedback.

The main idea is not so much replacing any screenshot-taking tool (like shutter) but to enhance them with easy-to-use image editor capabilities.

About the "sketch circle" and fuzzy arrows: I like the idea, though it is currently not on the top priority list unless someone likes to contribute (I don't know straightaway how to implement such a circle).

Yes, Screenie is different class of software, so it is out-of-scope (for now).

About sending to image hosting site: generally, you can use 'copy to clipboard' and then paste to the Plasma Pastebin widget. For now, the idea is to take the screenshot with ksnapshot, edit it with kreenshot-editor and then use ksnapshot's SendTo menu for further actions which include sending to various image hosting sites.
User avatar
Mamarok
Manager
Posts
6071
Karma
16
OS

Re: kreenshot-editor discussion

Sat Apr 25, 2015 1:54 pm
I am still wondering: why not add these additional features to KSnapshot? I don't really see the point in having yet another image editor, if it is just that.


Running Kubuntu 22.10 with Plasma 5.26.3, Frameworks 5.100.0, Qt 5.15.6, kernel 5.19.0-23 on Ryzen 5 4600H, AMD Renoir, X11
FWIW: it's always useful to state the exact Plasma version (+ distribution) when asking questions, makes it easier to help ...
User avatar
google01103
Manager
Posts
6668
Karma
25

Re: kreenshot-editor discussion

Sat Apr 25, 2015 2:54 pm
gregormi wrote:<snip>
About sending to image hosting site: generally, you can use 'copy to clipboard' and then paste to the Plasma Pastebin widget. For now, the idea is to take the screenshot with ksnapshot, edit it with kreenshot-editor and then use ksnapshot's SendTo menu for further actions which include sending to various image hosting sites.

afik pastebin is not being ported to Plasma 5, it's not currently available (at least on my install) and it is not listed on todo.kde.org (the plasma addons section)


OpenSuse Leap 42.1 x64, Plasma 5.x

User avatar
gregormi
Registered Member
Posts
87
Karma
1
OS

Re: kreenshot-editor discussion

Sat Apr 25, 2015 7:41 pm
Mamarok wrote:I am still wondering: why not add these additional features to KSnapshot?

They will be eventually. On the one hand, kreenshot-editor is also designed as a shared library. On the other hand, there is work in progress to integrate external image editors better into ksnapshot. Rather than building this features hard-wired to ksnapshot it is better to make them reusable so that other application can benefit.

Mamarok wrote:I don't really see the point in having yet another image editor, if it is just that.

Seen from far above it is just another image editor. If you go into details you'll see there are (will be) some unique features. The role model is http://getgreenshot.org which is very popular.

google01103 wrote:afik pastebin is not being ported to Plasma 5, it's not currently available (at least on my install) and it is not listed on todo.kde.org (the plasma addons section)

As far as I know Pastebin is now called Quickshare in KF5.
User avatar
google01103
Manager
Posts
6668
Karma
25

Re: kreenshot-editor discussion

Sat Apr 25, 2015 8:51 pm
gregormi wrote:<snip>
As far as I know Pastebin is now called Quickshare in KF5.

not available (yet?) in my distro (update see on todo.kde.org it is in review/fix stage)

do you know the fate of Quicklaunch?


OpenSuse Leap 42.1 x64, Plasma 5.x

User avatar
gregormi
Registered Member
Posts
87
Karma
1
OS

Re: kreenshot-editor discussion

Sat Apr 25, 2015 9:16 pm
google01103 wrote:do you know the fate of Quicklaunch?

I don't know, sorry. I use it though.
User avatar
Sudhir Khanger
Registered Member
Posts
237
Karma
0
OS
User avatar
Mte90
Registered Member
Posts
108
Karma
0
OS

Re: kreenshot-editor discussion

Mon Apr 27, 2015 1:55 pm
I get this errors when i try to compile on my debian sid :(

Code: Select all
root@siduction:/home/mte90/Desktop/kde/kreenshot-editor# mkdir build; cd build; cmake ..
[INFO] root/start
-- The C compiler identification is GNU 4.9.2
-- The CXX compiler identification is GNU 4.9.2
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
[INFO] root/add_subdirectory(src)
[INFO] src/start
[INFO] src/find_package Qt5
[INFO] src/include_directories
[INFO] src/add_executable
[INFO] src/target_link_libraries
[INFO] src/add_subdirectory(lib/kreenshoteditor)
[INFO] lib/kreenshoteditor/start
[DEBUG] kreenshoteditor_FORMS=ui/settings/preferencesdialog.ui;ui/settings/pageoutput.ui
[DEBUG] kreenshoteditor_FORMS_HEADERS=/home/mte90/Desktop/kde/kreenshot-editor/build/src/lib/kreenshoteditor/ui_preferencesdialog.h;/home/mte90/Desktop/kde/kreenshot-editor/build/src/lib/kreenshoteditor/ui_pageoutput.h
[DEBUG] LIB_SUFFIX=
[DEBUG] env var XDG_DATA_DIRS=/usr/share:/usr/share:/usr/local/share
[DEBUG] env var from spec file KREEN_SPEC_ARCH=
[DEBUG] after WORKAROUND LIB_SUFFIX=
[INFO] src/lib/kreenshoteditor add_custom_target: prepare-include-files
[INFO] lib/kreenshoteditor/end
[INFO] src/end
[INFO] root/add_subdirectory(tests)
[INFO] tests
[INFO] CMAKE_SOURCE_DIR=/home/mte90/Desktop/kde/kreenshot-editor
[INFO] CMAKE_CURRENT_SOURCE_DIR=/home/mte90/Desktop/kde/kreenshot-editor/tests
[INFO] CMAKE_CURRENT_BINARY_DIR=/home/mte90/Desktop/kde/kreenshot-editor/build/tests
[INFO] root/end
-- Configuring done
CMake Warning (dev) at src/CMakeLists.txt:41 (add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "kreenshot-editor" links to target "Qt5::Network" but the target was
  not found.  Perhaps a find_package() call is missing for an IMPORTED
  target, or an ALIAS target is missing?
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at tests/CMakeLists.txt:25 (add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "Test1" links to target "Qt5::Network" but the target was not found.
  Perhaps a find_package() call is missing for an IMPORTED target, or an
  ALIAS target is missing?
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at tests/CMakeLists.txt:31 (add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "TestCore" links to target "Qt5::Network" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at tests/CMakeLists.txt:37 (add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "TestSettings" links to target "Qt5::Network" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at tests/CMakeLists.txt:43 (add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "TestUi" links to target "Qt5::Network" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at tests/CMakeLists.txt:49 (add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "TestWidgetsInteractive" links to target "Qt5::Network" but the
  target was not found.  Perhaps a find_package() call is missing for an
  IMPORTED target, or an ALIAS target is missing?
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Generating done
-- Build files have been written to: /home/mte90/Desktop/kde/kreenshot-editor/build


Code: Select all
root@siduction:/home/mte90/Desktop/kde/kreenshot-editor/build# make
Scanning dependencies of target kreenshot-editor_automoc
[  2%] Automatic moc for target kreenshot-editor
Generating moc_mainwindow.cpp
[  2%] Built target kreenshot-editor_automoc
Scanning dependencies of target prepare-include-files
[prepare-include-files.sh] start (first argument is INC_TARGET_DIR)
[prepare-include-files.sh] pwd=/home/mte90/Desktop/kde/kreenshot-editor/src/lib/kreenshoteditor
[prepare-include-files.sh] INC_TARGET_DIR=generated-include
/home/mte90/Desktop/kde/kreenshot-editor/src/lib/kreenshoteditor/prepare-include-files.sh: 18: /home/mte90/Desktop/kde/kreenshot-editor/src/lib/kreenshoteditor/pude-files.sh: Syntax error: "(" unexpected
src/lib/kreenshoteditor/CMakeFiles/prepare-include-files.dir/build.make:49: set di istruzioni per l'obiettivo "src/lib/kreenshoteditor/CMakeFiles/prepare-include riuscito
make[2]: *** [src/lib/kreenshoteditor/CMakeFiles/prepare-include-files] Errore 2
CMakeFiles/Makefile2:229: set di istruzioni per l'obiettivo "src/lib/kreenshoteditor/CMakeFiles/prepare-include-files.dir/all" non riuscito
make[1]: *** [src/lib/kreenshoteditor/CMakeFiles/prepare-include-files.dir/all] Errore 2
Makefile:127: set di istruzioni per l'obiettivo "all" non riuscito
make: *** [all] Errore 2


http://www.mte90.net/

Linux Counter.li #482442
User avatar
gregormi
Registered Member
Posts
87
Karma
1
OS

Re: kreenshot-editor discussion

Mon Apr 27, 2015 8:06 pm
donniezazen wrote:Have you seen HotShots?

Yes, I have seen it, thanks for the hint. See also a list of similar programs on kreenshot-editor's website: http://feinstaub.github.io/kreenshot-editor/#!index.md#Background. I tried it out several times. While the HotShot's image editor is quite feature-rich, there are two severe drawbacks:
1. The undo/redo buttons are always grayed out so it seems undo/redo is not implemented.
2. The command line interface does not work (at least on my maschine). So I cannot pass a file to be opened in the editor which would be needed to integrate it for example with ksnapshot.

Mte90 wrote:I get this errors when i try to compile on my debian sid :(

Thanks for trying to compile kreenshot-editor. Please, post technical issues as bug report here: https://bugs.kde.org/buglist.cgi?compon ... lution=---. I already did it with your report: https://bugs.kde.org/show_bug.cgi?id=346814 So, see there for further discussion.
User avatar
google01103
Manager
Posts
6668
Karma
25

Re: kreenshot-editor discussion

Mon Apr 27, 2015 8:24 pm
why not just call it and the executable kreenshot instead of kreenshot-editor?


OpenSuse Leap 42.1 x64, Plasma 5.x

User avatar
gregormi
Registered Member
Posts
87
Karma
1
OS

Re: kreenshot-editor discussion

Mon Apr 27, 2015 8:45 pm
google01103 wrote:why not just call it and the executable kreenshot instead of kreenshot-editor?

I wanted to make clear that kreenshot-editor is not clone of Greenshot (which includes screenshot capturing, editing and send to applications) but "only" an image editor specifically designed for screenshots. Having said that, I find the name a bit bumpy to type. Maybe this can be renamed later.
User avatar
simgunz
Registered Member
Posts
56
Karma
0
OS

Re: kreenshot-editor discussion

Tue Apr 28, 2015 7:20 am
I want also to report another KDE screenshot program with editing capabilities: Kaption (http://kde-apps.org/content/show.php/Ka ... ent=139302)
It may be of inspiration.


Bookmarks



Who is online

Registered users: Bing [Bot], Evergrowing, Google [Bot], ourcraft