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

#158457042 Bug fixes to the UI of the actual gaming screen #16

Merged
merged 4 commits into from
Jul 17, 2018

Conversation

bekomay26
Copy link
Contributor

What does this PR do?

Fix Redesign issues on the gaming area of the application

Fix Redesign issues

  • Fixed bug with login and signup screen where error is not displayed when signing in or up
  • Fixed the avatar to reduce when on larger screen
  • Made sure all test are passing
  • Changed the answer display color
  • Added lines to the questions.

How should this be manually tested?

Clone the repo. Run ’npm install’ and run ’npm start’. Visit ‘http://localhost:3000/’ and click on ‘Play as a guest’

Any background context you want to provide?

None

What are the relevant pivotal tracker stories?

#158457042

- Fixed bug with login and signup screen where error is not displayed when signing in or up
- Fixed the avatar to reduce when on larger screen
- Made sure all test are passing

[Finishes #158457042]
@adetola-ralph adetola-ralph temporarily deployed to et-cfh-staging-pr-16 July 16, 2018 07:48 Inactive
    - Changed the answer display color
    - Added lines to the questions.

[Finishes #158457042]
@bekomay26 bekomay26 force-pushed the ft-gaming-screen-ui-ux-158457042 branch from 557f509 to 7dfea3f Compare July 16, 2018 09:03
@adetola-ralph adetola-ralph temporarily deployed to et-cfh-staging-pr-16 July 16, 2018 09:03 Inactive
@bekomay26 bekomay26 changed the title Ft gaming screen ui ux 158457042 #158457042 gaming screen ui ux Jul 16, 2018
@bekomay26 bekomay26 changed the title #158457042 gaming screen ui ux #158457042 Bug fixes to the UI of the actual gaming screen Jul 16, 2018
@bekomay26 bekomay26 requested review from Nkemjiks and removed request for Nkemjiks July 16, 2018 11:26
    - Changed the answer display color
    - Added lines to the questions.

[Finishes #158457042]
@adetola-ralph adetola-ralph temporarily deployed to et-cfh-staging-pr-16 July 16, 2018 15:19 Inactive
        - Update stylesheet

    [Finishes #158457042]

$scope.closeReusedModal = function () {
$('#reuse-modal').modal('close');
game.isFilledUp = null;

Choose a reason for hiding this comment

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

Why null and not false?

Copy link
Contributor

Choose a reason for hiding this comment

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

That is the function that Ikechukwu wrote. I just moved it to the invitePlayer controller since the test was not passing.

@adetola-ralph adetola-ralph merged commit 7765ee9 into develop Jul 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants