Change Details

Let's not hardcode the skin module here: https://github.com/wikimedia/mediawiki/blob/73c20062e0d3e0dcd94d841baa7ee7e605a688c7/includes/installer/WebInstallerOutput.php#L154 The existing setup also double loads CSS. Instead let's use the hooks system to determine what module should be used (if any). In future when the skin system is in a better place we should be able to use any associated templates with the skin to render the associated HTML but that is out of scope for now.

Let's not hardcode the skin module here to avoid risking the desktop refresh from breaking the rendering of the installer: https://github.com/wikimedia/mediawiki/blob/73c20062e0d3e0dcd94d841baa7ee7e605a688c7/includes/installer/WebInstallerOutput.php#L154 The existing setup also double loads CSS. Instead let's use the hooks system to determine what module should be used (if any). In future when the skin system is in a better place we should be able to use any associated templates with the skin to render the associated HTML but that is out of scope for now.

Let's not hardcode the skin module here to avoid risking the desktop refresh from breaking the rendering of the installer: https://github.com/wikimedia/mediawiki/blob/73c20062e0d3e0dcd94d841baa7ee7e605a688c7/includes/installer/WebInstallerOutput.php#L154 The existing setup also double loads CSS. Instead let's use the hooks system to determine what module should be used (if any). In future when the skin system is in a better place we should be able to use any associated templates with the skin to render the associated HTML but that is out of scope for now.