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

Implemented Cancel Button On Dialog (Blood Request Fragment). #96

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

animeshroydev
Copy link
Contributor

Included cancel button in blood request fragment

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Initially there was not back button in Blood Request Fragment Activity. I have added a rounded corner CANCEL button.

  • What is the current behavior? (You can also link to an open issue here)
    When click on the CANCEL button it will get back to the HomeActiviity.

  • Other information:
    I have added shapeone.xml in drawable folder for making the button rounded.

Screenshot (two screenshot (gif format): first runs In Android 5.1 and another in 4.4.2 Android OS Version)

screenshot_2018-05-06-01-20-22-animation

Copy link
Collaborator

@groverankush groverankush left a comment

Choose a reason for hiding this comment

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

Is the cancel button required? In the latest commit to develop the dialog has been made cancellable. I suggest that adding cancel button is just taking the screen space. The dialog is not of Alert type, presumably there's no need to add cancel button for this.

If it has to be added. I'll suggest a simple cross button on the top.

Your gitignore file is not properly configured. I request you to take pull of develop branch and get it rectified.

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

2 participants