znuny / Znuny

Znuny/Znuny LTS is a fork of the ((OTRS)) Community Edition, one of the most flexible web-based ticketing systems used for Customer Service, Help Desk, IT Service Management.
https://www.znuny.org
GNU General Public License v3.0
355 stars 85 forks source link

Sector Nord AG: Fixed popup description long were cut off in Customer Interface. #331

Closed jsinagowitz closed 1 year ago

jsinagowitz commented 1 year ago

Proposed change

In the customer portal the hover popup "description long" will be cut off if it is set for the first field.

image

Type of change

Additional information

I identified the CSS setting in var/httpd/htdocs/skins/Customer/default/css/Core.Form.css:

fieldset h2 + div {
    padding-top: 13px;
    border-top: 1px solid #FFF;
    overflow: hidden;
}

I think the overflow: hidden doesn't make sense here, because it was only used for the first field in the field list so far.

Checklist

dennykorsukewitz commented 1 year ago

Hi @jsinagowitz , thanks for your report.

It only happens in the customer portal. It only happens for the first field.

I cannot reproduce this error. Can you please export a process that contains this error?

Process_ticket_-_Ticket_-_znuny_dev

@NiklasSchmitt Can you confirm this or can you reproduce the error?

Please, keep your fork up-to-date with the upstream repository. Syncing a fork 🚀 🆙

Regards :rocket: Denny

NiklasSchmitt commented 1 year ago

I can not reproduce this reported behavior. I've added a very very long Lorem ipsum-string into "description long" for the first field and it's all to see into customer portal and also on agent portal. grafik

jsinagowitz commented 1 year ago

I did some more research. I think you can't reproduce that because of the description in the dialog. You might need to remove that.

image

The "wrong" css setting is only active, if the div-tag of the first field follows the h2-tag directly.

image

$('fieldset h2 + div')

If you can't reproduce that eather I will send you my process. I can reproduce the bug on multiple systems.

NiklasSchmitt commented 1 year ago

Hi @jsinagowitz,

thank you for your additional information. I was able to reproduce the issue and I want to accept & merge your pull-request. Bur first: Please, keep your fork up-to-date with the upstream repository. See here how to sync a fork.

Best regards, Niklas Schmitt :coffee: :rocket: