Bug 36315 - Add drive wizard - Input fields look wrong
Add drive wizard - Input fields look wrong
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Virtual machines (UVMM)
UCS 4.0
Other Linux
: P5 normal (vote)
: UCS 4.0
Assigned To: Alexander Kramer
Erik Damrose
: interim-3
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-10-27 14:57 CET by Stefan Gohmann
Modified: 2014-11-26 06:54 CET (History)
2 users (show)

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
umc-uvmm-kvm-create.png (83.41 KB, image/png)
2014-10-27 15:00 CET, Stefan Gohmann
Details
umc-uvmm-kvm-create2.png (70.16 KB, image/png)
2014-10-27 15:01 CET, Stefan Gohmann
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Gohmann univentionstaff 2014-10-27 14:57:25 CET
See attached screenshots.

umc-uvmm-kvm-create.png: The dialog window is too high.
umc-uvmm-kvm-create2.png: The input fields are too small.
Comment 1 Stefan Gohmann univentionstaff 2014-10-27 15:00:25 CET
Created attachment 6242 [details]
umc-uvmm-kvm-create.png
Comment 2 Stefan Gohmann univentionstaff 2014-10-27 15:01:47 CET
Created attachment 6243 [details]
umc-uvmm-kvm-create2.png
Comment 3 Alexander Kramer univentionstaff 2014-10-28 16:22:18 CET
r55025:
Adapted the wizard widget so we can overwrite the bootstrap classes inside the DriveWizard

Package: univention-management-console-frontend
Version: 4.1.78-1.921.201410281540
Branch: ucs_4.0-0
Comment 4 Alexander Kramer univentionstaff 2014-10-28 17:11:46 CET
r55036:
Switched parameter for mixin

Package: univention-management-console-frontend
Version: 4.1.79-1.922.201410281657
Branch: ucs_4.0-0
Comment 5 Alexander Kläser univentionstaff 2014-10-30 15:49:18 CET
Fixed.

@Andreas: Could you check whether it works fine on your machine, please?

univention-management-console-frontend-theme (0.0.76-1):
* Bug #36315: adjust button styling within grids
Comment 6 Alexander Kläser univentionstaff 2014-10-30 16:31:05 CET
(In reply to Alexander Kläser from comment #5)
> Fixed.
> 
> @Andreas: Could you check whether it works fine on your machine, please?
> 
> univention-management-console-frontend-theme (0.0.76-1):
> * Bug #36315: adjust button styling within grids

Wrong bug :) .
Comment 7 Andreas Peichert univentionstaff 2014-10-30 19:53:20 CET
r55175:
WIP: fix add drive wizard size
Comment 8 Alexander Kläser univentionstaff 2014-10-30 20:22:18 CET
Some more fixes for the InterfaceWizard, as well. Should be fine now, but I could not test it.

univention-virtual-machine-manager-daemon (4.0.8-1):
* Bug #36315: adjusted InterfaceWizard to new layout, some minor fixes

@AlexKr: Your changes were not committed.
Comment 9 Alexander Kramer univentionstaff 2014-10-31 07:56:26 CET
(In reply to Alexander Kläser from comment #8)
> Some more fixes for the InterfaceWizard, as well. Should be fine now, but I
> could not test it.
> 
> univention-virtual-machine-manager-daemon (4.0.8-1):
> * Bug #36315: adjusted InterfaceWizard to new layout, some minor fixes
> 
> @AlexKr: Your changes were not committed.

r55189:

Sorry seems to be that my svnci throws an error i didn't reconized.
Add committed it again. Just two lines of code to add the
bootstrap classes to the DriveWizzard so the input elements will use
the whole dialog width.
Comment 10 Alexander Kläser univentionstaff 2014-10-31 13:02:52 CET
(In reply to Alexander Kramer from comment #9)
> Sorry seems to be that my svnci throws an error i didn't reconized.
> Add committed it again. Just two lines of code to add the
> bootstrap classes to the DriveWizzard so the input elements will use
> the whole dialog width.

As discussed, the bootstrap classes are not necessary to be set explicitely. We removed headerText + helpText such that the nav container is not displayed. The helpText had been moved into a separate Text widget. (Instead we could have left the helpText set with the additional boot classes, that is true, as well.)
Comment 11 Erik Damrose univentionstaff 2014-11-03 16:37:01 CET
Verified: Drive Wizard works for new instances and editing existing instances
Comment 12 Stefan Gohmann univentionstaff 2014-11-26 06:54:33 CET
UCS 4.0-0 has been released:
 http://docs.univention.de/release-notes-4.0-0-en.html
 http://docs.univention.de/release-notes-4.0-0-de.html

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