Bug 48966 - badge tooltips don't scroll
badge tooltips don't scroll
Status: NEW
Product: UCS
Classification: Unclassified
Component: UMC - App-Center
UCS 4.4
Other Linux
: P5 normal (vote)
: ---
Assigned To: App Center maintainers
App Center maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2019-03-12 11:42 CET by Jannik Ahlers
Modified: 2022-02-03 15:01 CET (History)
3 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 1: Cosmetic issue or missing function but workaround exists
Who will be affected by this bug?: 5: Will affect all installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.029
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): bitesize
Max CVSS v3 score:
best: Patch_Available+


Attachments
badge tooltips (164.52 KB, image/png)
2019-03-12 11:42 CET, Jannik Ahlers
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jannik Ahlers univentionstaff 2019-03-12 11:42:17 CET
Created attachment 9919 [details]
badge tooltips

the new scrolling behaviour in the app center leads to the badge tooltips stay in place, while the background is scrolling.
Comment 1 Lukas Zumvorde univentionstaff 2021-10-12 15:13:18 CEST
This was also filed as a bug by Orange. We could reproduce it in several UMC modules.

Steps for reproduction:
- Login to UMC with any user (we used Administrator)
- open a UMC Module (we used User)
- Search for a little questionmark symbol and click on it to open the tooltip (er used the one of "override password check")
- scroll
- you will see that the tooltip does not move with the symbol it belongs to.
Comment 2 Thomas Bach univentionstaff 2022-02-03 14:26:47 CET
We have a working patch!

Note that I have not developed the patch myself, but Toni Mägel. I replicated the issue, tested the patch and verified that it does indeed solve the issue. 

After applying the patch the tooltip is hidden after a certain timeout as well as on a scroll event.

Here is the patch:

diff --git a/management/univention-web/js/widgets/LabelPane.js b/management/univention-web/js/widgets/LabelPane.js
index 685a5ccc57..b42036486c 100644
--- a/management/univention-web/js/widgets/LabelPane.js
+++ b/management/univention-web/js/widgets/LabelPane.js
@@ -38,12 +38,13 @@ define([
        "dojo/dom-construct",
        "dojo/dom-attr",
        "dojo/on",
+       "dojo/query",
        "dijit/_WidgetBase",
        "dijit/_TemplatedMixin",
        "dijit/_Container",
        "dijit/Tooltip",
        "umc/tools"
-], function(declare, lang, array, event, Deferred, domClass, domConstruct, attr, on, _WidgetBase, _TemplatedMixin, _Container, Tooltip, tools) {
+], function(declare, lang, array, event, Deferred, domClass, domConstruct, attr, on, query, _WidgetBase, _TemplatedMixin, _Container, Tooltip, tools) {
        lang.extend(_WidgetBase, {
                // displayLabel: Boolean?
                //              If specified as false, LabelPane will not display the label value.
@@ -284,6 +285,13 @@ define([
                                                        on.once(window, "click", lang.hitch(this, function(event) {
                                                                Tooltip.hide(this.tooltipNode);
                                                        }));
+                                                       on(query('.umcPageMain'), "scroll",
+                                                               lang.hitch(this, function(event) {
+                                                                       Tooltip.hide(this.tooltipNode);
+                                                               }));
+                                                       setTimeout(lang.hitch(this, function(event) {
+                                                               Tooltip.hide(this.tooltipNode);
+                                                       }), 3000);
                                                })));
                                        }
                                } else {