Moving in the trace using 6 rewrites the time delay

Bug #808276 reported by Jiri Srba
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
TAPAAL
Fix Released
Medium
Unassigned
2.0
Won't Fix
Medium
Unassigned
2.1
Fix Released
Medium
Unassigned

Bug Description

Open the introduction net, click verify, simulation trace appears. Click once the arrow to the right moving one step
forward. Continue pressing 6 for moving forward. When you arrive at the end of the trace, the time delay get rewritten
with 66666666

Related branches

Jiri Srba (srba)
Changed in tapaal:
milestone: 2.0.1 → 2.1
Revision history for this message
Morten Jacobsen (mortenja) wrote :

This is not reproducible on Windows, though i remember it being a problem on ubuntu. However, i think it is a "feature" of the Swing implementation's focus handling and not a problem with our code.

Revision history for this message
Jiri Srba (srba) wrote :

Well, I just tried on Windows XP and the same problem is there. Did you, as the very first thing in the simulation mode
with some trace, try to click with mouse on the right arrow (just to the left of the delay field) and then continue pressing 6 until the
end of the trace and the delay then starts to be rewritten.

Revision history for this message
Morten Jacobsen (mortenja) wrote :

I see, it is only a problem when you have given focus to the right arrow with the mouse button. The problem is the that when you reach the end of the trace, the right button is disabled, at which point swing automatically moves focus to the next component, which is the textfield.. I am not sure what to do about this, besides choosing new shortcut keys, since it is a feature in Swing itself and not our code.

Revision history for this message
Jiri Srba (srba) wrote :

Renaming the 4 and 6 to "F" (forward) and "B" (backward) could be a solution, however, on should
make sure that the delay field does nto accept any nonnumering letters (apart from a dot). As it
is now even F would be written to the delay field when reaching the end of the trace.

Changed in tapaal:
status: New → Triaged
Jiri Srba (srba)
Changed in tapaal:
assignee: nobody → Kenneth Yrke Jørgensen (yrke)
Changed in tapaal:
assignee: Kenneth Yrke Jørgensen (yrke) → nobody
assignee: nobody → Kenneth Yrke Jørgensen (yrke)
Changed in tapaal:
milestone: 2.1 → none
status: Triaged → Confirmed
assignee: Kenneth Yrke Jørgensen (yrke) → nobody
Jiri Srba (srba)
Changed in tapaal:
status: Confirmed → Triaged
Revision history for this message
Jiri Srba (srba) wrote :

This has been fixed by replacing 4 and 6 with arrows up and down.

Changed in tapaal:
status: Triaged → Fix Committed
Changed in tapaal:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.