ziad
ziad
@dotnet-policy-service agree
Applied your suggestion, thanks @hishamco
I removed `shape.Attributes.Remove("href")` because when a pager item is disabled, we now skip rendering the `` tag entirely. Since there’s no `` in that case, there’s no need to remove...
Hi @hishamco, just wanted to follow up on your last comment. Since the tag is no longer rendered for disabled items, I removed the shape.Attributes.Remove("href") call as it’s not needed...
As shown in the GIF below, the pager now behaves correctly:  - On the **first page**, the “Previous” and “First” links are disabled without rendering invalid `` tags.
Hi @hishamco, @Piedone — just wanted to follow up to see if there’s anything else needed from my side to move this PR forward. Thanks again for the review and...
Hi @hishamco, It’s been a while since the last update, and I noticed the PR has already passed all checks and received approvals. Could you please let me know if...