[ACS-9158] Remove 'View Details' button from node Details page #4351
Conversation
|
| @@ -461,7 +461,7 @@ export const canToggleSharedLink = (context: RuleContext): boolean => | |||
| * @param context Rule execution context | |||
| */ | |||
| export const canShowInfoDrawer = (context: RuleContext): boolean => | |||
| [hasSelection(context), navigation.isNotLibraries(context), navigation.isNotTrashcan(context)].every(Boolean); | |||
| [hasSelection(context), navigation.isNotLibraries(context), navigation.isNotTrashcan(context), navigation.isNotDetails(context)].every(Boolean); | |||
There was a problem hiding this comment.
In the context of the visibility rules PR, the canShowInfoDrawer rule and method will be removed in favour of individual array based rules. So the reference of this rule in app.extensions.json at line num 498 and 1057 will no longer be present once that PR is merged, and would be replaced with separate arrays as can be seen in that PR. Can you please update the code here so that instead of adding the navigation.isNotDetails(context) call here, its added as an element of the visible array in the json file itself. I see that this rule is already registered in aca-content.module.ts in this PR, so you only need to modify the rule in the json. The change should be something similar to
"visible": [ "canShowInfoDrawer", "app.navigation.isNotDetails" ]
https://hyland.atlassian.net/browse/ACS-9158
The "View Details" button has been removed from the Details page because the button's logic causes the page to redirect on refresh and the link cannot be accessed directly. At the same time "Reduce panel" button is retained and performs the same action.