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

Unnest treeview examples for consistency (issue 2418 task) #2517

Merged
merged 20 commits into from
Oct 27, 2022

Conversation

alflennik
Copy link
Contributor

This PR makes the file structure of all examples consistent by fixing the inconsistently nested examples in the treeview pattern directory.

See a comment in issue #2418 for more information about this task.

This PR is dependent on other PRs. Once PRs #2499 and #2451 have been merged into #2417, the base of this PR can be changed from move-examples-next to move-examples. I created the move-examples-next branch to make writing and reviewing this PR easier.

Copy link
Contributor

@mcking65 mcking65 left a comment

Choose a reason for hiding this comment

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

@alflennik, In treeview-1a.html and treeview-1b, the href of links to the design pattern and treeview-navigation.html need revision.

@alflennik
Copy link
Contributor Author

Thank you @mcking65, it seems in my haste to scan for links to treeview-1a and treeview-1b I neglected to fix all the links going in the other direction. I fixed the broken links you referred to, and I went through all the links in the treeview directory to make sure there weren't any others.

@mcking65 mcking65 changed the base branch from move-examples-next to move-examples October 27, 2022 06:23
@mcking65
Copy link
Contributor

Thank you @alflennik! I changed base, fixed conflicts, and fixed one more href that was easy to miss. This is good to go now. I'll merge.

@mcking65 mcking65 merged commit 07859e9 into move-examples Oct 27, 2022
@mcking65 mcking65 deleted the move-examples-treeview-1 branch October 27, 2022 06:55
@alflennik
Copy link
Contributor Author

Big thanks Matt for getting it over the finish line!

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