Итак, на данный момент вы возвращаете функцию, которую вы не вызываете, чтобы вызвать функцию, измените каждый из кликов следующим образом:
onclick="changeSize(12)()"
Это делает так, что changeSize (12) возвращает внутреннюю функцию, а затем, используя вторые скобки, вызывает внутреннюю функцию:)
РЕДАКТИРОВАТЬ: Как уже говорили другие, здесь не нужно возвращать внутреннюю функциюи если бы вместо этого содержимое функции changeSize () было что-то вроде
function changeSize(size) {
document.body.style.fontSize = size + 'px';
}
, это было бы гораздо более кратким