Skip to content
This repository has been archived by the owner on Sep 13, 2022. It is now read-only.

[6.5.x] JBPM-5217 - Link each element of the process model to its location in the process documentation #369

Merged
merged 1 commit into from
Jul 22, 2016

Conversation

tsurdilo
Copy link

(JS changes only)

@JLindop : adds a new button in the shape menu for each node in the process.
when the new documentation icon in shape menu is clicked, the tab changes from process modeling to process documentation and the page scrolls the location where this element is defined.

Note: for sequence flows we only show sequence flows in the docs that have either their name, documentation or conditionexpression properties set. If the shape menu docs button is clicked on a sequence flow which does not have at least one of these elements set, designer shows a notification telling the user that element does not exist in the docs and does not switch to the docs tab. if the sequence flow has at least one of these elements the same applies as for any other node type.

this.facade.raiseEvent({
type : ORYX.CONFIG.EVENT_NOTIFICATION_SHOW,
ntype : 'info',
msg : ORYX.I18N.ShapeMenuPlugin.unableToFindNodeSource,
Copy link

@ghost ghost Jul 22, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should line 343 be ORYX.I18N.ShapeMenuPlugin.unableToLinkElementToDocs ?
For this constant, maybe "Element not included in documentation" might be better, as the current message "Unable to..." makes it sound as though there's an error.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah that's better wording. Will change

@tsurdilo
Copy link
Author

@JLindop amended per your code comment

@ghost
Copy link

ghost commented Jul 22, 2016

+1

@csadilek csadilek merged commit d740bbf into kiegroup:6.5.x Jul 22, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants