[MGNLUI-6599] Scroll to selected item does not always work within an app Created: 05/Mar/21  Updated: 10/Jun/21  Resolved: 22/Apr/21

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 6.2.6, 6.2.7
Fix Version/s: 6.2.8

Type: Bug Priority: Neutral
Reporter: Simon Lutz Assignee: Adam Siska
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: 2h 7m
Original Estimate: Not Specified

Issue Links:
Relates
relates to MGNLUI-6242 Scroll to selected item does not work... Closed
relates to MGNLUI-6509 Item added via detail SubApp is not s... Closed
relates to PAGES-438 Pages editor should remember last pos... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[X]* Doc/release notes changes? Comment present?
[X]* Downstream builds green?
[X]* Solution information and context easily available?
[X]* Tests
[X]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
[X]* http://nightly.magnolia-cms.com/.magnolia/admincentral#app:dam:jcrBrowser;/stories-demo/found-in-switzerland/sylwia-bartyzel-112721:edit:
Bug DoR:
[X]* Steps to reproduce, expected, and actual results filled
[X]* Affected version filled
Date of First Response:
Epic Link: Scroll to selected item
Sprint: UI FW 25, UI FW 26
Story Points: 5

 Description   

Steps to reproduce

  1.  Open a M6 UI app (e.g. pages/assets)
  2.  Expand the tree so you have non visible items
  3. Open findbar to search for one of the non-visible items

alternatively:

  1. Open M6 pages (it happens in all content apps using UI 6 Framework (contacts-v8, dam, custom))
  2. Expand the tree so you have non visible items
  3. Edit a page at the bottom of the tree
  4. Close the editor tab to return to the browser view

Expected results

  • the view scrolls to the selected item

Actual results

  • the item is properly selected but the view does not scroll

Workaround

  • n/a

Development notes

  • Scroll for search and deeplink fixed by MGNLUI-6242
  • Selection fixed by MGNLUI-6509
  • Nice to have if in any form related: MGNLUI-6330
  • avoid performance issues
    • manually selecting item doesn't trigger scrolling into view
    • scrolling into view is not called multiple times if already scrolled

Make sure these still works:

  1. https://nightly.magnolia-cms.com/.magnolia/admincentral#app:dam:jcrBrowser;/stories-demo/found-in-switzerland/sylwia-bartyzel-112721:edit: item is visible without scrolling
  2. Switch to list view, back to tree view, item is still visible.


 Comments   
Comment by Roman Kovařík [ 11/Mar/21 ]
  • with a selected item, switching between views scrolls to selected item
  • without a selected item, switching between views doesn't restore scrollbar position (might be a new ticket)
  • we should scroll to selected item when selection is changed programatically
Comment by Roman Kovařík [ 24/Mar/21 ]

Hi,

the ticket is at top of the backlog so most likely next release (6.2.8).

Roman

Comment by Adam Siska [ 12/Apr/21 ]

Known issue (not fixed due to current PR complexity, not ticketized):

  • if user collapse the folder with selected node, folder stays collapsed even after switching views (to list, to tree again) and also after closing detail subapp of edited item

Note:

  • this might be wanted to ticketize (and fix), because similar thing can be seen when using multiselection.. If user clicks to add item to multiselection, collapsed nodes with previously selected items are expanded by the way.
Comment by Roman Kovařík [ 22/Apr/21 ]

Reopened: possible UnsupportedOperationException when scrolling to selected item.

Comment by Annick Boehler [ 09/Jun/21 ]

Hi, 

I was just checking this issue and while executing the first search it worked fine. But if I repeat the search (with another search term) the issue occurs again.

May I ask if this was already addressed (is there already a ticket)? 

Thanks 

Comment by Roman Kovařík [ 10/Jun/21 ]

Hi Annick,

can you reproduce on http://demo.magnolia-cms.com/?

I've collected related tickets under epic MGNLUI-6741. Feel free to add more.

 

Comment by Simon Lutz [ 10/Jun/21 ]

I can't reproduce it on http://demo.magnolia-cms.com/.

Comment by Annick Boehler [ 10/Jun/21 ]

Hi Roman,

no I can not reproduce it on http://demo.magnolia-cms.com/

Will double check with the project team..

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