Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Switched the icon docs page to a lazy loaded OcTable and added a filter field #1867

Merged
merged 1 commit into from
Jul 20, 2022

Conversation

kulmann
Copy link
Contributor

@kulmann kulmann commented Jan 11, 2022

Description

This PR switches the icon docs page to a lazy loaded OcTable and adds a filter field.

Motivation and Context

Make the icon docs page better usable.

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)
  • Technical debt
  • Tests

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation added/updated

@update-docs
Copy link

update-docs bot commented Jan 11, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@kulmann kulmann force-pushed the icon-docs-page branch 2 times, most recently from ba8bb96 to e14e813 Compare July 19, 2022 14:47
@kulmann kulmann marked this pull request as ready for review July 20, 2022 10:30
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@kulmann kulmann merged commit 4d48ead into master Jul 20, 2022
@delete-merged-branch delete-merged-branch bot deleted the icon-docs-page branch July 20, 2022 10:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants