El código que tienes funciona muy bien, solo tienes un problema al hacer la simplificación del dividendo y el divisor.
En la simplificación debes reducir ambos números dividiéndolos por sus factores primos hasta que ambos sean primos entre si (que no tengan ningún factor primo en común).
Para esto en tu código verificas el resultado de modulo
, de ser 0 implica que la división es exacta y solo se muestra un resultado entero.
De no ser así deberás descomponer los números, para eso comienzas comprobando el modulo de ambos números empezando por el 2 hasta la mitad del dividendo, (podría ser mas optimo con la raíz cuadrada ya que el numero mas grande que puede dividir exactamente a un numero es menor o igual a su raíz cuadrada).
Cada vez que ambos módulos resulten en 0, divides el numero sobre el factor actual. Luego se va incrementando dicho factor.
Al final para mostrar el numero mixto puedes verificar si la parte entera es mayor a 0, de no ser así no hay necesidad de mostrarla.
return `${numeral > 0 ? numeral+' ' : ''}${modulo}/${s[1]}`
El código queda de la siguiente forma:
function convertirFraccion(s) {
s = s.split('/').map(a => Number(a))
let modulo = s[0] % s[1]
let resto = modulo / s[1]
let numeral = Math.floor(s[0] / s[1])
if(modulo!=0){
act=2;
while(act<s[1]/2){
if(modulo%act==0 && s[1]%act==0){
modulo/=act;
s[1]/=act;
}
act++;
}
return `${numeral > 0 ? numeral+' ' : ''}${modulo}/${s[1]}`
}
else
return numeral
}
//ejemplos
console.log(convertirFraccion('24/9'));
console.log(convertirFraccion('4/3'));
console.log(convertirFraccion('28/7'));
console.log(convertirFraccion('2/36'));
console.log(convertirFraccion('53/83'));
Esta es solo una forma en bruto de resolverlo ,considero que puede optimizarse un poco mas, saludos.