Skip to content

Conversation

spadgett
Copy link
Member

@spadgett spadgett commented Nov 3, 2017

  • Prevent page scrolling under the wizard dialog, making it difficult to scroll the wizard content itself
  • Don't add custom scrollbar styles on iOS, which cause layout problems on an iPhone
  • Reduce wizard max-height on iPhone to prevent the buttons being hidden on iPhone X

Closes #2450

@rhamilto @jwforres

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 3, 2017
@spadgett spadgett force-pushed the mobile-overlay-scroll branch 2 times, most recently from d4be115 to c23d8f7 Compare November 3, 2017 18:32
@openshift-ci-robot openshift-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 3, 2017
@spadgett spadgett force-pushed the mobile-overlay-scroll branch from c23d8f7 to b49544b Compare November 3, 2017 18:34
@spadgett spadgett changed the title [WIP] Prevent page scrolling under wizard dialogs on iOS Wizard fixes for iPhone Nov 3, 2017
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 3, 2017
@spadgett
Copy link
Member Author

spadgett commented Nov 3, 2017

/kind bug

Largely whitespace changes.

@openshift-ci-robot openshift-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Nov 3, 2017
@spadgett
Copy link
Member Author

spadgett commented Nov 3, 2017

/assign @rhamilto @jwforres

@jwforres
Copy link
Member

jwforres commented Nov 3, 2017

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 3, 2017
@spadgett
Copy link
Member Author

spadgett commented Nov 3, 2017

/hold

typo in one rule

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 3, 2017
* Prevent page scrolling under the wizard dialog, making it difficult to scroll the
  wizard content itself
* Don't add custom scrollbar styles on iOS, which cause layout problems on an iPhone
* Reduce wizard max-height on iPhone to prevent the buttons being hidden on iPhone X
@spadgett spadgett force-pushed the mobile-overlay-scroll branch from b49544b to 0b72b1e Compare November 3, 2017 18:40
@openshift-merge-robot openshift-merge-robot removed the lgtm Indicates that a PR is ready to be merged. label Nov 3, 2017
@spadgett
Copy link
Member Author

spadgett commented Nov 3, 2017

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 3, 2017
@jwforres
Copy link
Member

jwforres commented Nov 3, 2017

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 3, 2017
@spadgett
Copy link
Member Author

spadgett commented Nov 3, 2017

/retest

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit 9387547 into openshift:master Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants