This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 204378 - Previous/Next/Cancel buttons are inconsistent in the Exception window
Summary: Previous/Next/Cancel buttons are inconsistent in the Exception window
Status: NEW
Alias: None
Product: ide
Classification: Unclassified
Component: Logger (show other bugs)
Version: 7.1
Hardware: PC Other
: P3 normal (vote)
Assignee: Martin Entlicher
URL:
Keywords: UI
Depends on:
Blocks:
 
Reported: 2011-10-29 02:58 UTC by 280Z28
Modified: 2014-08-20 12:24 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description 280Z28 2011-10-29 02:58:58 UTC
The Exception window is able to show information for multiple exceptions that have occurred. To switch between exceptions, the buttons "Previous", "Next", and "Cancel" are available. In the current implementation, buttons are hidden rather than disabled when the option isn't available. For example, when viewing the first exception the Previous button is hidden.

The dynamic visibility is causing a number of problems, all of which can be resolved by using a disabled state rather than hidden visibility for the buttons whose actions are not available.

1. When 3 (or more) exceptions are displayed and the first is showing, then "Next" and "Cancel" buttons are the same size. When the user presses "Next", the "Previous" button appears and all three buttons resize to fit the word "Previous". This is especially problematic if the user pressing on the Next button twice to move forward 2 items - the resizing will result in inadvertantly clicking "Cancel".

2. When 3 (or more) exceptions are present, and the user presses "Previous" to move back 2 items, the user will actually press "Previous"-"Next" due to an extra button appearing under the cursor.
Comment 1 Tomas Danek 2013-06-03 09:18:47 UTC
passing to platform, since this dialog is a level below exception reporter.
Comment 2 Stanislav Aubrecht 2013-06-03 13:23:53 UTC
(In reply to comment #1)
> passing to platform, since this dialog is a level below exception reporter.

The exception reporter shows/hides the Previous button, does it not?
Comment 3 Marian Mirilovic 2013-07-11 09:24:56 UTC
(In reply to comment #2)
> (In reply to comment #1)
> > passing to platform, since this dialog is a level below exception reporter.
> 
> The exception reporter shows/hides the Previous button, does it not?

I don't think so Standa (but might be wrong) ... I had removed ER from the IDE and risen an Exception -> the behaviour decribed by reporter was there as well. 
I would really vote for switching from shown/hidden to enabled/disabled buttons.


Another problem is that sometimes (in this case just with ER in), when you'll get >1 exceptions at once (in Unexpected Exception) dialog at once, and this dialog comes for the first time -> it's too small (not all buttons fit into the layout), so some of them are screwed-up. I think it was already reporterd, but I can't find the issue.
Comment 4 Stanislav Aubrecht 2013-07-11 09:32:03 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > (In reply to comment #1)
> > > passing to platform, since this dialog is a level below exception reporter.
> > 
> > The exception reporter shows/hides the Previous button, does it not?
> 
> I don't think so Standa (but might be wrong) ... I had removed ER from the IDE
> and risen an Exception -> the behaviour decribed by reporter was there as well. 
Exception reporter doesn't use Wizard API so the previous/next buttons are provided by the exception dialog. It doesn't use any Dialogs API for that.

> I would really vote for switching from shown/hidden to enabled/disabled
> buttons.

+1

> 
> 
> Another problem is that sometimes (in this case just with ER in), when you'll
> get >1 exceptions at once (in Unexpected Exception) dialog at once, and this
> dialog comes for the first time -> it's too small (not all buttons fit into the
> layout), so some of them are screwed-up. I think it was already reporterd, but
> I can't find the issue.
I think it's just another manifestation of this problem. Call Swing API from EDT only and it should be fine.
Comment 5 Ondrej Vrabec 2013-07-11 10:21:16 UTC
the dialog is o.n.core/src/org/netbeans/core/NotifyExcPanel.java and it belongs to ide/logger probably