Skip to content
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

IBX-4937: Content Tree has CSS issues when loading plenty of children under a node #692

Merged
merged 1 commit into from
Feb 8, 2023

Conversation

GrabowskiM
Copy link
Contributor

Question Answer
Tickets https://issues.ibexa.co/browse/IBX-4937
Bug fix? yes
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

This is fix for OSS version. For Content and above fix is in Tree Builder bundle:

To whomever testing this - after clicking enter for executing script, you can go make a tea, script to generate 2000 contents takes around 10 minutes. ;)

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

@GrabowskiM GrabowskiM force-pushed the IBX-4937-content-tree-many-items branch from 7ccad34 to c4b49e7 Compare February 8, 2023 09:13
@sonarcloud
Copy link

sonarcloud bot commented Feb 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@micszo micszo left a comment

Choose a reason for hiding this comment

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

QA Approved on Ibexa OSS 4.3.4-dev.

around 10 minutes

It wasn't that bad. 😉

@micszo micszo removed their assignment Feb 8, 2023
@dew326 dew326 merged commit 926e4ec into 4.3 Feb 8, 2023
@dew326 dew326 deleted the IBX-4937-content-tree-many-items branch February 8, 2023 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants