search

ProcessMaker / modeler

A vue.js based BPMN modeler
MIT License
85 stars 45 forks source link

observation/FOUR-17081: The process is not centered on the screen #1854

Closed agustinbusso closed 3 months ago

agustinbusso commented 3 months ago

Description

This PR fixes the following observation FOUR-17081

This PR implements a fix to center the process in the paper

Related PRs

package-ai PR modeler PR

ci:next

cypress[bot] commented 3 months ago

2 failed tests on run #1396 ↗︎

2 22 0 0 Flakiness 0

Details:

observation/FOUR-17081: The process is not centered on the screen
Project: modeler Commit: f76fe1e222
Status: Failed Duration: 06:19 💡
Started: Jul 10, 2024 7:08 PM Ended: Jul 10, 2024 7:14 PM
Failed  Modeler.cy.js • 2 failed tests View Output
Test Artifacts
Modeler > Create a simple process Test Replay Screenshots
Modeler > Hides element label on drag Test Replay Screenshots
Failed  IntermediateMessageThrowEvent.cy.js • 0 failed tests View Output
Test Artifacts
Failed  MessageFlows.cy.js • 0 failed tests View Output
Test Artifacts
Failed  UndoRedo.cy.js • 0 failed tests View Output
Test Artifacts
Failed  SequenceFlows.cy.js • 0 failed tests View Output
Test Artifacts

The first 5 failed specs are shown, see all 92 specs in Cypress Cloud.

Review all test suite changes for PR #1854 ↗︎

processmaker-sonarqube[bot] commented 3 months ago

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
0.0% 0.0% Coverage on New Code
0.0% 0.0% Duplication on New Code

See analysis details on SonarQube

agustinbusso commented 3 months ago

Implemented in this PR

https://github.com/ProcessMaker/modeler/pull/1855/files