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

Redesign broken on mobile #46771

Closed
szaimen opened this issue Jul 26, 2024 · 10 comments · Fixed by #46857
Closed

Redesign broken on mobile #46771

szaimen opened this issue Jul 26, 2024 · 10 comments · Fixed by #46857
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 30-feedback accessibility bug design Design, UI, UX, etc. regression ui-refresh-feedback
Milestone

Comments

@szaimen
Copy link
Contributor

szaimen commented Jul 26, 2024

Just tested on daily.ltd2.nextcloud.com:

Screenshot_20240726_101124_Brave
Screenshot_20240726_101143_Brave

FYI @nextcloud/designers

@szaimen szaimen added design Design, UI, UX, etc. 0. Needs triage Pending check for reproducibility or if it fits our roadmap ui-refresh-feedback labels Jul 26, 2024
@szaimen szaimen added this to the Nextcloud 30 milestone Jul 26, 2024
@szaimen szaimen moved this to 👓 Design review in 🖍 Design team Jul 26, 2024
@szaimen
Copy link
Contributor Author

szaimen commented Jul 26, 2024

Another thing: this files filter input box is not easy to discover on mobile as you have to open the sidebar first. (So basically you need to know that it is there):
Screenshot_20240726_101458_Brave

@susnux
Copy link
Contributor

susnux commented Jul 26, 2024

Another thing: this files filter input box is not easy to discover on mobile as you have to open the sidebar first. (So basically you need to know that it is there):

Was discussed during the development but decided that this is the best location.

First screen shot

The border radius of the widgets?

Second screenshot

The copped filter? Known, already fixed but needs a release of nextcloud-vue

@szaimen
Copy link
Contributor Author

szaimen commented Jul 26, 2024

Another thing: this files filter input box is not easy to discover on mobile as you have to open the sidebar first. (So basically you need to know that it is there):

Was discussed during the development but decided that this is the best location.

I disagree. #41609 was the best location. Or we move it to the main content, next to the filter actions.

First screen shot

The border radius of the widgets?

yes and the icons in the header are crowded with scrollbar

Second screenshot

The copped filter? Known, already fixed but needs a release of nextcloud-vue

yes and cool

@susnux
Copy link
Contributor

susnux commented Jul 26, 2024

icons in the header are crowded with scrollbar

Nothing new, I am pretty sure I saw a bug report for this before.
But hard to find on GitHub 😅

@marcoambrosini
Copy link
Member

Or we move it to the main content, next to the filter actions

I think we should do this. This is not a good look @jancborchardt

@jancborchardt
Copy link
Member

@marcoambrosini @szaimen we discussed the location of the search field at length. The result was that the placement on top of the left navigation is a good standard – I communicated as much with you @marcoambrosini. On mobile we expect people to use our mobile apps if it comes to anything except the basics.

@jancborchardt
Copy link
Member

@szaimen could you either open separate issues for the separate things, or make a list of what you identify? As @susnux already noted, it’s not easy to guess with just the screenshots.

@szaimen
Copy link
Contributor Author

szaimen commented Aug 2, 2024

The points:

  1. icons in the header are too crowded and a scollbar is shown
  2. the border-radius is set to 0 for widgets on the dashboard
  3. the Filter filenames input box is very hard to discover on mobile

@susnux
Copy link
Contributor

susnux commented Aug 2, 2024

Splitted the issue as this are 3 independent issues that will need to be tracked independent.

@susnux susnux closed this as not planned Won't fix, can't repro, duplicate, stale Aug 2, 2024
@szaimen
Copy link
Contributor Author

szaimen commented Aug 2, 2024

Thanks Ferdinand! Sorry will do so myself next time :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 30-feedback accessibility bug design Design, UI, UX, etc. regression ui-refresh-feedback
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants