Skip to content

Add BulletedList, SiteMapPath, DataPager, and Table#331

Merged
csharpfritz merged 5 commits intoFritzAndFriends:devfrom
csharpfritz:feature-NewControls
Feb 2, 2026
Merged

Add BulletedList, SiteMapPath, DataPager, and Table#331
csharpfritz merged 5 commits intoFritzAndFriends:devfrom
csharpfritz:feature-NewControls

Conversation

@csharpfritz
Copy link
Collaborator

This pull request adds documentation and navigation support for the BulletedList, SiteMapPath, and DataPager controls, and updates the test suite and documentation navigation accordingly. The main focus is on improving coverage and discoverability of these controls in both the documentation and test infrastructure.

Key changes:

Documentation Additions:

  • Added comprehensive documentation for the BulletedList control, including Blazor and Web Forms usage, migration notes, and examples.
  • Added full documentation for the SiteMapPath navigation control, covering Blazor features, migration from Web Forms, and customization examples.
  • Added a detailed guide for the DataPager control, describing supported features, usage in Blazor, and migration notes.

Navigation and Index Updates:

  • Linked the new BulletedList and SiteMapPath documentation pages in the mkdocs.yml navigation file for easier access. [1] [2]
  • Updated the README.md to link to the BulletedList and SiteMapPath docs, and removed the unimplemented Substitution control from the list. [1] [2] [3]

Test Suite Enhancements:

  • Added test coverage for the /ControlSamples/BulletedList and /ControlSamples/Table sample pages in the Playwright-based test suite. [1] [2]

@csharpfritz
Copy link
Collaborator Author

@copilot We're missing documentation for some of the new components and links in README. Verify all documentation is completed for the new components

Comment on lines +89 to +94
foreach (var child in ChildNodes)
{
var found = child.FindByUrl(url);
if (found != null)
return found;
}

Check notice

Code scanning / CodeQL

Missed opportunity to use Select Note

This foreach loop immediately
maps its iteration variable to another variable
- consider mapping the sequence explicitly using '.Select(...)'.
@csharpfritz csharpfritz merged commit bf91856 into FritzAndFriends:dev Feb 2, 2026
4 checks passed
@csharpfritz csharpfritz deleted the feature-NewControls branch February 2, 2026 09:46
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