![]() |
|
|
|
|
Change By:
|
Federico Grilli
(27/May/14 4:20 PM)
|
|
Description:
|
Some improvements have been made in MGNLUI-2004 which causes correct behaviour when you switch views - for example switching from the search results view to the tree view. But there are other cases that still do not work correctly.
Known cases:
* When you click on a favorite for an app that is already open, the list is not properly scrolled.
* When you use the Move action, the list is not properly scrolled.
The reason I consider this critical is that the above functionality appears to not be working properly, you see the app, but not the item that you acted on.
Having a selected item that is offscreen is dangerous because you could perform an action - and not know which item you are acting on.
Note 1:
For the Move case, I thought the problem might be the early return in TreeViewImpl.select:
if (firstItemId == null || treeTable.isSelected(firstItemId)) {
return;
}
Because basically it can't work with the above code because the item being moved was previously selected.
But even after commenting the 'return;' out it still did not work.
Note 2:
I notice that opening a bookmark does scroll to the item if the app is open - but a different subapp is open. It seems that something happens when the subapp switches which allows the treeTable.setCurrentPageFirstItemId(firstItemId); to work, but just when the subapp is already open (and somehow the state of the table has changed?) that the setCurrentPageFirstItemId does not work.
Note 3
Changes to the client-side of MagnoliaTreeTable connector made in MGNLUI-961 might have caused the current misbehaviour.
|
|
|
|
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
|
----------------------------------------------------------------
For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: <
[email protected]>
----------------------------------------------------------------
