Bug 55103 - why does the focus listener now get called after modifyTestElement?
Summary: why does the focus listener now get called after modifyTestElement?
Status: RESOLVED FIXED
Alias: None
Product: JMeter - Now in Github
Classification: Unclassified
Component: Main (show other bugs)
Version: Nightly (Please specify date)
Hardware: All All
: P2 minor (vote)
Target Milestone: ---
Assignee: JMeter issues mailing list
URL:
Keywords:
Depends on: 55027 55459
Blocks:
  Show dependency tree
 
Reported: 2013-06-15 17:28 UTC by Philippe Mouawad
Modified: 2013-08-21 12:34 UTC (History)
1 user (show)


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Mouawad 2013-06-15 17:28:24 UTC 
Milamber analysis:
The issue was introduce by his commit (works before, not after)

Author: pmouawad
Date: Wed Apr 17 20:16:53 2013
New Revision: 1469057

URL: http://svn.apache.org/r1469057
Log:
Bug 54834 - Improve Drag & Drop in the jmeter tree 
Bugzilla Id: 54834

Added:
    jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeTransferHandler.java   (with props)
Removed:
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/DragNDrop.java
Modified:
    jmeter/trunk/src/core/org/apache/jmeter/gui/MainFrame.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/ActionNames.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/tree/JMeterTreeListener.java
    jmeter/trunk/xdocs/changes.xml


sebb comment:
I agree that the fix to TestActionGui was sensible; it would have been a sensible fix even without the bug as the extra field was unnecessary.

However why does the focus listener now get called after modifyTestElement?

That might affect other GUI elements.
And it might affect future GUI coding where a focus listener might be required.
Comment 1 Philippe Mouawad 2013-06-15 17:29:10 UTC 
Affects nightly build after r1469057
Comment 2 Philippe Mouawad 2013-08-21 12:34:34 UTC 
Fixed by fix to 55459
Comment 3 The ASF infrastructure team 2022-09-24 20:37:54 UTC 
This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/3144