Univention Bugzilla – Bug 30796
Automatic scrollbars in dijit/Dialog
Last modified: 2013-06-13 14:36:53 CEST
Currently, the handling of scrollbars in the Dialog is not always correct. Nice would be if scrollbars were shown automatically for content that exceeds the dialog container.
In the App Center, it should be already working correctly, however, for the list of installed packages, the scrollbars are not shown.
This bug has been fixed. - ucs3.1-2 & changelog: Package: univention-management-console-module-appcenter Version: 2.0.143-1.121.201304151716 Package: univention-management-console-frontend Version: 2.0.254-1.631.201304151720 -ucs3.1-1 & YAML: Package: univention-management-console-module-appcenter Version: 2.0.139-3.122.201304151732 Package: univention-management-console-frontend Version: 2.0.244-9.632.201304151736
Package has been updated to make sure that the new appcenter is not installed without the new frontend. -ucs3.1-1 Package: univention-management-console-module-appcenter Version: 2.0.139-4.123.201304161021
(In reply to comment #3) > Package has been updated to make sure that the new appcenter is not installed > without the new frontend. > > -ucs3.1-1 > Package: univention-management-console-module-appcenter > Version: 2.0.139-4.123.201304161021 After discussing it, a better approach would be to remove the dependency on a particular version from the debian control file. In the App Center, the JavaScript code can evaluate tools.status("version") and if necessary set the width or not. In this way, the changes are more flexible.
(In reply to comment #4) > After discussing it, a better approach would be to remove the dependency on a > particular version from the debian control file. In the App Center, the > JavaScript code can evaluate tools.status("version") and if necessary set the > width or not. In this way, the changes are more flexible. This has been fixed. - ucs3.1-1: Package: univention-management-console-module-appcenter Version: 2.0.139-5.124.201304161436
(In reply to comment #5) > (In reply to comment #4) > > After discussing it, a better approach would be to remove the dependency on a > > particular version from the debian control file. In the App Center, the > > JavaScript code can evaluate tools.status("version") and if necessary set the > > width or not. In this way, the changes are more flexible. > > This has been fixed. > > - ucs3.1-1: > Package: univention-management-console-module-appcenter > Version: 2.0.139-5.124.201304161436 Rather than a simple string comparison I have added a function called "compareVersion". The appcenter packages have been updated. Now both versions are identical. - ucs3.1-2: Package: univention-management-console-module-appcenter Version: 2.0.144-1.126.201304161716 - ucs3.1-1: Package: univention-management-console-module-appcenter Version: 2.0.139-6.125.201304161708
3.1-2: Fail: Comparing versions is not necessary here as the frontend is already patched when 3.1.2 is released.
(In reply to comment #7) > 3.1-2: Fail: Comparing versions is not necessary here as the frontend is > already patched when 3.1.2 is released. This has been fixed in univention-management-console-module-appcenter (2.0.152-1)
The App Center part is VERIFIED meaning: This bugfix does not break anything. As for the frontend part itself: Does this mean that from now on every dijit/Dialog should be able to handle very long words and show a scrollbar automatically? I cannot confirm that this works.
I do not know whether the bug is fixed as I do not which bug we are talking about. But as far as I can see nothing is broken. CSS was adapted to use max-width so this may be some kind of improvement. So I may call it VERIFIED. As always, if this bug occurs again, feel free to clone this bug. But this time we mean it!
Created attachment 5235 [details] AutoScrollbar does not work In 3.1-1 the many HALLO?s respect the max-width (good!) but there is no auto scrollbar although there is a big HALLO?HALLO?... (not good!?) So is this bug really fixed? Was this the scope of this bug?
http://errata.univention.de/ucs/3.1/102.html
Created attachment 5238 [details] Sample dialog with scrollbars The behaviour can be tested with the following JavaScript-Code: > var s = ''; for (var i = 0; i < 2000; ++i) s += i + ' '; require('umc/dialog').alert(s) On an updated system, the scrollbars work fine. Note, the "Hallo?Hallo?Hallo?..." line does not contain white spaces, horizontal scrollbars are not shown automatically, this bug was supposed to handle vertical scrollbars which seem to work as expected (see screenshot).
The bug has not been published yet as erratum AFAIS → VERIFIED
http://errata.univention.de/ucs/3.1/123.html