-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
File modification date shine through on scroll #37871
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
Conversation
szaimen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fear this will reintroduce #33872
|
It's not the case as far as i see Screen.Recording.2023-04-21.at.18.30.57.mov |
szaimen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See
2023-04-21-201810.mp4
|
It should be good now, It looks good on my end. Can you please give it a test |
Screen.Recording.2023-04-23.at.17.29.29.mov |
szaimen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alternative, less intrusive approach in #37895
53b62ec to
0cfdce5
Compare
|
This is the only work around I found without any side effects on other components. I'm open for other suggestions 🤨 |
szaimen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| <p></p> | ||
| </div> | ||
| <table class="files-filestable list-container <?php p($_['showgridview'] ? 'view-grid' : '') ?>" data-allow-public-upload="<?php p($_['publicUploadEnabled'])?>" data-preview-x="250" data-preview-y="250"> | ||
| <table class="files-filestable list-container <?php p($_['showgridview'] ? 'view-grid' : '') ?>" data-allow-public-upload="<?php p($_['publicUploadEnabled']) ?>" data-preview-x="250" data-preview-y="250"> |
Check notice
Code scanning / Psalm
PossiblyUndefinedArrayOffset
szaimen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works now
Signed-off-by: hamza221 <[email protected]>
3cfab16 to
2d694ea
Compare
|
Squashed and rebased |
|
So is anything happening here? Just updated 2 major versions to current 27.1.1 and it is still broken? |

Summary
Steps to reproduce:
Before :

After :
## TODOChecklist