-
Notifications
You must be signed in to change notification settings - Fork 1
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
LPD-33712 Increase z-index for maximized editor #4402
base: master
Are you sure you want to change the base?
LPD-33712 Increase z-index for maximized editor #4402
Conversation
CI is automatically triggering the following test suites:
|
❌ ci:test:sf - 0 out of 1 jobs passed in 3 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPD-33712 1 Failed Jobs:For more details click here.format-source-files: [java] java.lang.Exception: Found 2 formatting issues: [java] 1: Chaining on 'jsonObject.put' is preferred: ./modules/apps/frontend-editor/frontend-editor-ckeditor-sample-web/src/main/java/com/liferay/frontend/editor/ckeditor/sample/web/internal/editor/configuration/ClassicEditorConfigContributor.java 39 (Checkstyle:ChainingCheck) [java] 2: Missing test.properties in /opt/dev/projects/github/liferay-portal/modules/test/playwright/tests/frontend-editor-ckeditor-sample-web: ./modules/test/playwright/tests/frontend-editor-ckeditor-sample-web/config.ts (SourceCheck:TSConfigFileCheck) [java] [java] at com.liferay.source.formatter.SourceFormatter.format(SourceFormatter.java:469) [java] at com.liferay.source.formatter.SourceFormatter.main(SourceFormatter.java:301) [stopwatch] [run.batch.test.action: 1:47.018 sec] [echo] The following error occurred while executing this line: [echo] /opt/dev/projects/github/liferay-portal/portal-impl/build.xml:497: The following error occurred while executing this line: [echo] /opt/dev/projects/github/liferay-portal/portal-impl/build.xml:650: Java returned: 1 [get] Getting: http://test-1-8/job/test-portal-source-format/7083//consoleText [get] To: /opt/dev/projects/github/liferay-portal/20240903120525273.txt [delete] Deleting: /opt/dev/projects/github/liferay-portal/20240903120525273.txt [typedef] Could not load definitions from resource org/apache/maven/artifact/ant/antlib.xml. It could not be found. [taskdef] Could not load definitions from resource org/jacoco/ant/antlib.xml. It could not be found. [delete] Deleting: /opt/dev/projects/github/liferay-portal/null788105494.properties |
Jenkins Build:test-portal-source-format#7083 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-frontend#4402 Testray Routine:EE Pull Request Testray Build:[master] ci:test:sf - fortunatomaldonado > liferay-frontend - PR#4402 - 2024-09-03[12:02:10] Testray Build ID:79206885 Testray Importer:publish-testray-report#5007 |
1cc5b47
to
c4faa05
Compare
ci:test:sf |
❌ ci:test:sf - 0 out of 1 jobs passed in 5 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPD-33712 1 Failed Jobs:For more details click here.format-source-files: [java] java.lang.Exception: Found 1 formatting issues: [java] 1: Missing test.properties in /opt/dev/projects/github/liferay-portal/modules/test/playwright/tests/frontend-editor-ckeditor-sample-web: ./modules/test/playwright/tests/frontend-editor-ckeditor-sample-web/config.ts (SourceCheck:TSConfigFileCheck) [java] [java] at com.liferay.source.formatter.SourceFormatter.format(SourceFormatter.java:469) [java] at com.liferay.source.formatter.SourceFormatter.main(SourceFormatter.java:301) [stopwatch] [run.batch.test.action: 3:01.772 sec] [echo] The following error occurred while executing this line: [echo] /opt/dev/projects/github/liferay-portal/portal-impl/build.xml:497: The following error occurred while executing this line: [echo] /opt/dev/projects/github/liferay-portal/portal-impl/build.xml:650: Java returned: 1 [get] Getting: http://test-1-32/job/test-portal-source-format/8917//consoleText [get] To: /opt/dev/projects/github/liferay-portal/20240903123312679.txt [delete] Deleting: /opt/dev/projects/github/liferay-portal/20240903123312679.txt [typedef] Could not load definitions from resource org/apache/maven/artifact/ant/antlib.xml. It could not be found. [taskdef] Could not load definitions from resource org/jacoco/ant/antlib.xml. It could not be found. [delete] Deleting: /opt/dev/projects/github/liferay-portal/null186945728.properties |
Jenkins Build:test-portal-source-format#8917 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-frontend#4402 Testray Routine:EE Pull Request Testray Build:[master] ci:test:sf - fortunatomaldonado > liferay-frontend - PR#4402 - 2024-09-03[12:28:06] Testray Build ID:79224537 Testray Importer:publish-testray-report#33646 |
c4faa05
to
efcb228
Compare
ci:test:sf |
ci:test:relevant |
✔️ ci:test:stable - 27 out of 27 jobs passed✔️ ci:test:relevant - 33 out of 33 jobs passed in 1 hour 12 minutesClick here for more details.Base Branch:Branch Name: master Upstream Comparison:Branch GIT ID: 297100fc337aa03c975698b6189db55a8ce26d4d ci:test:stable - 27 out of 27 jobs PASSED27 Successful Jobs:
ci:test:relevant - 33 out of 33 jobs PASSED33 Successful Jobs:
For more details click here.Test bundle downloads:
|
Jenkins Build:test-portal-acceptance-pullrequest(master)#17831 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-frontend#4402 Testray Routine:EE Pull Request Testray Build:[master] ci:test:relevant - fortunatomaldonado > liferay-frontend - PR#4402 - 2024-09-03[14:11:55] Testray Build ID:79317093 Testray Importer:publish-testray-report#38586 |
efcb228
to
bd2099a
Compare
ci:test:sf |
8ec45ca
to
c238244
Compare
ci:test:sf |
❌ ci:test:sf - 0 out of 1 jobs passed in 4 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPD-33712 1 Failed Jobs:For more details click here.format-source-files: [java] java.lang.Exception: Found 1 formatting issues: [java] 1: ./test.properties expected:<...# Gradle Plugins [java] ## [java] [] [java] gradle.plugins....> but was:<...# Gradle Plugins [java] ## [java] [ ] [java] gradle.plugins....> [java] [java] at com.liferay.source.formatter.SourceFormatter.format(SourceFormatter.java:473) [java] at com.liferay.source.formatter.SourceFormatter.main(SourceFormatter.java:302) [stopwatch] [run.batch.test.action: 1:47.190 sec] [echo] The following error occurred while executing this line: [echo] /opt/dev/projects/github/liferay-portal/portal-impl/build.xml:497: The following error occurred while executing this line: [echo] /opt/dev/projects/github/liferay-portal/portal-impl/build.xml:650: Java returned: 1 [get] Getting: http://test-1-27/job/test-portal-source-format/7274//consoleText [get] To: /opt/dev/projects/github/liferay-portal/20241003095507671.txt [delete] Deleting: /opt/dev/projects/github/liferay-portal/20241003095507671.txt [typedef] Could not load definitions from resource org/apache/maven/artifact/ant/antlib.xml. It could not be found. [taskdef] Could not load definitions from resource org/jacoco/ant/antlib.xml. It could not be found. [delete] Deleting: /opt/dev/projects/github/liferay-portal/null561962277.properties |
Jenkins Build:test-portal-source-format#7274 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-frontend#4402 Testray Routine:EE Pull Request Testray Build:[master] ci:test:sf - fortunatomaldonado > liferay-frontend - PR#4402 - 2024-10-03[09:51:48] Testray Build ID:30952434 Testray Importer:publish-testray-report#27770 |
ci:test:sf |
✔️ ci:test:sf - 1 out of 1 jobs passed in 4 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPD-33712 1 Successful Jobs:For more details click here. |
Jenkins Build:test-portal-source-format#3066 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-frontend#4402 Testray Routine:EE Pull Request Testray Build:[master] ci:test:sf - fortunatomaldonado > liferay-frontend - PR#4402 - 2024-10-03[10:46:25] Testray Build ID:31020938 Testray Importer:publish-testray-report#12566 |
ci:test:relevant |
✔️ ci:test:stable - 25 out of 25 jobs passed✔️ ci:test:relevant - 32 out of 32 jobs passed in 55 minutesClick here for more details.Base Branch:Branch Name: master Upstream Comparison:Branch GIT ID: 285e3aa41c3c8f2ff4c64008eb8e04c2d71e4fe4 ci:test:stable - 25 out of 25 jobs PASSED25 Successful Jobs:
ci:test:relevant - 32 out of 32 jobs PASSED32 Successful Jobs:
For more details click here.Test bundle downloads: |
Jenkins Build:test-portal-acceptance-pullrequest(master)#18115 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-frontend#4402 Testray Routine:EE Pull Request Testray Build:[master] ci:test:relevant - fortunatomaldonado > liferay-frontend - PR#4402 - 2024-10-03[11:02:55] Testray Build ID:31189456 Testray Importer:publish-testray-report#39227 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fortunatomaldonado LGTM! ✨
@@ -2,7 +2,7 @@ | |||
## Playwright | |||
## | |||
|
|||
playwright.test.project[playwright-js-tomcat90-mysql57-jdk8][relevant][frontend-editor-rule]=frontend-editor-ckeditor-sample-web | |||
playwright.test.project[playwright-js-tomcat90-mysql57-jdk8][relevant][frontend-editor-rule]=frontend-editor-ckeditor-web |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
Requries #4401
https://liferay.atlassian.net/browse/LPP-55194
https://liferay.atlassian.net/browse/LPD-33712
With the update of CKEditor v.4.21.0-liferay.4, the following commits resolves the stylescombo not being visible when editor is maximized.
Please let me know if there are any questions or comments about this.
Thank you!