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

Removed hard reference to StatusBar, changed to type name to prevent … #144

Merged
merged 1 commit into from Apr 27, 2016

Conversation

thebeekeeper
Copy link
Contributor

@thebeekeeper thebeekeeper commented Apr 27, 2016

Description of Change

Modified calls to ApiInformation.IsTypePresent() in WinRT Platform.cs to compile and run when the StatusBar type is not present.

Bugs Fixed

https://bugzilla.xamarin.com/show_bug.cgi?id=40740

API Changes

None

Behavioral Changes

None

PR Checklist

  • Has tests (Omitted -- can't run tests since my machine is missing the required type. Did verify that my application starts up with change)
  • Rebased on top of master at time of PR
  • Changes adhere to coding standard
  • Consolidate commits as makes sense

…crash when StatusBar type is not present

@hartez
Copy link
Contributor

hartez commented Apr 27, 2016

👍

Also, congrats on being our first outside pull request!

@thebeekeeper
Copy link
Contributor Author

Ha! I was wondering if this would be the first. I saw the announcement on Twitter and a light bulb immediately appeared over my head

@acaliaro
Copy link

@hartez I think you will have a lot of work on next weeks 😉

@rmarinho
Copy link
Member

👍

@rmarinho rmarinho merged commit 6b9adc7 into xamarin:master Apr 27, 2016
@ghuntley
Copy link
Contributor

@thebeekeeper
Copy link
Contributor Author

Woooooooo

@danielbnelson
Copy link

XF is about to become freaking amazing!

@rmarinho
Copy link
Member

rmarinho commented May 4, 2016

Hey the @thebeekeeper we forget to ask you to sign the CLA for being able to contribute, can you do it now using this link https://cla2.dotnetfoundation.org

Thanks.

@thebeekeeper
Copy link
Contributor Author

@rmarinho signed it

@samhouts samhouts added this to the 2.3.1 milestone Jun 27, 2018
mattleibow pushed a commit that referenced this pull request Jan 28, 2021
* Change compass to event based.

* Additional cleanup

* Fixup compass on uwp to stop.

* Put in dispose functionality
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

7 participants