Bug 43508 - scrollToBottom needs to scroll on the body not the text node
scrollToBottom needs to scroll on the body not the text node
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Software update
UCS 4.2
Other Linux
: P5 normal (vote)
: UCS 4.2
Assigned To: Jürn Brodersen
Alexander Kläser
: interim-2
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2017-02-07 11:38 CET by Jürn Brodersen
Modified: 2017-04-04 18:30 CEST (History)
2 users (show)

See Also:
What kind of report is it?: Development Internal
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jürn Brodersen univentionstaff 2017-02-07 11:38:39 CET
Because we changed the grid to endless scrolling, the updater needs to scroll on the body and not on the text node for the log view.

Alternately we could give the text node a static height. But I think that would break the overall design.
Comment 1 Alexander Kläser univentionstaff 2017-02-08 11:53:16 CET
Good hint.
Comment 2 Jürn Brodersen univentionstaff 2017-02-23 16:42:52 CET
r77043: fix scroll to bottom in the log view. Added a confirmation dialog for status changes, e.g. the update is finished.
Package: univention-updater
Version: 12.0.3-8A~4.2.0.201702231328
Branch: ucs_4.2-0

I added the confirmation dialog because the log view page now auto scrolls to the bottom, while the update status is shown at the top of the page. Resulting in no clear indicator for the user when the update is finished.
Comment 3 Alexander Kläser univentionstaff 2017-03-29 14:51:22 CEST
I opened Bug 44131 has it snaps even if I am further away from the end (let's say 2 times the viewport height on a longer log view). Otherwise I tested the features in several browsers. Changelog not required.

I added the changelog entry:

changelog-4.2-0.xml:
r78470 | Bug #43508: Added changelog entry


Otherwise, this bug can be set to VERIFIED.
Comment 4 Stefan Gohmann univentionstaff 2017-04-04 18:30:06 CEST
UCS 4.2 has been released:
 https://docs.software-univention.de/release-notes-4.2-0-en.html
 https://docs.software-univention.de/release-notes-4.2-0-de.html

If this error occurs again, please use "Clone This Bug".