-
Notifications
You must be signed in to change notification settings - Fork 69
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
Feature/xpt2046 #916
Feature/xpt2046 #916
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sample needs to be cleaned up as well - folder names and such
public int X { get; set; } | ||
public int Y { get; set; } | ||
} | ||
|
||
/// <summary> | ||
/// Represents a circle in the user interface. | ||
/// </summary> | ||
public class Circle : ThemedControl | ||
{ | ||
private Color _foreColor; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you heard the boss - underscores need to go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doh! Muscle memory
@@ -7,12 +7,6 @@ | |||
<Nullable>enable</Nullable> | |||
</PropertyGroup> | |||
|
|||
<ItemGroup> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume this is intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. libmpsse.dll is no longer required for FT232, so it's just removing library cruft
Source/Meadow.Foundation.Peripherals/Sensors.Hid.Xpt2046/Driver/Sensors.Hid.Xpt2046.csproj
Outdated
Show resolved
Hide resolved
...and_Frameworks/Graphics.MicroLayout/Samples/Ili9488Charts_Sample/Ili9488Charts_Sample.csproj
Show resolved
Hide resolved
public int X { get; set; } | ||
public int Y { get; set; } | ||
} | ||
|
||
/// <summary> | ||
/// Represents a circle in the user interface. | ||
/// </summary> | ||
public class Circle : ThemedControl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering now if circle makes sense as a control ... we should talk about how we should handle drawing primitives
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have preferred an Elipse, but we don't have the ability to draw that. It matches with the Box control. But yes, we can definitely discuss.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nothing to block on of course
...ce/Meadow.Foundation.Libraries_and_Frameworks/Graphics.MicroLayout/Driver/Controls/Circle.cs
Outdated
Show resolved
Hide resolved
Source/Meadow.Foundation.Peripherals/Sensors.Hid.Xpt2046/Samples/Tsc2004_Sample/MeadowApp.cs
Outdated
Show resolved
Hide resolved
Source/Meadow.Foundation.Peripherals/Sensors.Hid.Xpt2046/Driver/Sensors.Hid.Xpt2046.csproj
Outdated
Show resolved
Hide resolved
Source/Meadow.Foundation.Peripherals/Sensors.Hid.Xpt2046/Driver/Sensors.Hid.Xpt2046.csproj
Outdated
Show resolved
Hide resolved
...adow.Foundation.Peripherals/Sensors.Hid.Xpt2046/Samples/Tsc2004_Sample/Tsc2004_Sample.csproj
Outdated
Show resolved
Hide resolved
Source/Meadow.Foundation.Peripherals/Sensors.Hid.Xpt2046/Driver/Sensors.Hid.Xpt2046.csproj
Show resolved
Hide resolved
public int X { get; set; } | ||
public int Y { get; set; } | ||
} | ||
|
||
/// <summary> | ||
/// Represents a circle in the user interface. | ||
/// </summary> | ||
public class Circle : ThemedControl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nothing to block on of course
No description provided.