Bug 37742

Summary: Edit window is too small
Product: UCS Reporter: Stefan Gohmann <gohmann>
Component: UMC - Univention Configuration RegistryAssignee: Florian Best <best>
Status: CLOSED FIXED QA Contact: Alexander Kläser <klaeser>
Severity: normal    
Priority: P5 CC: best, jmm, klaeser
Version: UCS 4.0   
Target Milestone: UCS 4.0-1-errata   
Hardware: Other   
OS: Linux   
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: ucr.png

Description Stefan Gohmann univentionstaff 2015-02-10 07:13:17 CET
Created attachment 6659 [details]
ucr.png

See attached screenshot. The edit window is too small.
Comment 1 Florian Best univentionstaff 2015-02-25 16:11:18 CET
There was a max-width: 350px set to the dialog. This was changed to min-width: 450px (inherited max-width of 650px).
The variable with the longest description (german) is "mail/antispam/requiredhits" → 775 characters.

"ldap/index/eq" has the longest value which of course does not fit into one line but IMHO this is fine.

The longest UCR variable name is "zarafa/default/ZARAFA_USERSCRIPT_LOCALE".

They all look fine now. I think it is not necessary to make the font smaller in the input field.

Fix: svn r58407
YAML: 2015-02-25-univention-management-console-module-top.yaml
Comment 2 Alexander Kläser univentionstaff 2015-02-27 11:46:18 CET
This fix does not work on small screens, as it fixes the dialog with to 450px. The class umcLargeDialog addresses a similar problem, yet assumes a size of 650px:

.umc .dijitDialog.umcLargeDialog {
  width: 100%;
}
@media (min-width: 651px) {
  .umc .dijitDialog.umcLargeDialog {
    width: 650px;
  }
}
@media (min-width: 812.5px) {
  .umc .dijitDialog.umcLargeDialog {
    width: 80%;
  }
}

Similar CSS rules should be defined for this case here.
Comment 3 Florian Best univentionstaff 2015-02-27 11:56:32 CET
OK, rules adapted.
Comment 4 Alexander Kläser univentionstaff 2015-03-03 11:33:23 CET
Changes: OK, looks good!
YAML file: OK, I adjusted the entry a bit [r58599]
Comment 5 Moritz Muehlenhoff univentionstaff 2015-03-11 15:08:30 CET
http://errata.univention.de/ucs/4.0/101.html