PHP Notice: Undefined property: Joomla\Component\Templates\Administrator\View\Template\HtmlView::$mediaFiles in \administrator\components\com_templates\tmpl\template\default.php on line 100

Warning: count(): Parameter must be an array or an object that implements Countable in \administrator\components\com_templates\tmpl\template\default.php on line 100

@Quy I cannot replicate your error report and there is nothing in this PR remotely related to the line mentioned

Sorry false alarm. Applied to v4.0. OK in v4.1.

I have tested this item ✅ successfully on b17288b

I want to test, but I can't find these switches. ?
How to do?

@sandewt They only display when you are working on a template override. So use the create override functionality and then select one of the newly created overrides and then you will see the switches in the top right

With a little trick I can make the buttons visible and get the following. The text is not inline with the buttons. This depends on the screen size.

@sandewt Please run npm i to compile the CSS changes or download/install the prebuilt package.

I am not in favour of this kind of switches. Switches should be for yes/no only (I know that we have several on other places, and i think it is always wrong).

Why not use a label? like "Hide original file: yes/no?"

I know that we have several on other places, and i think it is always wrong).

so propose a change. this pr is consistent with current usage

What is current usage?

What is current usage?

Verbs shoud be avoided in labels

Probably there is a reason for this rule, and maybe I am wrong with my my personal opinion, that switches should be used only for yes/no and lists for text-alternatives.

If you were German I could help very easily, we can make a noun from every verb :) "Versteckthaltung der Originaldatei"

I will start a discussion for that, because I think it is important. But not directly related to this PR.

I have tested this item ✅ successfully on 9b37b14

This solution is better than before.

RTC

Thx

thanks