Bug 44431
| Summary: | UMC DNS: Zone entry in grid looks broken after adding a TXT record or forward zone | ||
|---|---|---|---|
| Product: | UCS | Reporter: | Michael Grandjean <grandjean> |
| Component: | UMC - DNS | Assignee: | Jürn Brodersen <brodersen> |
| Status: | CLOSED FIXED | QA Contact: | Florian Best <best> |
| Severity: | normal | ||
| Priority: | P5 | CC: | best, brodersen, damrose, klaeser, oliver.denk |
| Version: | UCS 4.2 | ||
| Target Milestone: | UCS 4.2-1-errata | ||
| Hardware: | Other | ||
| OS: | Linux | ||
| See Also: |
https://forge.univention.org/bugzilla/show_bug.cgi?id=44194 https://forge.univention.org/bugzilla/show_bug.cgi?id=44651 |
||
| 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?: | 3: Will affect average number of installed domains | How will those affected feel about the bug?: | 2: A Pain – users won’t like this once they notice it |
| User Pain: | 0.034 | Enterprise Customer affected?: | |
| School Customer affected?: | Yes | ISV affected?: | |
| Waiting Support: | Flags outvoted (downgraded) after PO Review: | ||
| Ticket number: | Bug group (optional): | Design, External feedback, Usability | |
| Max CVSS v3 score: | |||
| Bug Depends on: | |||
| Bug Blocks: | 44194 | ||
| Attachments: |
Screenshot of DNS module
Screenshot of DNS module with reversed sorting |
||
|
Description
Michael Grandjean
Created attachment 8793 [details]
Screenshot of DNS module with reversed sorting
If I reverse the sorting, it gets even worse (see screenshot).
I tested with Firefox 53.0 and Google Chrome 58.0
*** Bug 44651 has been marked as a duplicate of this bug. *** *** Bug 44746 has been marked as a duplicate of this bug. *** I didn't expected two dimensional arrays as values. Rows with varying height don't look very good. We should try and limit all data to one line. Maybe we can show additional information as a popup instead? (In reply to Jürn Brodersen from comment #4) > I didn't expected two dimensional arrays as values. Rows with varying height > don't look very good. We should try and limit all data to one line. > > Maybe we can show additional information as a popup instead? I don't think so. Multi rows should also be supported. What if third party UMC modules have them? This shouldn't be fixed with a workaround. Also the information have been added on purpose to improve the whole usability of the DNS/DHCP modules. As discussed, please put these values into a <div> or <span> container with the following CSS properties: line-height: 1.25; display: block; text-overflow: ellipsis; You need to modify height to "auto" and min-height to "48px" in .umc .umcGrid .dgrid .dgrid-row. I was able to use the already existing cell div from the grid. That way only the css needed some changes. r81326/81327: yaml r81325: fix multiline entries in grid Package: univention-web Version: 1.0.42-24A~4.2.0.201707241035 Branch: ucs_4.2-0 Scope: errata4.2-1 Forgot to test IE... The ellipsis don't work in IE, sorry (In reply to Jürn Brodersen from comment #9) > Forgot to test IE... The ellipsis don't work in IE, sorry This was broken in firefox, too. I fixed it by adding a height to ".dgrid-row-table" that removed the need for line-height because now vertical-allign works (the tables height was set indirectly through line-height). Note1: height for tables is interpreted as min-height (https://www.w3.org/TR/CSS21/tables.html#height-layout) Note2: In IE no ellipses are shown after the first <br> (https://stackoverflow.com/questions/31022769/text-overflowellipsis-does-not-work-with-br-in-ie) r81339: fix ellipses in grid Package: univention-web Version: 1.0.42-25A~4.2.0.201707241347 Branch: ucs_4.2-0 Scope: errata4.2-1 Okay, this looks very nice now! I also had a look at UVMM and various UCS@school modules. OK: YAML (adjusted in r81371) |