Occhio ai segni di interpunzione.

Dentro i parametri di window.open ci vanno solo le virgole per separare gli argomenti (e togli anche gli spazi).

Per il colore non puoi definirlo cosi`: ti manca una concatenazione tra due stringhe:
window.open('modules/form2.php?act=color&colore='+document ...

Pero` io farei in due passaggi:

<form name=fromA>
<select id="colorselect"
onchange="var cc = this.options[this.selectedIndex].value;
window.open('modules/form2.php?act=color&colore='+cc, 'popup', 'width=550,height=250,scrollbars=no,resizable=no') ;">
</form>


Ciao
Michele