Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Long text in Asset's "Note (Protected)" field breaks view asset page #15501

Open
2 tasks done
scorchmeister13 opened this issue Sep 13, 2024 · 4 comments
Open
2 tasks done

Comments

@scorchmeister13
Copy link

Debug mode

Describe the bug

If assets have long text inserted into the "Notes (Protected)" field, the minified text does not wrap properly and ends up hiding the Asset update buttons on the right side (Attached an image). In the cases where the text is even longer the buttons are hidden completely.

Screenshot from 2024-09-13 14-09-19

The issue looks to be similar to #13028 but for Protected fields.

Reproduction steps

  1. Edit asset

  2. Add the following text to "Notes (Protected)":
    ABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZ

  3. Check view asset page

  4. Make the window smaller hotizontally if the options on right side are not hidden automatically
    ...

Expected behavior

Line that doesnt fit to screen needs have individual horizontal scroll bar

Screenshots

No response

Snipe-IT Version

v7.0.11 and v7.0.12

Operating System

Debian

Web Server

Nginx

PHP Version

Latest

Operating System

No response

Browser

No response

Version

No response

Device

No response

Operating System

No response

Browser

No response

Version

No response

Error messages

No response

Additional context

No response

Copy link

welcome bot commented Sep 13, 2024

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

@scorchmeister13
Copy link
Author

I was able to resolve the issue by updating the attached file on path /var/www/html/resources/views/layouts/default.blade.php.

If possible can anyone guide me or help on the merge for it to be included in next release?

default.blade.php.zip

@snipe
Copy link
Owner

snipe commented Sep 16, 2024

What's the use case for having such a long string with no line breaks?

@scorchmeister13
Copy link
Author

The issue can be reproduced if line breaks are present as well. 4+ lines will do the same. I added the long string as an example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants