DeepBlueCLtd / LegacyMan

Legacy content for Field Service Manual
https://deepbluecltd.github.io/LegacyMan/index.html
Apache License 2.0
2 stars 0 forks source link

Also need quick links on ImageLinksTable page #593

Closed IanMayo closed 10 months ago

IanMayo commented 10 months ago

Some countries start with an ImageLinksTable page.

We need to support QuickLinksTable collections on these pages.

(I'll create some mock data, and push to develop) No need, Britain1.html has an image links table already.

robintw commented 10 months ago

I haven't come across an ImageLinksTable before. Is this just an additional QuickLinks table, but it happens that all of the links have images as well? Do we just want to extract the hrefs from the links in it, and put them in the related-links section, as we do with the QuickLinksTable links?

IanMayo commented 10 months ago

I haven't come across an ImageLinksTable before.

Oh, I didn't realise. For some nations, where there are a lot of records of data, the country page is collection of floating images, one per category - and these link to the relevant category page.

But, it's a nightmare to process these floating images. So, I created an ImageLinksTable in each page, that presents the images/links in an easy to find, easy to process table.

The processing for the ImageLinksTable works fine, it's just that also need to handle QuickLinksTable if it appears on these pages. We just need to create the related-links like we do for other QuickLinksTable instances.

robintw commented 10 months ago

Ah, that makes sense. I'll get on that this evening.