[ 
https://issues.apache.org/jira/browse/ISIS-909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dan Haywood updated ISIS-909:
-----------------------------
    Fix Version/s:     (was: 2.3.0)
                   1.20.0
      Description: 
See also ISIS-842.

 

~~~

There is a logic [1] at BookmarkedPagesPanel.java that tries to detect whether 
an item in the bookmarks sliding menu is the currently viewed/edited item and 
sets CSS class "currentBookmark".

Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. 
It uses Objects.equal(current, pageParameters) and often those have similar oid 
but additional parameters make the check failing.

1. 
[https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167]

  was:
There is a logic [1] at BookmarkedPagesPanel.java that tries to detect whether 
an item in the bookmarks sliding menu is the currently viewed/edited item and 
sets CSS class "currentBookmark".

Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. 
It uses Objects.equal(current, pageParameters) and often those have similar oid 
but additional parameters make the check failing.

1. 
https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167


> BookmarksPagesPanel often cannot detect the current/active model
> ----------------------------------------------------------------
>
>                 Key: ISIS-909
>                 URL: https://issues.apache.org/jira/browse/ISIS-909
>             Project: Isis
>          Issue Type: Bug
>          Components: Core: Viewer: Wicket
>    Affects Versions: viewer-wicket-1.6.0
>            Reporter: Martin Grigorov
>            Priority: Minor
>             Fix For: 1.20.0
>
>
> See also ISIS-842.
>  
> ~~~
> There is a logic [1] at BookmarkedPagesPanel.java that tries to detect 
> whether an item in the bookmarks sliding menu is the currently viewed/edited 
> item and sets CSS class "currentBookmark".
> Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. 
> It uses Objects.equal(current, pageParameters) and often those have similar 
> oid but additional parameters make the check failing.
> 1. 
> [https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to