Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Multi classes #134

Merged
merged 3 commits into from
Apr 25, 2016
Merged

Multi classes #134

merged 3 commits into from
Apr 25, 2016

Conversation

StephaneDelcroix
Copy link
Member

@StephaneDelcroix StephaneDelcroix commented Apr 25, 2016

Description of Change

VisualElement supports a list of StyleClass, so you can have

Bugs Fixed

I do not fix bugs, I create them

API Changes

Changed:

  • Visualelement.ClassStyle is now an List

Behavioral Changes

/

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Rebased on top of master at time of PR
  • Changes adhere to coding standard
  • Consolidate commits as makes sense

ClassStyle?.Apply(bindable);
if (ClassStyles != null)
foreach (var classStyle in ClassStyles)
((IStyle)classStyle)?.Apply(bindable);
Copy link
Member

Choose a reason for hiding this comment

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

isn't it better (classStyle as IStyle)?.Apply

Copy link
Member Author

Choose a reason for hiding this comment

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

No, it should NOT be

@rmarinho
Copy link
Member

Left a comment but non blocking
👍

@samhouts
Copy link
Member

👍

@samhouts samhouts merged commit 1a7a25f into master Apr 25, 2016
@samhouts samhouts deleted the multiClasses branch April 25, 2016 22:03
@samhouts samhouts added this to the 2.3.0 milestone Jun 27, 2018
mattleibow added a commit that referenced this pull request Jan 28, 2021
* Cleaned up the API a bit:
 - removed unnecessary static fields
 - removed unnecessary braces
 - a few other tweaks to reduce weird line wraps

* Stopping the sensors requires that they are actually supported.

* Added hardware traits to the tests to avoid evil code

* Fixed some more tests.

* We need to keep a reference to the sensors.

* Android needs to filter some events to avoid large amounts of unnecessary updates

* Gyroscopes are not emulated for Android
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants