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

Homepage - Widget height includes overflow when dragging in edit mode #1216

Closed
xqses opened this issue Jan 21, 2022 · 2 comments · Fixed by #1260
Closed

Homepage - Widget height includes overflow when dragging in edit mode #1216

xqses opened this issue Jan 21, 2022 · 2 comments · Fixed by #1260
Assignees
Labels
team: homepages Issues for the homepages team type: bug 🐛 [2] Velocity rating (Fibonacci)

Comments

@xqses
Copy link

xqses commented Jan 21, 2022

Describe the bug
Dragging a widget with widget height overflow (such as a listview with many items) while editing will include the overflow. This causes the widget(s) below the current widget to be visible as part of the currently dragged widget.

To Reproduce
Steps to reproduce the behavior:

  1. Create an editable homepage and include enough widgets to fill at least two rows.
  2. Add enough content to a widget on the first row so it will overflow.
  3. Enter the edit mode and drag the widget horizontally.
  4. See error

Expected behavior
The widget overflow should not be part of the widget height when the widget is being dragged.

Version

  • ids-enterprise: 4.55.2 | ids-enterprise-ng: 10.6.2 (observed in 10.9 as well)

Screenshots
https://i.imgur.com/XmsjV0z.png
Can be reproduced here.
https://anhallbe.github.io/ids-repro/homepage-editing

@tmcconechy tmcconechy added [2] Velocity rating (Fibonacci) team: homepages Issues for the homepages team type: bug 🐛 labels Jan 21, 2022
@tmcconechy tmcconechy added this to Triage in Enterprise (Next) Sprint Grooming via automation Jan 21, 2022
@tmcconechy
Copy link
Member

Maybe as a fix we need to set overflow off on the draggable element or set the height.

@jbrcna
Copy link

jbrcna commented Mar 25, 2022

This has been QA tested and passed on v4.62.0-dev.
https://main-enterprise.demo.design.infor.com/components/homepage/example-filled-widget
image

@jbrcna jbrcna moved this from Ready for QA (beta) to Done in Enterprise 4.62.x (Mar 2022) Sprint Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team: homepages Issues for the homepages team type: bug 🐛 [2] Velocity rating (Fibonacci)
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants