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

File name is omitted if it is a smartphone with NC17 #17364

Closed
Bables55 opened this issue Oct 2, 2019 · 20 comments
Closed

File name is omitted if it is a smartphone with NC17 #17364

Bables55 opened this issue Oct 2, 2019 · 20 comments
Labels
1. to develop Accepted and waiting to be taken care of bug design Design, UI, UX, etc.

Comments

@Bables55
Copy link

Bables55 commented Oct 2, 2019

After updating to Nextcloud 17.0.0, I accessed the NC with an iPhone web browser.
When I hold the iPhone vertically and look at the file list, the file name is omitted and the icon, file size, menu, and update date and time are displayed. The share icon is covered by the file size. Some browsers display some characters of the file extension by the Web browser. Of course I usually see it if I hold it horizontally.
If I use the iOS app of NC, it will display normally even if I have it vertically.

I think the display when the user holds the smartphone vertically can omit the file size and update date.

@Bables55 Bables55 added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Oct 2, 2019
@kesselb
Copy link
Contributor

kesselb commented Oct 2, 2019

Is it possible to add a screenshot to your report?

@Bables55
Copy link
Author

Bables55 commented Oct 3, 2019

Is it possible to add a screenshot to your report?

smartphonesize
*I'm Japanese and the NC language is set to English, but the time display is Japanese.

@Octopus2
Copy link

Octopus2 commented Oct 3, 2019

Screenshot_2019-10-03_16-30-28

Same with Nextcloud 17, Android 8 on an Samsung Galaxy S7, Newest Chrome

@AndyFx12
Copy link

AndyFx12 commented Oct 3, 2019

same here - Nextcloud 17 with Android 7.1.1 on an MotoG4-Play. Chrome and Firefox.

@DmDS
Copy link

DmDS commented Oct 4, 2019

same thing....

@kesselb
Copy link
Contributor

kesselb commented Oct 4, 2019

cc @nextcloud/designers

@elpraga
Copy link

elpraga commented Oct 6, 2019

Also present on Android devices.

It makes it impossible to browse folders and files when the phone is in vertical position.

@justasomeone
Copy link

justasomeone commented Oct 6, 2019

it works in landscape mode, on my pcs and in grid mode. but both my phones (IphoneSE and Samsung S8) have this issue in portrait mode, which is the mode I use often. :(

It was working fine on 16.0.5. Updating to 17 broke it.

5WM7kgf

@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of 17-feedback design Design, UI, UX, etc. and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Oct 7, 2019
@skjnldsv skjnldsv added this to the Nextcloud 17.0.1 milestone Oct 7, 2019
@GretaD
Copy link
Contributor

GretaD commented Oct 8, 2019

this was suppose to be fixed with this PR: #17162
It looks good on the 18th version, but it still exist on the 17th one

@GretaD
Copy link
Contributor

GretaD commented Oct 8, 2019

After consulting this with @jancborchardt looks like the problem is that the Size and Modified column is not suppose to be there at the first place on mobile view. This might be a regression of some other change. I am working to fix this.

GretaD added a commit that referenced this issue Oct 8, 2019
Signed-off-by: Greta Doci <gretadoci@gmail.com>
@weeman1337 weeman1337 unpinned this issue Oct 8, 2019
@kesselb kesselb mentioned this issue Oct 15, 2019
backportbot-nextcloud bot pushed a commit that referenced this issue Oct 25, 2019
Signed-off-by: Greta Doci <gretadoci@gmail.com>
@ghost

This comment has been minimized.

@kesselb kesselb pinned this issue Oct 27, 2019
@kesselb
Copy link
Contributor

kesselb commented Oct 27, 2019

Pinned this issue a few days ago because I had in mind that a lot of people are going to report this. I don't know why someone unpinned it but given that we already had 4 more reports since then I'm pinning it again ;)

Nextcloud 17.0.1 (scheduled for 2019-11-07) will contain the fix: #17676

@skjnldsv skjnldsv unpinned this issue Oct 27, 2019
@skjnldsv skjnldsv pinned this issue Oct 27, 2019
@skjnldsv

This comment has been minimized.

@jancborchardt

This comment has been minimized.

@kesselb

This comment has been minimized.

@ghost
Copy link

ghost commented Oct 31, 2019

This is now fixed on 17 RC1! 😄

@rollanders
Copy link

It is still present in NC17.0.1 RC1 on Chrome in Android mobile. Firefox for Android scales OK.
Another issue related to mobile screen is that in the contacts list I can only swipe sideways on the top of the screen, above the contacts names.

@ghost
Copy link

ghost commented Oct 31, 2019

Sorry. I only checked with Opera Mobile (which is based on Chrome). No scaling issue there either.

Screenshot_20191031-185829_Opera

@rollanders
Copy link

Maybe a problem with caching. Now it works OK!

@jancborchardt
Copy link
Member

Thanks a lot for testing and confirming the fix everyone! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug design Design, UI, UX, etc.
Projects
None yet
Development

No branches or pull requests