As a user I can interact within a content app and all components (workbench, actionbar, details) are up to date so that I am not confused (MGNLUI-1288)

[MGNLUI-897] All content views are refreshed upon workbench refresh Created: 15/Mar/13  Updated: 17/May/13  Resolved: 16/May/13

Status: Closed
Project: Magnolia UI
Component/s: framework
Affects Version/s: 5.0
Fix Version/s: 5.0

Type: Sub-task Priority: Critical
Reporter: Mikaël Geljić Assignee: Mikaël Geljić
Resolution: Fixed Votes: 0
Labels: workbench
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
relation
is related to MGNLUI-901 Apps are not fully destroyed after th... Closed
Template:
Date of First Response:
Sprint: Beta 2

 Description   

If multiple workbench subapps are open simultaneously, e.g. Security app + Contacts app = 5 Workbenches; then they are all updated when performing a repository operation in one of them.

ContentChangeEvents are fired on admincentral event bus, and are then listened to in the ContentWorkbenchPresenter(s). The problem is that this handler also refreshes selection and updates action bar preview, so it will try to do it as well in inactive subapps, whose selection should be left untouched, and which might even operate on a different workspace.



 Comments   
Comment by Tobias Mattsson [ 10/Apr/13 ]

ContentChangeEvents should be fired on admincentral event bus, no problem there. The event is intended to allow apps to update their view state when content is changed elsewhere. Apps must not use it as a way to coordinate selection between the different components of an app.

Comment by Tobias Mattsson [ 16/May/13 ]

As part of MGNLUI-1362 this was fixed as the event now refreshes the workbench only if the change is for the current workspace and doesn't modify selection (unless the selected item was removed).

Comment by Mikaël Geljić [ 16/May/13 ]

Commits on that ticket used to prevent refreshing all 3/4 content views in each workbench, but only the active content view instead.

This is still valid: WorkbenchPresenter only refreshes its active content presenter.

As for the more global issue, this was indeed fixed in MGNLUI-1362 as Tobias mentioned.

Will rename this issue to the more specific content-view pb.

Generated at Mon Feb 12 08:41:23 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.