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

Temp fixes of scrolling-related issues on mobile #1002

Merged
merged 1 commit into from Jan 10, 2019

Conversation

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 3, 2019
@rhamilto
Copy link
Member Author

rhamilto commented Jan 3, 2019

Note: will need to be updated once #989 lands.

.pf-l-page --> .pf-c-page

@spadgett
Copy link
Member

spadgett commented Jan 4, 2019

We should consider tracking things we want to rollback later somehow, maybe with issues.

Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 4, 2019
@spadgett
Copy link
Member

spadgett commented Jan 4, 2019

jenkins rebuild

@rhamilto
Copy link
Member Author

rhamilto commented Jan 4, 2019

/hold for #989

@rhamilto rhamilto added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 4, 2019
@spadgett
Copy link
Member

spadgett commented Jan 7, 2019

jenkins rebuild

@rhamilto rhamilto removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 8, 2019
@rhamilto
Copy link
Member Author

rhamilto commented Jan 8, 2019

#989 needs rebase, so I'm gonna let this go in with the understanding @dtaylor113 will update it in his rebase.

@spadgett
Copy link
Member

spadgett commented Jan 8, 2019

/hold

It looks like #989 is already rebased. Just want to make sure we don't miss the name change.

@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 Jan 8, 2019
@rhamilto rhamilto removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 9, 2019
@openshift-ci-robot openshift-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 9, 2019
@rhamilto
Copy link
Member Author

rhamilto commented Jan 9, 2019

Rebased following the merge of #989

Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 9, 2019
@spadgett
Copy link
Member

spadgett commented Jan 9, 2019

jenkins rebuild

1 similar comment
@spadgett
Copy link
Member

jenkins rebuild

@openshift-merge-robot openshift-merge-robot merged commit 19c480e into openshift:master Jan 10, 2019
@rhamilto rhamilto deleted the temp-nav-fixes branch January 10, 2019 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants