Bug 344540 - Allow scene graphic items deletion by pressing the 'Delete' key
Summary: Allow scene graphic items deletion by pressing the 'Delete' key
Status: RESOLVED FIXED
Alias: None
Product: marble
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Rares
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2015-02-24 20:40 UTC by Cruceru Calin-Cristian
Modified: 2015-03-01 11:06 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Allow scene graphic items deletion by pressing the 'Delete' key of particular item (1.09 KB, patch)
2015-02-28 21:32 UTC, hardik beladiya
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Cruceru Calin-Cristian 2015-02-24 20:40:50 UTC
Currently, it is only possible to remove scene graphic items (polygons, polylines, placemarks) by right-clicking on them and pressing 'Remove'. Add the possibility to delete them by clicking the 'Delete' key.

The working directory is src/plugins/render/annotate and the targeted class is AnnotatePlugin. There is already an implementation of canceling focus using the Escape Key. Therefore, you can use that as an example.

To enable the Edit Maps toolset, right click the Menu Bar and check Edit Tours. A new dockwidget will show up on the left panel.

Submit a review request at https://git.reviewboard.kde.org with the marble groups as reviewers.

Please leave a comment here (e.g. "I want to work on this task.") if you start working on this task to avoid multiple people working on it at the same time.

Reproducible: Always
Comment 1 Rares 2015-02-24 21:45:24 UTC
I want to work on this task!
Comment 2 hardik beladiya 2015-02-28 21:32:52 UTC
Created attachment 91352 [details]
Allow scene graphic items deletion by pressing the 'Delete' key  of particular item

Tested the patch and it worked, in this patch if graphic item has focus we just find if it is node than we delete node othewise we delete polygon
Comment 3 Cruceru Calin-Cristian 2015-03-01 11:02:29 UTC
It has been already solved[0]. Sorry.

[0]: http://quickgit.kde.org/?p=marble.git&a=commit&h=05912deda09621911ca2d0729b5c878734282e93