-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[6.1] Implementation of graphical workflow editor feature #46021
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 6.1-dev
Are you sure you want to change the base?
Conversation
…x method (ApiController not configured yet)
…ccessibility improved
|
I can see that you have tried to make this accessible and supporting screen readers and I applaud you for trying. But have you actually tested it with a screen reader as I don't think it's working as you expect. I'm not even sure if it is even realistically achievable to support screen readers in this visual presentation |
…g the edge. The edge now only responds to mouseenter and mouseleave events for hover effects, ensuring a more intuitive user experience.
|
In the front end view some of the keyboard shortcuts still work but they are not documented. Also you dont get the mini map at the bottom left that you do in the admin |
Thanks for pointing out, I found a small bug, will fix it soon. |
We thought that the view only panel should be as minimal as possible as it has no other use case other than just viewing the connections and the flow, so we felt the mini-map is redundant, do we really need it? |
whatever the justification is for having it in the main admin view has not changed in the site view or article view. |
administrator/components/com_workflow/resources/scripts/components/canvas/ControlsPanel.vue
Show resolved
Hide resolved
administrator/components/com_workflow/resources/scripts/components/canvas/CustomControls.vue
Show resolved
Hide resolved
|
Please revert the last commit updating tinymce |
This reverts commit cd68c7b.
|
Thanks for fixing the transitions all being placed on top of each other |
|
The hidden screenreader text uses "published/unpublished" but the visible text uses "enabled/disabled" |





Pull Request for Issue # .
Summary of Changes
Added a Vue.js based Workflow Graph:
View only Graph:
Testing Instructions
1. Workflow Graph Editor (Backend)
Login as Administrator.
Enable Workflows in Article Integration and navigate to the Workflows page.
You should see a Graph button - click it.
A Workflow Graph page should open showing all workflow details including stages and transitions.
Verify the following:
Editing: If you have permissions, edit options should appear in the menu of each stage/transition.
Keyboard Shortcuts: Check navigation and available shortcuts (reference button at the top-right).
Adding Items:
Layout Changes: Drag stages to rearrange them - the layout should auto-save.
Zoom Controls: Test zoom in, zoom out, and fit-to-view using buttons at the bottom-right.
2. View-Only Graph (Frontend)
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Link to documentations
Please select:
Documentation link for docs.joomla.org: Link
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed