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 EllipseDiameterScale (issue #2204) #2382

Merged
merged 1 commit into from
Feb 29, 2016
Merged

Added EllipseDiameterScale (issue #2204) #2382

merged 1 commit into from
Feb 29, 2016

Conversation

petvetbr
Copy link
Contributor

Added EllipseDiameterScale (issue #2204)

Added EllipseDiameterScale property as a multiplier for Ellipse Size, below the scaling of 1.0 (default), 0.5 (smaller) and 1.5 larger values for this property are shown.

This solves issue #2204 as now it is possible do manually ajust the size of the Ellipse. In the control the Ellipse size is calculated using the formula below:

EllipseDiameter =(width / 8)*EllipseDiameterScale;

ring0_5
ring1
ring1_5

@punker76
Copy link
Member

@petvetbr Nice to set the ellipse size, but it solves not the issue "... the standard version has five dots instead of six and..."

@petvetbr
Copy link
Contributor Author

@punker76 The control will show 5 ellipse when it is set to the IsLarge=false property, otherwise it shows 6 ellipses as @dsaf mentioned in the original comment:

The number of dots is configurable via the IsLarge property, but the size of dots EllipseDiameter is unfortunately readonly.

@punker76
Copy link
Member

@petvetbr ah, sorry, you're right 🙈

@punker76 punker76 added this to the 1.3.0 milestone Feb 29, 2016
punker76 added a commit that referenced this pull request Feb 29, 2016
Added EllipseDiameterScale (issue #2204)
@punker76 punker76 merged commit 90f80e1 into MahApps:develop Feb 29, 2016
@punker76
Copy link
Member

@petvetbr 👍

@petvetbr
Copy link
Contributor Author

@punker76 Glad to help. I´m with some spare time this week, so if there is any issue that you want me to look, just ask.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants