[MGNLUI-3251] Inefficient component hierarchies in pulse list views Created: 11/Nov/14  Updated: 06/Aug/15  Resolved: 27/Nov/14

Status: Closed
Project: Magnolia UI
Component/s: pulse
Affects Version/s: 5.3.5
Fix Version/s: 5.3.6

Type: Bug Priority: Major
Reporter: Mikaël Geljić Assignee: Mikaël Geljić
Resolution: Fixed Votes: 0
Labels: support
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Cloners
is cloned by MGNLUI-3382 CLONE - Inefficient component hierarc... Closed
relation
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

While looking into MGNLUI-3151, it appeared we generate a fair bunch of Vaadin components in the Pulse table cells, for mainly plain text/icons.

This is acknowledged to cause performance issues with the Vaadin Table, when dealing with big amounts of rows.

As a matter of fact, we've already stopped doing this in content app lists/trees for a long time.



 Comments   
Comment by Mikaël Geljić [ 20/Nov/14 ]

cloned ticket for porting to 5.4; since this will need to happen after vaadin upgrade itself is integrated.

Removing 5.4 from fixVersions here.

Comment by Michael Mühlebach [ 26/Nov/14 ]

The pulse view has now a small glitch in Google Chrome: The separator line of the tabs (Tasks, Messages) is too far below and strikes through the state titles (New, Assigned, etc).

To reproduce: Reload the page directly in the pulse view: .../.magnolia/admincentral#shell:pulse:;
On switching to the launcher and back the formatting is correct again.

Comment by Mikaël Geljić [ 27/Nov/14 ]

Fixed the CSS failure directly:
I removed the 'hidden' placeholder for bottom border of top navigator in the Pulse, fixing/cleaning CSS accordingly;
I also cleaned up static builders in PulseItemCategoryNavigator component, where plain constructor does just fine.

Generated at Mon Feb 12 09:04:44 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.