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

Web: Improves responsive design #775

Closed
wants to merge 5 commits into from

Conversation

Sesquipedalian
Copy link

@Sesquipedalian Sesquipedalian commented Dec 4, 2018

Changes:

  1. Loads mobile.css based on max-width instead of max-device-width.
    The device-width CSS media feature is deprecated, and using it can cause unexpected behaviour if the viewport width is 480px or less but the device's total screen width is greater than 480px.

  2. Prevents the torrent inspector from overflowing the viewport.

  3. Positions the footer at the bottom of the viewport on narrow devices, just like on larger ones.

  4. Prevents the toolbar from overflowing the viewport on narrow devices.

The device-width CSS media feature is deprecated, and using it can cause unexpected behaviour if the viewport width is 480px or less but the device's total screen width is greater than 480px.
bennap
bennap previously approved these changes Feb 4, 2019
@ckerr
Copy link
Member

ckerr commented Oct 12, 2021

I know how long this PR has been in limbo and am trying to go through the project's backlog of older PRs.

@Sesquipedalian is this PR still germane after the web client refresh that landed in #1476? I believe many of the items in the description got addressed, e.g. the footer and mobile use, but I don't know if there are items that are still not addressed in master.

@ckerr ckerr added scope:web needs clarification More info is needed before work can proceed labels Oct 12, 2021
@Sesquipedalian
Copy link
Author

No, this PR is far too stale to be relevant anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs clarification More info is needed before work can proceed scope:web
Development

Successfully merging this pull request may close these issues.

None yet

3 participants