Bug 32168 - Tooltips are not shown
Tooltips are not shown
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Domain management (Generic)
UCS 3.2
Other Linux
: P5 normal (vote)
: UCS 3.2
Assigned To: Dirk Wiesenthal
Florian Best
: interim-3
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-08-06 16:41 CEST by Alexander Kläser
Modified: 2013-11-19 06:44 CET (History)
0 users

See Also:
What kind of report is it?: ---
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 Alexander Kläser univentionstaff 2013-08-06 16:41:23 CEST
It seems that the tooltips are not shown on the Detailpage of UDM objects.
Maybe due to the Dojo update (Bug 31639)?
Comment 1 Dirk Wiesenthal univentionstaff 2013-10-10 13:08:00 CEST
Problem was in app.js:

// happens globally. *Every* click will trigger this
on(baseWin.body(), 'click', lang.hitch(this, function() {
  this._closeModuleContextMenu();
}

// closes global Tooltip
and in _closeModuleContextMenu():
  Tooltip._masterTT && Tooltip._masterTT.fadeOut.play();

So, effectively, every click anywhere in UMC closed any Tooltip currently shown. Also applies to Tooltips currently popping up because of invalid widgets.

I doubt that
  Tooltip._masterTT && Tooltip._masterTT.fadeOut.play();
is useful. I have not seen any glitches that required any tooltip to be closed immediately.

Anyway, I rather added a check in the onclick event if the selected module is the overview page.

  univention-management-console-frontend 3.0.97-1.745.201310101301

No changelog needed (dev bug).
Comment 2 Florian Best univentionstaff 2013-10-21 14:15:05 CEST
If you switch to another tab while a context menu is opened you are not able to close it anymore.
Comment 3 Dirk Wiesenthal univentionstaff 2013-10-22 13:12:56 CEST
True, fixed in
  univention-management-console-frontend 3.0.116-2.763.201310221307

Removed that error prone handle completely. Extended the GalleryPane with code taken from my personal backlog. Downside: Quite some changes in app.js

Please take special care when testing touch devices as I lack one and just copied the code blindly.
Comment 4 Alexander Kläser univentionstaff 2013-10-24 12:35:44 CEST
Probably due to adaptations of this bug, the long touch on overview items to open the context menu does not work anymore.
Comment 5 Dirk Wiesenthal univentionstaff 2013-10-28 14:18:06 CET
(In reply to Alexander Kläser from comment #4)
> Probably due to adaptations of this bug, the long touch on overview items to
> open the context menu does not work anymore.

Yes. Error while copying touch event handler from app.js to GalleryPane.js
  univention-management-console-frontend 3.0.132-1.777.201310281408
Comment 6 Florian Best univentionstaff 2013-10-29 16:25:32 CET
OK, tested also with iPad.
Comment 7 Stefan Gohmann univentionstaff 2013-11-19 06:44:27 CET
UCS 3.2 has been released:
 http://docs.univention.de/release-notes-3.2-en.html
 http://docs.univention.de/release-notes-3.2-de.html

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