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

Feature/xpt2046 #916

Merged
merged 12 commits into from
Mar 5, 2024
Merged

Feature/xpt2046 #916

merged 12 commits into from
Mar 5, 2024

Conversation

ctacke
Copy link
Contributor

@ctacke ctacke commented Mar 4, 2024

No description provided.

Copy link
Contributor

@adrianstevens adrianstevens left a 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;
Copy link
Contributor

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

Copy link
Contributor Author

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>
Copy link
Contributor

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?

Copy link
Contributor Author

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

public int X { get; set; }
public int Y { get; set; }
}

/// <summary>
/// Represents a circle in the user interface.
/// </summary>
public class Circle : ThemedControl
Copy link
Contributor

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

Copy link
Contributor Author

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.

Copy link
Contributor

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

public int X { get; set; }
public int Y { get; set; }
}

/// <summary>
/// Represents a circle in the user interface.
/// </summary>
public class Circle : ThemedControl
Copy link
Contributor

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

@adrianstevens adrianstevens merged commit 117c46c into develop Mar 5, 2024
1 check passed
@adrianstevens adrianstevens deleted the feature/xpt2046 branch March 5, 2024 16:36
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