Page list: Display a <span/> instead of a <a/> if the page is trashed

[millerf]

Changes proposed in this Pull Request

• Page list: Display a un-clickable text (a ) when the page item is deleted,. to prevent a 404

Testing instructions

• Go to page list
• Either when searching or in the "deleted" tab, page items cannot be clicked

Fixes #41640
cc @kwight

[gwwar]

Thanks for looking into this @millerf ✨ ! This PR tests well for me, but I did notice that the posts list takes a slightly different approach with keeping the link, but not setting an href. I don't have strong feelings on a span vs empty link, but would prefer if we could align the approach for both sections. Let us know which approach you prefer!

[millerf]

@gwwar Done. I display a <span/>. I am not a great fan of <a/> with empty links...

As a side note, merging <PostItem/> and. <Page/> might be to consider as they do pretty much the same thing and are now visually equivalent...

[mmtr]

Thanks @millerf!