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

CSS #oc-dialog-filepicker-content: wrong display of date #5968

Closed
dartcafe opened this issue Aug 3, 2017 · 6 comments
Closed

CSS #oc-dialog-filepicker-content: wrong display of date #5968

dartcafe opened this issue Aug 3, 2017 · 6 comments

Comments

@dartcafe
Copy link
Contributor

dartcafe commented Aug 3, 2017

The date information is positioned not correctly. Same on 12.0.0 and 12.0.1 RC4

grafik

Adding following CSS fixes the display:
#oc-dialog-filepicker-content .filelist .date { display: table-cell; position: initial; }

grafik

I am no css pro, so maybe there will be another or better fix for this.

Problem was visible in the mail add, when trying to svae an attachment to files.

@dartcafe dartcafe changed the title CSS #oc-dialog-filepicker-content: wrong diplay of date CSS #oc-dialog-filepicker-content: wrong display of date Aug 3, 2017
@MorrisJobke
Copy link
Member

@dartcafe Which browser do you use? I can't reproduce this on master. I tested Safari, Chrome and Firefox.

@MorrisJobke
Copy link
Member

I even tested Edge and it works fine there.

@juliusknorr
Copy link
Member

As stated in the PR:

That issue seems to be related only to the mail app, so i guess we should try to fix it over there. All other file choosers work as expected.

@dartcafe
Copy link
Contributor Author

dartcafe commented Aug 4, 2017

I looked for a file picker in another app, but I didn't find. :-) Any hint?

Issue in mail app is reproducible in Safari (iPad) Firefox (iPad and Windows), Chrome (Windows) and Edge.

@dartcafe
Copy link
Contributor Author

dartcafe commented Aug 4, 2017

Ha. As I write it down, I found it in the files app, when rying to move a file. I will make a new PR in the mail app repository. Thanks for review.

@MorrisJobke
Copy link
Member

@dartcafe To trigger it type this in the console in your webdev tools in the browser:

OC.dialogs.filepicker('A')

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

No branches or pull requests

3 participants