Bug 801399 - Designer blocked after viewing process diff if the diff pop-up is blocked
Summary: Designer blocked after viewing process diff if the diff pop-up is blocked
Keywords:
Status: VERIFIED
Alias: None
Product: JBoss Enterprise BRMS Platform 5
Classification: JBoss
Component: BRM (Guvnor)
Version: BRMS 5.3.0.GA
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ER5
: BRMS 5.3.0.GA
Assignee: manstis
QA Contact: Jiri Locker
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-03-08 12:56 UTC by Jiri Locker
Modified: 2021-04-09 12:47 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Firefox 3.6; 8
Last Closed:
Type: Bug


Attachments (Terms of Use)
blockedDesigner.png (77.84 KB, image/png)
2012-03-08 12:56 UTC, Jiri Locker
no flags Details
updateddiffwindow (96.35 KB, image/png)
2012-03-15 01:10 UTC, Tihomir Surdilovic
no flags Details

Description Jiri Locker 2012-03-08 12:56:15 UTC
Created attachment 568615 [details]
blockedDesigner.png

Description of problem:
In Firefox, pop-ups are blocked by default. In case the user has not yet allowed pop-ups for Guvnor's host and tries to view a process version diff, error message is displayed and designer's workspace is blocked forever. There is no way to cancel the operation. The only way to continue editing the process is to close the process and open it again. Fortunately, any unsaved work can be saved.

Version-Release number of selected component (if applicable):
BRMS 5.3.0 ER4

How reproducible:
if pop-ups are blocked for Guvnor's host (is default in Firefox)

Steps to Reproduce:
1. make sure pop-ups are blocked and exception for localhost has not been added yet
2. access designer by opening any BPMN2 process or by creating a new one in Guvnor
3. if you have just created the process, save it first so that at least one version is checked in
4. try viewing any version diff, the result should match the attached screenshot
  
Actual results:
"Creating diff..." progress message never disappears and blocks the designer workspace forever

Expected results:
option 1) In case of failure to view the diff (popup is blocked) the progress message should disappear so that user can dismiss the error message and continue working with designer

option 2) don't use pop-ups to view the diff, use HTML the same way it is used to view BPMN2 or JSON source of the process

Additional info:

Comment 1 Tihomir Surdilovic 2012-03-15 01:09:47 UTC
Diff no longer creates a popup and displays the diff inline (see attachment). Fix is checked into Designer master and 2.1.x branch. let me know if you need a war for testing provided.

Comment 2 Tihomir Surdilovic 2012-03-15 01:10:15 UTC
Created attachment 570138 [details]
updateddiffwindow

Comment 3 Jiri Locker 2012-03-15 09:48:21 UTC
I will test with designer in ER5.

Comment 4 Jiri Locker 2012-05-04 15:19:22 UTC
Verified in ER6. Much better, thanks!


Note You need to log in before you can comment on or make changes to this bug.