[211] Don't display Profile test

Hello,

new to 211 is the menu option "Profile xxxx" on the test icons. If there is
no profiler installed, you display a message. Please, detect the presens of
the profiler before building the menu and don't show the menu option if
the profiler is not installed.


Sincerely,
Stefan Lieser


5 comments
Comment actions Permalink

Showing the menu item even when no profiler installed is intentional. Otherwise
how would user know about possiblity to profile tests and where to download
the profiler?

Valentin Kipiatkov
Chief Scientist, Vice President of Product Development
JetBrains, Inc
http://www.jetbrains.com
"Develop with pleasure!"

Hello,

new to 211 is the menu option "Profile xxxx" on the test icons. If
there is no profiler installed, you display a message. Please, detect
the presens of the profiler before building the menu and don't show
the menu option if the profiler is not installed.

Sincerely,
Stefan Lieser



0
Comment actions Permalink

"Valentin Kipiatkov (JetBrains)" <valentin@jetbrains.com> schrieb im
Newsbeitrag news:3fdb29a69c2888c7cdf89490afc7@news.intellij.net...

Showing the menu item even when no profiler installed is intentional.
Otherwise how would user know about possiblity to profile tests and where
to download the profiler?


;) Ok, you want to advertise your profiler.... What about an option to
disable the advertising?


Sincerely,
Stefan Lieser


0
Comment actions Permalink

;) Ok, you want to advertise your profiler.... What about an option
to disable the advertising?


Well, we can add a checkbox into the dialog shown that profiler is not installed.
But is the menu item really annoying?

Valentin Kipiatkov
Chief Scientist, Vice President of Product Development
JetBrains, Inc
http://www.jetbrains.com
"Develop with pleasure!"

"Valentin Kipiatkov (JetBrains)" <valentin@jetbrains.com> schrieb im
Newsbeitrag news:3fdb29a69c2888c7cdf89490afc7@news.intellij.net...

>> Showing the menu item even when no profiler installed is intentional.
>> Otherwise how would user know about possiblity to profile tests and
>> where to download the profiler?
>>

;) Ok, you want to advertise your profiler.... What about an option
to disable the advertising?

Sincerely,
Stefan Lieser



0
Comment actions Permalink

Well, we can add a checkbox into the dialog shown that profiler is not
installed. But is the menu item really annoying?


Yes, for me it is. At least the menu should be disabled to give a fast
response that this function is not accessible.


Sincerely,
Stefan Lieser


0
Comment actions Permalink

Ok, we'll add a checkbox.

Valentin Kipiatkov
Chief Scientist, Vice President of Product Development
JetBrains, Inc
http://www.jetbrains.com
"Develop with pleasure!"

>> Well, we can add a checkbox into the dialog shown that profiler is
>> not installed. But is the menu item really annoying?
>>

Yes, for me it is. At least the menu should be disabled to give a fast
response that this function is not accessible.

Sincerely,
Stefan Lieser



0

Please sign in to leave a comment.