Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 418298

Summary:

F3 should always open the selected element in its corresponding editor

Product:

[Technology] Jubula

Reporter:

Sebastian Struckmann <sebastian.jubula>

Component:

Assignee:

Marvin Mueller <marvin_m>

Status:

CLOSED FIXED

QA Contact:

Oliver Goetz <Oliver.Goetz>

Severity:

enhancement

Priority:

CC:

Achim.Loerke, alexandra.schladebeck, marvin_m

Version:

unspecified

Keywords:

triaged

Target Milestone:

Kepler SR2

Hardware:

OS:

All

Whiteboard:

Bug Depends on:

Bug Blocks:

418310

Attachments:

Description	Flags
Patch for Bug 418298	none
a simpler solution	marvin_m: iplog+
patch	marvin_m: iplog+

Description Sebastian Struckmann

2013-09-30 03:47:17 EDT

Opening the specification of an element via F3 only opens the corresponding editor when the selected element itself is not a specification. This might be confusing for a user who is using F3 in the test case browser or test suite browser when he wants to open a test case, a test suite or a test job. Therefore F3 should open the corresponding editor to the selected element.

Comment 1 Sebastian Struckmann

2013-09-30 03:49:35 EDT

Created attachment 235946 [details]
Patch for Bug 418298

Comment 2 Sebastian Struckmann

2013-09-30 11:09:00 EDT

Created attachment 235976 [details]
a simpler solution

Comment 3 Alexandra Schladebeck

2013-10-07 03:16:28 EDT

If this can be patched for 2.2, that would be good.

Comment 4 Marvin Mueller

2013-10-31 05:21:29 EDT

I have to delete one line which leads to a bug that the Execs in the Test Suite Browser could not be opened.

Reviewed and applied with changeset: http://git.eclipse.org/c/jubula/org.eclipse.jubula.core.git/commit/?id=010cb8e60c1f926ff1a1ab41343b18162a54ace2

Comment 5 Marvin Mueller

2013-11-04 05:09:58 EST

Our tests have shown us a misbehavior of opening via F3 and also there is a problem that double click is not working in some cases. F3 should open the specification. If you have a new version feel free to add the patch.

Comment 6 Sebastian Struckmann

2013-11-04 06:31:14 EST

Created attachment 237166 [details]
patch

I created a new patch.

Comment 7 Marvin Mueller

2013-11-05 03:05:28 EST

Reviewed and applied. The behavior now seems to be correct.

I have contributed your work with changeset:http://git.eclipse.org/c/jubula/org.eclipse.jubula.core.git/commit/?id=0f5c0ce25eb222f529e906488c4a3c4bc14f8340

Comment 8 Oliver Goetz

2013-11-06 03:27:34 EST

Tested.