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

RadioButtons cleanup #500

Merged
merged 6 commits into from
Jul 14, 2020
Merged

RadioButtons cleanup #500

merged 6 commits into from
Jul 14, 2020

Conversation

stmoy
Copy link
Contributor

@stmoy stmoy commented Jul 11, 2020

Description

This change further delineates the samples for RadioButton vs. RadioButtons. Previously they shared an icon, were found in the same category, etc. - this change moves RadioButtons to Layouts and updates the wording for both samples.

Motivation and Context

RadioButton and RadioButtons are related, but conceptually different. This change further differentiates the two samples.

How Has This Been Tested?

Manual testing.

Screenshots (if appropriate):

image

image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@stmoy stmoy requested a review from YuliKl July 11, 2020 04:22
@stmoy
Copy link
Contributor Author

stmoy commented Jul 11, 2020

CC: @chingucoding

@stmoy
Copy link
Contributor Author

stmoy commented Jul 14, 2020

I'm planning on pulling this in early afternoon today - @chingucoding, are you ok with the PR now that I've made some changes?

Copy link
Collaborator

@marcelwgn marcelwgn left a comment

Choose a reason for hiding this comment

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

Yes sure, the changes look fine to me!

@stmoy stmoy merged commit 7acd3e2 into master Jul 14, 2020
bpulliam pushed a commit that referenced this pull request Feb 1, 2022
* Clarify the distinction between RadioButton and RadioButtons by moving RadioButtons into the Layouts section and clarify wording

* Remove unnecessary RelativePanel, update wording in json file

* Clean up RadioButton sample.

* Adding RadioButtons icon to differentiate from RadioButton

* Add sample code substitution for SelectedIndex on RadioButtons page.

* Responding to PR feedback
@bpulliam bpulliam deleted the w/stmoy/radioButtonGroupCleanup branch May 10, 2022 17:53
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