Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bootstrap
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 263
    • Issues 263
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 114
    • Merge requests 114
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Bootstrap
  • bootstrap
  • Merge requests
  • !9339

Workaround for #9023.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Administrator requested to merge github/fork/chrisseaton/issue-9023 into 3.0.0-wip Aug 11, 2013
  • Overview 0
  • Commits 4
  • Pipelines 0
  • Changes 6

Created by: chrisseaton

This is an ugly fix for #9023 (closed).

We make the background and the container fill the entire document. We can then scroll as normal within the document, and we aren't using fixed anywhere, as the virtual keyboard breaks that.

To get the modal into the right place, we add padding to the current scroll position.

This will break if the page changes shape (shouldn't happen on iOS, basic rotation is fine). It also sniffs for iOS, which isn't great, but I can't see another way.

So overall this is a pretty nasty fix, and I wouldn't blame you for not merging it, but it may be of interest to someone with the same problem as me. It works fine for my use case.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/chrisseaton/issue-9023