richelbilderbeek / BrainWeaver

BrainWeaver, tool for teacher education
https://github.com/richelbilderbeek/tools
GNU General Public License v3.0
5 stars 1 forks source link

No terminal visible #311

Closed richelbilderbeek closed 6 years ago

richelbilderbeek commented 6 years ago

Feedback employer 2018-08-23:

Heb maar een commentaar-puntje: als ik nu de student of de assessment-module open dan verschijnt er nu een zwart pop-up-deelscherm en daar ‘bovenop’ het startmenu van de aangeklikte module. Het zwarte pop upscherm is dus de achtergrond van het startmenu.Dit zwarte scherm blijft steeds op je scherm staan als je werkt met de aangeklikte module. Het blijft steeds de achtergrond van alle schermen van de module. Op het zwarte scherm verschijnen in code de ingetypte instructies. Verder werkt alles zoals het hoort en kan ik geen bugs vinden (fantastisch!). Misschien is er een goed argument voor het zwarte scherm, maar voor mij hoeft dit niet. Lijkt mij alleen maar af te leiden.

richelbilderbeek commented 6 years ago

Ah, the employer sees the qDebug output in a terminal!

richelbilderbeek commented 6 years ago

Fixed!

richelbilderbeek commented 6 years ago

Apparently not fixed yet:

ik heb nog maar een opmerking: het zwarte schermpje waarover ik in mijn twee vorige mails rapporteerde (zie commentaar vorige versie) verschijnt nog steeds. Hier heb ik nog geen reactie van je op gehad.

richelbilderbeek commented 6 years ago

It is still there:

2 2

richelbilderbeek commented 6 years ago

Fixed by f455e47413c323380b9df306fe43e5ebdde7c44a

I also had to remove the qCritical calls :rainbow:

richelbilderbeek commented 6 years ago

Still present, no new screenshot though...

richelbilderbeek commented 6 years ago

Ran with one, no terminal to be seen. Need confirmation by employer.

richelbilderbeek commented 6 years ago

Already at startup, the terminal is created! Will need to Duck how to prevent that...

2 2