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

[Feature Request]: Force list view on mobile devices #5012

Open
3 tasks done
jasonhenriquez opened this issue Apr 24, 2024 · 2 comments
Open
3 tasks done

[Feature Request]: Force list view on mobile devices #5012

jasonhenriquez opened this issue Apr 24, 2024 · 2 comments

Comments

@jasonhenriquez
Copy link
Collaborator

jasonhenriquez commented Apr 24, 2024

Guidelines

  • I have searched the issue tracker for open and closed issues that are similar to the feature request I want to file, without success.
  • I have searched the documentation for information that matches the description of the feature request I want to file, without success.
  • This issue contains only one feature request.

Problem Description

Mobile devices, below a certain width threshold, aren't actually ever getting a Video View Type of grid. Rather, we have styling rules that make the grid display attempt to emulate the list view. This is more code to maintain, and an additional scenario to test for each new feature.

Proposed Solution

Force the list view for mobile devices on all routes. This allows us to reduce the amount of code we have to maintain and test. See the resize event listener in ft-list-video.js to see how we check for this condition. If we have any unrelated grievances with the mobile List view that we have been putting off because not enough of us actually use the list view, we should also fit in some changes there as well if possible (most expediently by stealing certain rules from the grid mobile view, or more ambitiously by pursuing a broader rework of the ft-list-video mobile appearance).

This may be partly dependent on #5007 if we think that the visual real estate afforded by that change for mobile users will allow us to have more flexibility.

Alternatives Considered

None come to mind. Maybe switching to grid as the mobile default on specific views where it looks better, but that seems insufficient & somewhat complicated to keep track of.

Issue Labels

improvement to existing feature, visual improvement

Additional Information

No response

@absidue
Copy link
Member

absidue commented Apr 24, 2024

Currently the single column gird view is the best experience on mobile, as the list view is designed for wide screens and displays things horizontally next to each other, but the grid view displays things vertically, so it works a lot better.

@jasonhenriquez
Copy link
Collaborator Author

Thanks for that clarification @absidue. We can probably just get rid of the list mobile styling and supplant it with the grid's as the starting point (probably making it sound easier than it is with all the inherited styles and minor structural differences, but ya know).

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

No branches or pull requests

2 participants