Usas comillas dobles tanto para el html como para el JS. Eso causa un escape y la palabra "mundo" queda como referencia en vez de string.
O usa comillas simples, o escapas las dobles (recomiendo lo primero). Ese problema no lo tienes con el 123 porque no lo pasas como string.
Usar onclick ya se considera mala practica por lo general. Usa eventos en puro javascript. Igualmente, a no ser que sea 100% necesario, el javascript debe ir al final del body, no en el head.
Esta seria la solución a tu problema:
<input type="button" value="Hola mundo" onclick="HolaMundo('mundo')">
Esto seria usando puro javascript para los eventos:
<input type="button" value="Hola mundo" id="buttonHello">
document.getElementById('buttonHello').addEventListener('click', function() {
HolaMundo('mundo');
});
Todo combinado:
<input type="button" value="Hola mundo" id="buttonHello">
function HolaMundo(parametroMundo) {
var variableHola = "Hola";
alert(variableHola + " " + parametroMundo);
}
document.getElementById('buttonHello').addEventListener('click', function() {
HolaMundo('mundo');
});
https://developer.mozilla.org/es/docs/Web/API/Document/getElementByIdhttps://developer.mozilla.org/es/docs/Web/API/EventTarget/addEventListenerTe recomiendo que revises algun tutorial de javascript ACTUALIZADO (maximo hace 3 años).
Saludos