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

Added showMessageWithColorDuration #2

Merged
merged 2 commits into from
Apr 23, 2014
Merged

Conversation

ludolphus
Copy link
Contributor

Added showMessageWithColorDuration to allow custom background color and
duration of the message

Added showMessageWithColorDuration to allow custom background color and
duration of the message
@mpociot
Copy link
Owner

mpociot commented Apr 23, 2014

Hi,

thanks for the PR! Can you please extend the example app.js and more importantly the README?

@ludolphus
Copy link
Contributor Author

Hi Marcel, sure I will do that. After I did the pull request I thought hmmm I should add an example of the new method. I will probably change something else in the code too. The 'show' method seems to always use a hard-coded white background instead of transparent. Meaning that it hides the status bar (clock and such), only the battery indicator is shown. Using the showSuccessMessage and showErrorMessage and the added showMessageWithColorDuration are showing the contents of the status bar.

@mpociot
Copy link
Owner

mpociot commented Apr 23, 2014

Hi,

that's a good idea. If you want you can also take a look at the original module and make a pull request with your additions there. https://github.com/mwkoelb/MWKProgressIndicator

@ludolphus
Copy link
Contributor Author

good idea, I already used the PR that is pending there to improve the orientation issue. mwkoelb/MWKProgressIndicator#3

Added showMessageWithColor to show a message with a custom color.
Also update the README.md and example app.js showing how to use the new
methods
@ludolphus
Copy link
Contributor Author

New commit with updated README and example app
Also added showMessageWithColor to show a message with a custom color
ludolphus@b3f45ae

mpociot added a commit that referenced this pull request Apr 23, 2014
Added showMessageWithColorDuration
@mpociot mpociot merged commit f01de3b into mpociot:master Apr 23, 2014
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.

2 participants