[GitHub] [jmeter] FSchumacher commented on pull request #610: Add a switch to disable splash screen on startup

[GitBox <gi...@apache.org>]

FSchumacher commented on pull request #610:
URL: https://github.com/apache/jmeter/pull/610#issuecomment-676577785


   I haven't tried, what the splash screen does, when it is not instructed to be always on top. I fear, that it can come up behind the main window and irritate people, as JMeter will seem to stop being responsive until the (possibly hidden?) splash screen is removed.
   But I am really not sure, what the correct thing is.
   On the other hand, I think the real problems here are not the splash screen. If it would show for two seconds and then go away (as it usually does), nobody would really complain. This issue (the bugzilla one) has been opened as we have a lot of problems working together.
   
   * The splash screen stays always on top (or is displayed at all)
   * Every added element to the tree leads to an initialization of the complete tree with all already added and initialized elements
   * The construction of test elements via reflection seems to be slow (especially when a lot of elements are created serially)
   * (and this is the real killer) The usage of undo feature is used while constructing the tree on load and leads to a real bad performance
   
   So, all in all, I am pretty sure, that this PR is more of a cosmetic change and we should really focus on the other problems in that list.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org