Skip to content

Conversation

@rhamilto
Copy link
Member

Changes in #1053 resulted in the bug.

Before:

screen shot 2019-01-28 at 11 27 39 am
screen shot 2019-01-28 at 11 27 13 am

After:

screen shot 2019-01-28 at 11 24 39 am
screen shot 2019-01-28 at 11 24 33 am

fyi @sg00dwin

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 28, 2019
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 28, 2019
@serenamarie125
Copy link
Contributor

@rhamilto does the text field still have initial focus?

@spadgett
Copy link
Member

@serenamarie125 Looking at the diff, the change should have no affect on autofocus. It's still set to true.

@serenamarie125
Copy link
Contributor

Ok ! Just asking because of what I’m seeing in the screenshots. Thanks for verifying @spadgett

@openshift-merge-robot openshift-merge-robot merged commit 3c3e548 into openshift:master Jan 28, 2019
@rhamilto rhamilto deleted the delete-modal branch January 28, 2019 19:27
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/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants