Empty „see also” section

[Ryuno-Ki]

A few moments back, I noticed, that https://developer.mozilla.org/en-US/docs/Web/CSS/@media/forced-colors#See_also does not contain any values.

Looking into the repo, QuickLinks ought to be rendered there:

content/files/en-us/web/css/@media/forced-colors/index.html

Lines 118 to 120 in c79bf8d

	<h2 id="See_also">See also</h2>
	<p>{{QuickLinksWithSubpages("/en-US/docs/Web/CSS/@media/")}}</p>

Could it show a „nothing found” instead? (or hide the section altogether)

[vkWeb]

@Ryuno-Ki Interesting ticket!

I think we should hide the see also section if results are not found to {{QuickLinksWithSubpages("/en-US/docs/Web/CSS/@media/")}}.

@peterbe what do you say on this?

[Ryuno-Ki]

I agree.
However, from a templating perspective, that would require, that the macro would likely to be adjusted (e.g. return a list), so to run a check to decide, whether to include the headline or not.

If that's not possible (e.g. the macro returns HTML), it is in the responsibility to return something, that nothing was found (instead of an empty string).

[peterbe]

I think it's "foolish" to use a KS macro like that. Perhaps better to call it "overly complicated".
If the .../CSS/@media/forced-colors doesn't have a set of its own "See also" links, it shouldn't need to piggyback off another page. It leads to bugs/challenges like the one mentioned here in this issue where you can't programmatically know if it's going to yield anything. It also overly interconnects the documents. Now, this page has to assume that that other page has some juicy and good "See also" links. If it's really necessary, you can make it:

<h2>See also</h2>
<p>
  <a href="/en-US/docs/Web/CSS/@media#see_also">See the links for <code>@media</code></a>
</p>

or something like that.

[rachelandrew]

This doesn't appear to be the case anymore, the page in question has links in the See also section. So I'm going to close the issue, please re-open if you think I'm wrong.

If any of the other things about still require discussion, perhaps raise them in the Discussions section?

[Ryuno-Ki]

Fixed with #763 and #2819